Skip to content

test: remove low-value coverage - #23

Merged
yyyyaaa merged 2 commits into
mainfrom
phat/audit-test-suite
Aug 17, 2026
Merged

test: remove low-value coverage#23
yyyyaaa merged 2 commits into
mainfrom
phat/audit-test-suite

Conversation

@yyyyaaa

@yyyyaaa yyyyaaa commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove tests that only prove static existence, typed structure, visual implementation, or mock behavior
  • consolidate broad registry, keybinding, component, and command tests around observable runtime contracts
  • replace the Sheets migration golden harness with focused resolver, selection, and cell-transformation tests
  • document every changed test file and the remaining gaps in TEST_SUITE_AUDIT.md

Result

  • normal suite: 185 → 157 test files and 1,491 → 1,157 cases
  • deleted 31 test files, added 3 focused contract files, and rewrote or consolidated 27 files
  • removed 334 Vitest/Node cases plus 6 Playwright cases
  • removed 9 obsolete snapshot, golden-data, and parity-harness artifacts

Verification

  • pnpm check
  • pnpm build:pages
  • pnpm --filter blocks test:e2e — 25 passed
  • pnpm pack:local

Nx marked @constructive-io/registry:test as flaky during the full check, although all 17 registry tests passed.

@yyyyaaa
yyyyaaa merged commit 29c1e12 into main Aug 17, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant