test(readStateManager): consolidate suite 6929 → 3785 lines via harness + table-driven mirrors

Browse Source

Same 88 behavioral witnesses, same mutation coverage. All 100 tests pass.
Zero production-file changes; no sibling test files touched.

Approach:
- Shared scenario harness (makeFakeTimers, makeFakeRelay, makeFenceHandle,
  makeLocalStorage, wireDrainCallback) replaces 82 per-test manager
  constructions
- Table-driven mirror pairs collapse drain_crashAfter* read/unread symmetric
  pairs and incomplete-verdict cases into parameterized test() rows
- Comment pruning: one-line intent header retained per test; narrated
  step-by-step comments removed

Co-authored-by: Will Pfleger <pfleger.will@gmail.com>
Signed-off-by: Will Pfleger <pfleger.will@gmail.com>

...

This commit is contained in:

Duncan

2026-08-10 12:52:29 -04:00

parent f8a1a76c70

commit 18bdeec8e6

1 changed files with 543 additions and 3687 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

4230

desktop/src/features/channels/readState/readStateManager.test.mjs

File diff suppressed because it is too large Load Diff