Better lease renewal decisions

Fixes #223 (closed)

Created by: hacklschorsch

Review: Approved

LGTM!

I find the code itself good to understand and pythonic; I can't make much sense of the tests.

Maybe it's the combination of testtools with hypothesis. I fail to keep up with the many indirections.

Created by: codecov[bot]

Codecov Report

Merging #252 (3563169a) into main (d59a0b0a) will decrease coverage by 0.03%. The diff coverage is 85.41%.

@@            Coverage Diff             @@
##             main     #252      +/-   ##
==========================================
- Coverage   94.28%   94.24%   -0.04%     
==========================================
  Files          49       49              
  Lines        4096     4120      +24     
  Branches      517      526       +9     
==========================================
+ Hits         3862     3883      +21     
- Misses        191      194       +3     
  Partials       43       43              

Impacted Files	Coverage Δ
src/_zkapauthorizer/_storage_server.py	92.17% <75.00%> (+0.17%)
...rc/_zkapauthorizer/tests/test_lease_maintenance.py	95.74% <83.87%> (-0.26%)
src/_zkapauthorizer/lease_maintenance.py	94.69% <100.00%> (+0.85%)
src/_zkapauthorizer/tests/matchers.py	97.91% <100.00%> (+0.24%)
src/_zkapauthorizer/resource.py	98.75% <0.00%> (-1.25%)
src/_zkapauthorizer/tests/test_model.py	92.44% <0.00%> (-0.45%)

---

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 d59a0b0...3563169. Read the comment docs.