Skip to content

Conversation

@mikepenz
Copy link
Owner

This pull request updates all test files to use the vitest testing framework instead of jest. The changes ensure consistency across the test suite and remove unnecessary timeout settings. The most important changes are grouped below:

Test framework migration:

Test style and cleanup:

  • Removed unnecessary jest.setTimeout calls from all test files, as vitest handles timeouts differently. (All references above)
  • Updated test function names from it to test for consistency with vitest conventions. (All references above)
  • Minor formatting fixes for code style and consistency, such as quote usage and whitespace. (__tests__/main.test.ts [1] [2]

These changes modernize the test suite and make it compatible with the vitest framework, improving maintainability and consistency.

 @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
 ```
@mikepenz mikepenz requested a review from Copilot August 16, 2025 12:06
@mikepenz mikepenz added the test label Aug 16, 2025
@github-actions
Copy link

github-actions bot commented Aug 16, 2025

TestsPassed ✅SkippedFailed
JUnit Test Report66 ran66 passed0 skipped0 failed
TestResult
No test annotations available

Copy link

Copilot AI left a 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 expect and test from 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 or learn how to get started.

@mikepenz mikepenz merged commit 47f1220 into develop Aug 16, 2025
7 checks passed
@mikepenz mikepenz deleted the feature/vitest branch August 16, 2025 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant