Migrate vitest from jest#38
Merged
Merged
Conversation
- Replace Jest configuration with Vitest configuration - Update test files to use Vitest globals (vi instead of jest) - Remove Jest dependencies and add Vitest dependency - Update test runner command in package.json - Adjust test utilities to use Vitest's timer mocking
- Move vitest config from root to config/vitest.config.ts for better organization - Remove @reduxjs/toolkit dependency as it's no longer used in tests - Update test to use redux directly instead of redux toolkit - Clean up package.json and lock file accordingly
Update import in test file to use Redux Toolkit's legacy_createStore alias to maintain compatibility while addressing deprecation warnings
Include source directory TypeScript files to ensure proper type checking during test compilation.
This reverts commit 80ad08f.
Delete the top-level vitest.config.js file as it only re-exports the config from the config directory. Update the test script to directly reference the config file. Remove unused testing dependencies (@testing-library/dom and @testing-library/user-event) from package.json and pnpm-lock.yaml.
Update the @types/node dependency to the latest version to incorporate the latest TypeScript definitions for Node.js, ensuring compatibility and access to new type features. The lock file has been regenerated accordingly.
The test tsconfig reference is removed from the main tsconfig.json as it is no longer needed. The test tsconfig now explicitly includes all test files and source files, ensuring proper compilation for tests without project references.
The jsdom package was removed as it is no longer needed for testing or development. This reduces the dependency footprint and lockfile complexity.
Collaborator
|
Thanks for the PR, I will merge |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.