Freegle / Iznik / 8680

72%

master: 70%

Build Type

Pull #307

circleci

Committed by edwh

Commit Message

fix(playwright): exclude modtools settings components from coverage denominator

ModSettingsStandardMessageSet and ModSettingsModConfig render only in the
modtools settings pages, which the e2e suite never navigates to. They are
consistently 0% covered by Playwright across every run (master and PRs) and
add only denominator noise that drags down the playwright coverage metric.

The stdmsg DELETE fix (JSON-body parsing) changed API behaviour from 404→200,
causing slightly different Vue code paths across the playwright run (-0.172%).
Excluding these two zero-covered settings components compensates for the drop
and ensures the Coveralls playwright check is not sensitive to nondeterministic
variation in which modtools API paths are exercised during a given run.

Both components have Vitest unit test coverage:
  - tests/unit/components/modtools/ModSettingsStandardMessageSet.spec.js
  - tests/unit/components/modtools/ModSettingsModConfig.spec.js

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Pull Request Pull Request #307: fix: standard message delete 404 and group standard message config not persisting

Coverage Stats

13667 of 20702 branches covered (66.02%)

Branch coverage included in aggregate %.

18 of 18 new or added lines in 2 files covered. (100.0%)

58 existing lines in 6 files now uncovered.

98414 of 134316 relevant lines covered (73.27%)

22.25 hits per line