Skip to content

Exclude benchmark tests from default test run#114

Merged
jimmytacks merged 2 commits into
mainfrom
experiment/112-fix-benchmark-ci
Apr 2, 2026
Merged

Exclude benchmark tests from default test run#114
jimmytacks merged 2 commits into
mainfrom
experiment/112-fix-benchmark-ci

Conversation

@jimmytacks
Copy link
Copy Markdown
Collaborator

Summary

  • Adds test.exclude config to vite.config.ts to skip *benchmark* files during npm test
  • Benchmarks can still be run on demand: npx vitest run src/engine/__tests__/sweep-benchmark.test.ts
  • Reduces CI test time from ~123s to under 1s

Closes #112

jimmytacks and others added 2 commits April 2, 2026 19:36
Adds test.exclude config to vite.config.ts to skip *benchmark* files
during npm test. Benchmarks can still be run directly via:
  npx vitest run src/engine/__tests__/sweep-benchmark.test.ts

Closes #112

Co-Authored-By: Claude Opus 4.6 <[email protected]>
Fixes TS2769 build error — tsc needs the vitest type reference
to recognise the test property in defineConfig.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@jimmytacks jimmytacks merged commit bdabcb4 into main Apr 2, 2026
1 check passed
@jimmytacks jimmytacks deleted the experiment/112-fix-benchmark-ci branch April 2, 2026 18:38
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.

Fix long-running sweep-benchmark test blocking CI

1 participant