Skip to content

test: Migrate to Vitest#882

Open
timfish wants to merge 3 commits intogetsentry:mainfrom
timfish:timfish/test/vitest
Open

test: Migrate to Vitest#882
timfish wants to merge 3 commits intogetsentry:mainfrom
timfish:timfish/test/vitest

Conversation

@timfish
Copy link
Collaborator

@timfish timfish commented Feb 19, 2026

This PR:

  • Migrates all tests to Vitest apart from the integration tests.
    • This allows us to retain testing on older Node versions and delay introducing breaking changes for now.
  • I had to change the CI cache keys because changing a package.json resulted in no lock change and a build failure
  • Test usages of the Vite plugin had to add as unknown as vite.Plugin because the Vite versions varied between the plugin and integration tests. This does not impact end users because it will use their Vite version.

@github-actions
Copy link

github-actions bot commented Feb 19, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


Bug Fixes 🐛

  • (webpack) Deduplicate webpack deploys by chargome in #875

Internal Changes 🔧

  • Migrate to oxfmt by timfish in #880
  • Build with Rolldown by timfish in #872
  • Remove unplugin by timfish in #876
  • Rollup/Vite no longer uses unplugin by timfish in #858
  • Esbuild no longer uses unplugin by timfish in #871
  • Webpack no longer uses unplugin by timfish in #870

Other

  • test: Migrate to Vitest by timfish in #882

🤖 This preview updates automatically when you update the PR.

@timfish timfish force-pushed the timfish/test/vitest branch 3 times, most recently from 951a498 to abbdcc7 Compare February 19, 2026 14:33
@timfish timfish force-pushed the timfish/test/vitest branch from b215b75 to 844a6b8 Compare February 19, 2026 15:09
@timfish timfish marked this pull request as ready for review February 19, 2026 16:41
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.

1 participant

Comments