akvo / akvo-mis / #761

89%

main: 89%

Pull #279

coveralls-python

[#269] Trim the tenant scoping diff

Complexity-only pass over #279; no behaviour changes.

- The six tenant FKs were the same seven-line declaration copied out
  across three apps. They share a `tenant_fk()` helper now. The field it
  builds is identical, so `makemigrations --check` still detects nothing.

- The superuser root lookup built its fallback around an empty result
  rather than a missing tenant, so a tenanted user whose root was absent
  would run a second query for someone else's root. That row cannot be
  missing — registration creates it in the same transaction — so the
  branch keys off `tenant_id` directly and only ever runs one query.

- Dropped the backfill test asserting that a filtered update leaves
  non-matching rows alone, which tests Django rather than this code.
  Dropped the level assertion from the seeded-database registration
  test, since a 200 already means the level was created.

5705 of 6609 branches covered (86.32%)

Branch coverage included in aggregate %.

10788 of 12011 relevant lines covered (89.82%)

0.9 hits per line