Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork114
Replace Jest with Vitest#1472
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
@eslint/js ^9.31.0 → ^9.33.0 @types/node ^24.0.13 → ^24.3.0 @typescript-eslint/eslint-plugin ^8.36.0 → ^8.39.1 @typescript-eslint/parser ^8.36.0 → ^8.39.1 eslint ^9.31.0 → ^9.33.0 eslint-plugin-prettier ^5.5.1 → ^5.5.4 jest ^30.0.4 → ^30.0.5 jest-circus ^30.0.4 → ^30.0.5 ts-jest ^29.4.0 → ^29.4.1 typescript ^5.8.3 → ^5.9.2 ```
github-actionsbot commentedAug 16, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
| ||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This pull request migrates the test framework from Jest to Vitest across the entire test suite. The change modernizes the testing infrastructure by replacing Jest-specific imports and configurations with Vitest equivalents.
Key changes include:
- Replaced all Jest imports with Vitest in test files, updating to use
expectandtestfrom Vitest - Removed Jest timeout configurations (
jest.setTimeout) since Vitest handles timeouts differently - Updated package.json to use Vitest commands and dependencies instead of Jest
Reviewed Changes
Copilot reviewed 14 out of 17 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| package.json | Updated test scripts to use Vitest, replaced Jest dependencies with Vitest, and updated other dev dependencies |
| eslint.config.mjs | Removed Jest ESLint plugin and Jest globals configuration |
| All test files | Replaced Jest imports with Vitest, removed timeout configurations, and updated test function syntax |
Tip: Customize your code reviews with copilot-instructions.md.Create the file orlearn how to get started.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
47f1220 intodevelopUh oh!
There was an error while loading.Please reload this page.
This pull request updates all test files to use the
vitesttesting framework instead ofjest. The changes ensure consistency across the test suite and remove unnecessary timeout settings. The most important changes are grouped below:Test framework migration:
jestwithvitestin test files, updating usages toexpectandtestfromvitest. (__tests__/configuration.test.ts[1]__tests__/demo/demo.test.ts[2]__tests__/gitea/releaseNotesBuilderGitea.test.ts[3][4][5][6][7][8][9][10][11][12][13][14][15][16][17]__tests__/gitea/releaseNotesBuilderPullGitea.test.ts[18][19][20][21][22][23][24]__tests__/main.test.ts[25][26]__tests__/offline/offlineMode.test.ts[27]__tests__/regexUtils.test.ts[28][29][30][31]Test style and cleanup:
jest.setTimeoutcalls from all test files, asvitesthandles timeouts differently. (All references above)ittotestfor consistency withvitestconventions. (All references above)__tests__/main.test.ts[1][2]These changes modernize the test suite and make it compatible with the
vitestframework, improving maintainability and consistency.