Skip to content

fix(config): consolidate to single vitest.config.ts and remove vitest.config.mts (#523) - #525

Open
lmduy2612 wants to merge 1 commit into
ChainLearnOfficial:mainfrom
lmduy2612:fix/consolidate-vitest-config-523
Open

lmduy2612 wants to merge 1 commit into
ChainLearnOfficial:mainfrom
lmduy2612:fix/consolidate-vitest-config-523

Conversation

@lmduy2612

Copy link
Copy Markdown

Summary

Closes #523.

Consolidates the test configuration into a single canonical vitest.config.ts file and removes the duplicate vitest.config.mts and its orphaned setup file src/test/setup.ts.

Changes Made

  • Removed redundant vitest.config.mts from project root.
  • Removed unused src/test/setup.ts which was only referenced by vitest.config.mts.
  • Standardized both local and CI test execution on the primary vitest.config.ts containing the full test suite configuration, jsdom environment, aliases, and coverage setup.

Acceptance Criteria

  • Consolidate to a single vitest config file (vitest.config.ts)
  • CI and local test runs use the same config
  • Remove the unused config file (vitest.config.mts)

@netlify

netlify Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for chainlearn failed.

Name Link
🔨 Latest commit 10fc82b
🔍 Latest deploy log https://app.netlify.com/projects/chainlearn/deploys/6abf7073d0cf3000089b3a43

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.

Two vitest config files exist — vitest.config.ts and vitest.config.mts

1 participant