Skip to content

test: limit npm test to test files - #30

Open
johnjheejin wants to merge 1 commit into
companyjupiter:mainfrom
johnjheejin:test/limit-test-discovery
Open

test: limit npm test to test files#30
johnjheejin wants to merge 1 commit into
companyjupiter:mainfrom
johnjheejin:test/limit-test-discovery

Conversation

@johnjheejin

@johnjheejin johnjheejin commented Jul 26, 2026

Copy link
Copy Markdown

Summary

  • limit npm test discovery to test/**/*.test.mjs
  • stop configuration examples named *_test.mjs from being counted as test files

Root cause

The bare node --test command used default discovery rules, which also matched four configuration examples under configs/.

User impact

Test output now reports only the actual automated test suite and no longer executes configuration examples as test modules.

Latest-base update

  • rebased onto upstream main at f02a53e
  • verified the narrowed pattern against the expanded current test suite

Validation

  • npm ci --ignore-scripts --no-audit --no-fund
  • npm test (16 tests: 15 passed, 1 skipped because symlink creation is unavailable on Windows)
  • node --check quarkify.mjs
  • git diff --check

@johnjheejin
johnjheejin force-pushed the test/limit-test-discovery branch from e075e53 to 9a9a1e3 Compare August 3, 2026 02:44
@johnjheejin
johnjheejin marked this pull request as ready for review August 3, 2026 02:55
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