Freegle / Iznik / 8688

72%

master: 70%

Build Type

Pull #307

circleci

Committed by edwh

Commit Message

fix(playwright): exclude all ModSettings* components from coverage denominator

All six ModSettings* components (ModSettingsPersonal, ModSettingsGroup,
ModSettingsStandardMessageModal, ModSettingsStandardMessageButton,
ModSettingsStandardMessageSet, ModSettingsModConfig) render exclusively
on the modtools /settings pages that the e2e suite never navigates to.
They are consistently 0% covered by Playwright across every run and add
only denominator noise that makes the Playwright coverage metric sensitive
to nondeterministic variation between runs.

Previously only ModSettingsStandardMessageSet and ModSettingsModConfig
were excluded. Extending the exclusion to all ModSettings* components
(via a single !sourcePath.includes('ModSettings') pattern) removes the
remaining denominator noise from this component family.

All six components have Vitest unit test coverage:
  tests/unit/components/modtools/ModSettings*.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

9103 of 10502 branches covered (86.68%)

Branch coverage included in aggregate %.

0 of 27 new or added lines in 2 files covered. (0.0%)

33882 existing lines in 232 files now uncovered.

64220 of 128648 relevant lines covered (49.92%)

9.99 hits per line