Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: migrate to vitest #34475

Merged
merged 12 commits into from
Feb 26, 2025
Merged

test: migrate to vitest #34475

merged 12 commits into from
Feb 26, 2025

Conversation

viceice
Copy link
Member

@viceice viceice commented Feb 26, 2025

Changes

Migrate from jest to vitest. Added som compabillity functions to have less changes.
Will replace all jest.xxx calls in a followup PR.

Context

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

rarkins
rarkins previously approved these changes Feb 26, 2025
@viceice viceice requested a review from rarkins February 26, 2025 09:12
@viceice viceice enabled auto-merge February 26, 2025 09:13
viceice and others added 2 commits February 26, 2025 10:24
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
@viceice viceice requested a review from HonkingGoose February 26, 2025 09:24
@viceice viceice added this pull request to the merge queue Feb 26, 2025
Merged via the queue into main with commit 997c235 Feb 26, 2025
38 of 39 checks passed
@viceice viceice deleted the test/migrate-to-vitest branch February 26, 2025 09:43
@trim21
Copy link
Contributor

trim21 commented Feb 26, 2025

vitest now have built-in support for github actions reporter, doesn't really need vitest-github-actions-reporter anymore

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.

Try Vitest test runner on this repository
4 participants