ThePalaceProject / web-patron / 28798829674

89%

push

github

Clean up test output for expected errors/warnings (PP-4714) (#155)

## Description

- Adds a shared `expectAndSuppressConsole` test helper that spies on
`console.warn`/`console.error`, suppresses messages matching expected
patterns, and passes everything else through — so unexpected output
(e.g., mismatched error messages or test code errors) remains visible.
- Refactors inline per-test spy setup/teardown into describe-level
`beforeEach`/`afterEach` blocks across several test files.
- Patterns at each call site include the full expected message content
(including error object strings), serving as inline documentation of
what output is expected.

## Motivation and Context

Tests exercising error-handling paths were emitting expected
`console.warn`/`console.error` output with multi-line stack traces on
every run, making it harder to spot real failures. Blanket suppression
was not acceptable because some console output (like React `act()`
warnings) reflects genuine test quality issues.

[Jira PP-4714]

## How Has This Been Tested?

Changes are limited to test infrastructure — no production code was
modified. The existing test suite validates that affected code paths
still produce expected output (spy-call assertions remain where tests
verify the warning was actually produced).

## Checklist:

- N/A - I have updated the documentation accordingly.
- [x] All new and existing tests passed.

225 of 260 branches covered (86.54%)

Branch coverage included in aggregate %.

383 of 427 relevant lines covered (89.7%)

29.93 hits per line