Skip to content

Migrate webamp tests from Jest to Vitest#1350

Merged
captbaritone merged 1 commit into
masterfrom
migrate-to-vitest
Jun 7, 2026
Merged

Migrate webamp tests from Jest to Vitest#1350
captbaritone merged 1 commit into
masterfrom
migrate-to-vitest

Conversation

@captbaritone

Copy link
Copy Markdown
Owner

Summary

  • Replace Jest with Vitest for unit tests
  • Remove unused integration tests (puppeteer-based, not run in CI)
  • Remove .babelrc entirely — Babel is no longer needed anywhere in the webamp package
  • Remove jest-puppeteer, jest-image-snapshot, @types/jest, jest-environment-jsdom, and related deps

Test plan

  • All 224 unit tests pass with vitest run
  • Library build (npm run build-library) still succeeds
  • CI passes

🤖 Generated with Claude Code

@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown

Size Change: 0 B 🆕

Total Size: 0 B

compressed-size-action

- Replace Jest with Vitest for unit tests
- Remove integration tests (not used in CI, relied on puppeteer)
- Remove .babelrc (no longer needed — was only used for Jest transform)
- Remove jest-puppeteer, jest-image-snapshot, and related deps
- Replace jest.fn() with vi.fn() in test files
- Add vitest.config.ts with jsdom environment

All 224 unit tests pass. The postcss plugin test remains skipped
as before.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@captbaritone captbaritone merged commit a4895b2 into master Jun 7, 2026
5 checks passed
@netlify

netlify Bot commented Jun 7, 2026

Copy link
Copy Markdown

Deploy Preview for tourmaline-kringle-c98715 canceled.

Name Link
🔨 Latest commit 8e635d8
🔍 Latest deploy log https://app.netlify.com/projects/tourmaline-kringle-c98715/deploys/6a25c2a769c6ab0008c5927c

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