Skip to content

fix(testing): ensure jest v30 migration is run#33916

Merged
FrozenPandaz merged 1 commit intomasterfrom
jest/re-trigger-jest-v30-migration
Dec 18, 2025
Merged

fix(testing): ensure jest v30 migration is run#33916
FrozenPandaz merged 1 commit intomasterfrom
jest/re-trigger-jest-v30-migration

Conversation

@leosvelperez
Copy link
Member

Current Behavior

The replace-removed-matcher-aliases migration from @nx/jest is not run when migrating to Angular v21 (and updating Jest to v30). That migration targets the original package update for Jest v30 (Nx 21.3.0), which was incompatible with Angular < 21, so it wouldn't have run for Angular workspaces at the time. Now that Angular is being updated to v21, Jest is updated to v30, but the migration generator is not running.

Expected Behavior

The replace-removed-matcher-aliases migration from @nx/jest should run when migrating to Angular v21 (and updating Jest to v30).

@leosvelperez leosvelperez self-assigned this Dec 18, 2025
@leosvelperez leosvelperez requested a review from a team as a code owner December 18, 2025 07:50
@vercel
Copy link

vercel bot commented Dec 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
nx-dev Ready Ready Preview Dec 18, 2025 7:51am

@nx-cloud
Copy link
Contributor

nx-cloud bot commented Dec 18, 2025

View your CI Pipeline Execution ↗ for commit 5cddeed

Command Status Duration Result
nx affected --targets=lint,test,test-kt,build,e... ✅ Succeeded 5m 27s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 2m 24s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 11s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-12-18 09:51:08 UTC

@netlify
Copy link

netlify bot commented Dec 18, 2025

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit 5cddeed
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/6943b24ed9d5d60008cf2efc
😎 Deploy Preview https://deploy-preview-33916--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@FrozenPandaz FrozenPandaz merged commit d9ca27b into master Dec 18, 2025
21 of 22 checks passed
@FrozenPandaz FrozenPandaz deleted the jest/re-trigger-jest-v30-migration branch December 18, 2025 15:06
jaysoo pushed a commit that referenced this pull request Dec 19, 2025
## Current Behavior

The `replace-removed-matcher-aliases` migration from `@nx/jest` is not
run when migrating to Angular v21 (and updating Jest to v30). That
migration targets the original package update for Jest v30 (Nx 21.3.0),
which was incompatible with Angular < 21, so it wouldn't have run for
Angular workspaces at the time. Now that Angular is being updated to
v21, Jest is updated to v30, but the migration generator is not running.

## Expected Behavior

The `replace-removed-matcher-aliases` migration from `@nx/jest` should
run when migrating to Angular v21 (and updating Jest to v30).
@github-actions
Copy link
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 24, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants