Skip to content

feat(core): Add critial e2e tests file #31242

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ndcunningham
Copy link
Contributor

This PR adds a new “essential” (aka golden) test file to most packages, think of it as a quick health check that covers each plugin’s absolute must-have functionality.
These tests are kept deliberately small, just enough to prove that the basics still work. If one of these ever fails, it’s a showstopper: we’ll fix it right away to keep our plugins in good shape.

While running the updated e2e tests, we spotted a missing test dependency in the React plugin’s generated library.
This change also brings in @testing-library/dom for React, so you can run end-to-end and unit tests straight out of the box.

Next Steps

  • Enforce these rules for E2E Nightly (report code-owners if tests are failing)
  • Add essential tests to more plugins.

@ndcunningham ndcunningham self-assigned this May 16, 2025
@ndcunningham ndcunningham requested review from a team as code owners May 16, 2025 05:11
@ndcunningham ndcunningham requested review from JamesHenry and jaysoo May 16, 2025 05:12
Copy link

vercel bot commented May 16, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview May 16, 2025 6:01am

Copy link

nx-cloud bot commented May 16, 2025

View your CI Pipeline Execution ↗ for commit 4efdc34.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 24m 55s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 21s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 4s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 2s View ↗
nx documentation ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-16 06:28:32 UTC

@ndcunningham ndcunningham force-pushed the feat/add-e2e-golden-tests branch from 1ecf4b3 to 4161227 Compare May 16, 2025 05:16
@ndcunningham ndcunningham marked this pull request as draft May 16, 2025 05:30
fix(react): add missing testing dependency for libraries

update
@ndcunningham ndcunningham force-pushed the feat/add-e2e-golden-tests branch from 4161227 to 4efdc34 Compare May 16, 2025 05:58
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