Skip to content

test: Migrate to Vitest#894

Merged
timfish merged 16 commits intomainfrom
timfish/test/vitest
Feb 26, 2026
Merged

test: Migrate to Vitest#894
timfish merged 16 commits intomainfrom
timfish/test/vitest

Conversation

@timfish
Copy link
Collaborator

@timfish timfish commented Feb 26, 2026

This PR:

  • Migrates all tests to Vitest
  • 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 26, 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).


New Features ✨

  • Use crypto.randomUUID rather than uuid by timfish in #892

Other

  • test: Migrate to Vitest by timfish in #894

🤖 This preview updates automatically when you update the PR.

cursor[bot]

This comment was marked as outdated.

@timfish
Copy link
Collaborator Author

timfish commented Feb 26, 2026

Ah, just found that the e2e tests are disabled completely!

if:
(github.event_name != 'pull_request' || github.event.pull_request.head.repo.full_name == github.repository) &&
github.actor != 'dependabot[bot]' &&
false # Debug ID backend changes broke the E2E tests, we need to revisit them

@timfish timfish enabled auto-merge (squash) February 26, 2026 12:44
@timfish timfish merged commit f551c70 into main Feb 26, 2026
24 checks passed
@timfish timfish deleted the timfish/test/vitest branch February 26, 2026 12:44
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.

2 participants