Skip to content

fix(webpack): update e2e snapshot for vitest reportsDirectory change#34766

Merged
FrozenPandaz merged 1 commit intomasterfrom
fix-e2e-test
Mar 9, 2026
Merged

fix(webpack): update e2e snapshot for vitest reportsDirectory change#34766
FrozenPandaz merged 1 commit intomasterfrom
fix-e2e-test

Conversation

@FrozenPandaz
Copy link
Collaborator

Current Behavior

The webpack legacy e2e test (e2e-webpack:e2e-ci--src/webpack.legacy.test.ts) fails with a snapshot mismatch because the reportsDirectory value changed from ../coverage/app3224373 to coverage/app3224373.

Expected Behavior

The snapshot should match the new reportsDirectory path format introduced by #34720.

Related Issue(s)

Fixes the e2e test breakage introduced by #34720 (fix(vitest)!: resolve reportsDirectory against workspace root).

The vitest reportsDirectory is now resolved against workspace root
instead of project root, updating the snapshot to match.
@FrozenPandaz FrozenPandaz requested a review from a team as a code owner March 9, 2026 22:08
@FrozenPandaz FrozenPandaz requested a review from jaysoo March 9, 2026 22:08
@netlify
Copy link

netlify bot commented Mar 9, 2026

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit e9eba7e
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/69af44dcf49b6a000956defe
😎 Deploy Preview https://deploy-preview-34766--nx-dev.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.

@netlify
Copy link

netlify bot commented Mar 9, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit e9eba7e
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69af44dc2c9295000880fb6d
😎 Deploy Preview https://deploy-preview-34766--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.

@nx-cloud
Copy link
Contributor

nx-cloud bot commented Mar 9, 2026

View your CI Pipeline Execution ↗ for commit e9eba7e

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 8m 8s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 3m 29s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 8s 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 2026-03-09 22:21:00 UTC

@FrozenPandaz FrozenPandaz enabled auto-merge (squash) March 9, 2026 22:09
@FrozenPandaz FrozenPandaz merged commit 556ff2d into master Mar 9, 2026
24 checks passed
@FrozenPandaz FrozenPandaz deleted the fix-e2e-test branch March 9, 2026 22:21
@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 Mar 15, 2026
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