-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
fix(svelte): default mount log to true #23771
Conversation
Thanks for taking the time to open a PR!
|
Test summaryRun details
View run in Cypress Dashboard ➡️ Flakiness
This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
@@ -76,7 +76,7 @@ export function mount<T extends SvelteComponent> ( | |||
// by waiting, we are delaying test execution for the next tick of event loop | |||
// and letting hooks and component lifecycle methods to execute mount | |||
return cy.wait(0, { log: false }).then(() => { | |||
if (options.log) { | |||
if (options.log !== false) { |
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.
Nice, all our adapters do this - this could be part of GA, the standard API that adapters must comply with. Part of that could be logging the component name, and we could enforce it via an interface or something.
.within(() => | ||
cy | ||
.get(".command-name-mount") | ||
.should("contain", "mount<Counter ... />") |
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.
Maybe it would be helpful for future us to add a comment detailing the fact there is no space here purposefully, but I'm not too concerned with it.
Released in This comment thread has been locked. If you are still experiencing this issue after upgrading to |
User facing changelog
Logs the Svelte mount command by default
Additional details
The mount log should default to true, but we missed the fallback. Found when pairing with @rockindahizzy (thanks Rocky!)
Steps to test
Throw a
.only
on svelte component testing and runyarn workspace @tooling/system-tests test component_testing_spec.ts --no-exit --headed
, or test locally against a Svelte app. Check the reporter for amount <MyComponent ...>
log.How has the user experience changed?
Before:
After:
PR Tasks
cypress-documentation
?type definitions
?