Skip to content

Coverage floor is 95; raise it back to 100 as the gap closes #8

Description

@ericof

Where it stands

fail_under in backend/pyproject.toml is 95, lowered from 100 so continuous integration has a gate it can pass while the gap is closed. 100% branch coverage is still the standard this package holds itself to, and still the target: an authentication plugin has no line that is fine to leave unexercised.

The numbers, measured rather than estimated

suite tests coverage uncovered
full 2606 99.15% 124 statements
make test 2568 97.52% 498 statements

The 374-statement difference is not untested code. Those statements are covered by the docker-marked tests, which make test-coverage does not run — they need containers and an image built from this tree.

Two separate pieces of work

  1. 124 statements genuinely uncovered. This is the real work, and what raising the floor back to 100 requires.

  2. 374 statements covered but unmeasured. Fixed by measuring the full suite, which needs the coverage job to build the demo image first. The Backend: Container integration job already does exactly that, so folding coverage into it is the cheap version. Kept separate for now, deliberately.

A floor set near the container-free number would have written off the 374 as well, which is why it is 95.

This is a ratchet

Raise it as (1) closes. It is not a setting to be left alone.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions