akvo / akvo-mis / #763

89%

main: 89%

Pull #281

coveralls-python

[#270] Trim the read-filtering diff

Complexity-only pass; no behaviour changes.

- TenantManager was hand-written delegation of a single queryset method.
  models.Manager.from_queryset does exactly that, so the class collapses
  to one line. The soft-deletes and draft managers keep their explicit
  delegators: their get_queryset carries with_deleted/only_draft state
  that a generated manager would drop.

- The two-tenant fixture was copy-pasted across four isolation test
  modules, differing only in a trailing datapoint or assignment. It
  moves to a TenantIsolationTestCase base that each app extends by
  overriding make_tenant.

- fetchLevels expressed "use the array or an empty one" through two
  separate store writes. A two-argument then collapses both paths into
  one update; the rejection path is kept because it clears the previous
  tenant's levels on a refetch.

5714 of 6609 branches covered (86.46%)

Branch coverage included in aggregate %.

10862 of 12067 relevant lines covered (90.01%)

0.9 hits per line