Repository navigation
docs: document test expectations (CONTRIBUTING + CLAUDE.md) - #35
Merged
Carl723000 merged 2 commits intoJun 15, 2026
Merged
Conversation
Replace the "no test suite yet" note with how the node:test harness works: where tests live (src/test/*.test.ts -> out/test/), how to run (npm test), what belongs here (pure logic) vs what needs @vscode/test-electron, and a pointer to the worked example. Depends on the test-framework PR (ClaudeCodeUsage#32) landing first. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- CLAUDE.md: the node:test suite now exists (was "planned, see ClaudeCodeUsage#25"). - CONTRIBUTING: spell out that src/test/ must stay flat, since the test script globs out/test/*.test.js a single level deep (kept single-level for Node-version portability) so nested files would be skipped. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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
2 participants
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.
Documents the test harness now that the project has one, so contributors know what's expected.
CONTRIBUTING.md— rewrites the Tests section:npm test) and that CI runs the same on every PR.src/test/*.test.ts→ compiled toout/test/), and that the directory must stay flat — the script globsout/test/*.test.jsa single level deep (kept single-level for Node-version portability), so nested files would be silently skipped.vscodeAPI code needs@vscode/test-electron, not set up yet.CLAUDE.md— updates the Tests convention from "anode:testsuite is planned (see #25)" to reflect that it now exists.This documents the framework added in #32, so please merge #32 first. Kept as a separate, discussion-light docs PR on purpose — single concern, can merge right after the framework lands. Happy to rebase if #32 changes anything material.
🤖 Generated with Claude Code