mirrors/bun.sh
1
0
Fork 0
mirror of https://github.com/oven-sh/bun synced 2026-02-02 15:08:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(runner): filter out non-JS files from node tests (#26092)

Browse Source 
## Summary
- `isNodeTest()` was only checking if the path included the node test
directories but not verifying the file was actually a JavaScript file
- This caused `test/js/node/test/parallel/CLAUDE.md` to be incorrectly
treated as a test file
- Added `isJavaScript(path)` check to filter out non-JS files

## Test plan
- [x] Verify CLAUDE.md is no longer picked up as a test file

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Bot <claude-bot@bun.sh>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
robobun
2026-01-14 16:28:29 -08:00
committed by GitHub
parent b72af3d329
commit 05df51ff84
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
scripts/runner.node.mjs
View File

@@ -1585,6 +1585,9 @@ function isNodeTest(path) {
if (isCI && isMacOS && isX64) { if (isCI && isMacOS && isX64) {
return false; return false;
} }
if (!isJavaScript(path)) {
return false;
}
const unixPath = path.replaceAll(sep, "/"); const unixPath = path.replaceAll(sep, "/");
return ( return (
unixPath.includes("js/node/test/parallel/") || unixPath.includes("js/node/test/parallel/") ||