Null locks are ignored

[ewjoachim]

Closes #400

Successful PR Checklist:

• Tests < I'd like if we could add some tests, but not sure which
  • (not applicable?)
• Documentation
  • (not applicable?)
• If Add list_locks #401 if merged before, we may need to change a thing or two

[ewjoachim]

@elemoine I'd love your input on this one someday :)

[ewjoachim]

I think this will work with our deprecation policy but another brain validating that would be great :D

[elemoine]

Yes, it will work. The new SQL function has the same arguments signature as the old SQL function, so it is compatible with the Python code of previous Procrastinate versions.

[codecov]

Codecov Report

Merging #402 (195d8a7) into master (e9fc5e6) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #402      +/-   ##
==========================================
- Coverage   99.87%   99.87%   -0.01%     
==========================================
  Files          28       28              
  Lines        1612     1611       -1     
  Branches      175      175              
==========================================
- Hits         1610     1609       -1     
  Misses          1        1              
  Partials        1        1              

Impacted Files	Coverage Δ
procrastinate/manager.py	100.00% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e9fc5e6...195d8a7. Read the comment docs.

[ewjoachim]

🤙 @elemoine

[elemoine]

This looks good to me! Thank you again @ewjoachim.

[ewjoachim]

No idea how to test this cleanly though.

[ewjoachim]

Brain switched on. Tests added.

[github-actions]

Coverage report

The coverage rate went from 100.0% to 100.0% ➡️

The branch rate is 99.72999999999999%

100% of new lines are covered.