Repository navigation
test: group test files by feature - #1030
Merged
Merged
Conversation
Merge the ~200 per-fix test files into a few files per feature area. Source-reading cases move to <feature>.checks.* files, separate from behavior tests. No assertion changed: the same 1752 test names pass.
…iles AGENTS.md gains a 'Where a test goes' section: tests are grouped by feature, a new case joins its feature's file, and a new file is for a new feature or an isolated area. Comments, docs and package.json scripts that named the old per-fix files now name the file the case moved to.
This was referenced Oct 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
scripts/ had ~200 test files, one per fix. This merges them into a few files per feature area (tabs, navigation, windows, documents, editor, editing, preview, export, layout, shortcuts, settings, release):
<feature>.test.ts/<feature>.spec.tsfor behavior tests,<feature>.checks.*for the cases that read source as text, and<feature>.<area>.*where a part must run in its own process (module mocks, process-wide shims). 203 files become 58.No assertion changed and no test changed runner. Each old file's cases sit in a
describewith their setup moved into hooks, so one file's globals do not leak into another's. AGENTS.md now says where a new test goes, and comments, docs and thetest:frontmatter/test:workflowsscripts name the merged files.Verified: the sorted list of test names reported by
node --testand vitest is identical before and after (1752 names), all pass;vitest --sequence.shufflegreen; svelte-check 0 errors; cargo test green.