-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Test run all specs ct #33173
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
base: develop
Are you sure you want to change the base?
Test run all specs ct #33173
Conversation
…636-run-all-specs-for-component-testing
…636-run-all-specs-for-component-testing
…636-run-all-specs-for-component-testing
…ew projects, run-all-specs-ct-vite and run-all-specs-ct-webpack to test experimentalRunAllSpecs for component testing
…636-run-all-specs-for-component-testing
| <body> | ||
| <script> | ||
| window.__RUN_MODE_SPECS__ = ${JSON.stringify(this.ctx.project.specs)} | ||
| <!-- __RUN_ALL_SPECS__ is only set if experimentalRunAllSpecs is set to true --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
HTML comment inside script tag causes syntax error
An HTML comment (<!-- ... -->) is placed inside a <script> tag, but HTML comments are not valid JavaScript syntax. When the browser attempts to execute this script, it will throw a syntax error like "Unexpected token '<'", breaking the entire Cypress app initialization. The comment needs to use JavaScript syntax (// ...) instead of HTML comment syntax.
cypress
|
||||||||||||||||||||||||||||||||||||||||
| Project |
cypress
|
| Branch Review |
testRunAllSpecsCt
|
| Run status |
|
| Run duration | 18m 55s |
| Commit |
|
| Committer | Bill Glesias |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
|
|
0
|
|
|
15
|
|
|
1098
|
|
|
4
|
|
|
26988
|
| View all changes introduced in this branch ↗︎ | |
Warning
Partial Report: The results for the Application Quality reports may be incomplete.
UI Coverage
44.93%
|
|
|---|---|
|
|
196
|
|
|
164
|
Accessibility
97.94%
|
|
|---|---|
|
|
4 critical
8 serious
3 moderate
2 minor
|
|
|
102
|
eventually will close in favor of #32926
Additional details
Note
Makes
experimentalRunAllSpecsavailable for Component Testing and wires up dev servers and runtime to load multiple specs at once.experimentalRunAllSpecsin rootCoreConfigOptions; remove e2e-only validation/error and update typings/store logic to permit CTinitCypressTests.js): whenspecPathor spec is__all, import all selected specs via@fs; add comprehensive unit testsshouldLoadhandles__all; server socket updates CT dev-server to compile all specs when receivingRUN_ALL_SPECS_KEYHtmlDataSource: injectswindow.__RUN_ALL_SPECS__; add CT e2e/system tests and sample projects for Vite and Webpackwatchscript args)Written by Cursor Bugbot for commit 1c122f3. This will update automatically on new commits. Configure here.
Steps to test
How has the user experience changed?
PR Tasks
cypress-documentation?type definitions?