Skip to content

Commit b83a2cf

Browse files
docs: clarify npm test scope in contributing guide (#430)
## Summary\n- Clarify that `npm test` runs the fast suite only\n- Document that integration tests are excluded from `npm test`\n- Add the correct command for integration tests: `npm run test:slow`\n\n## Why\nThe previous wording said `npm test` runs all tests in `tests/**/*.test.js`, which is inconsistent with the actual npm script in `package.json`. This update aligns CONTRIBUTING docs with real behavior to reduce contributor confusion.\n\n## Local validation\n- `npm ci --ignore-scripts`\n- `npx mocha tests/model-selection.test.js` (10 passing) --------- Co-authored-by: haosenwang1018 <haosenwang1018@users.noreply.github.com> Co-authored-by: Tom Dupuis <60640908+tomdps@users.noreply.github.com>
1 parent 915766e commit b83a2cf

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

CONTRIBUTING.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,8 @@ zeroshot/
122122
npm test
123123
```
124124

125-
Runs all tests in `tests/**/*.test.js` using Mocha.
125+
Runs the fast test suite (unit + top-level tests) as defined in `package.json`.
126+
Integration tests in `tests/integration/**/*.test.js` are excluded from `npm test`; run them with `npm run test:slow`.
126127

127128
### Specific Test File
128129

0 commit comments

Comments
 (0)