-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Percy runner #923
Percy runner #923
Conversation
…isk miss that from some weird percy auto merge
dash/testing/browser.py
Outdated
@@ -456,6 +456,9 @@ def visit_and_snapshot(self, resource_path, hook_id): | |||
self.driver.get(self.server_url + resource_path) | |||
self.wait_for_element_by_id(hook_id) | |||
self.percy_snapshot(resource_path) | |||
assert not self.driver.find_elements_by_css_selector( | |||
"div.dash-debug-alert" | |||
), "devtools should not raise an error alert" |
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.
🏆
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.
though, might need an option to disable this in case someone wants to snapshot the errors
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.
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.
Your call if slash-flexibility #923 (comment) is important. Aside from that question this looks great. 💃
Make sure to go back and remove the ref to this branch from dash-docs after this is released.
…down-#808 sets autoComplete to off in Dropdown plotly#808
sets autoComplete to off in Dropdown #808
No description provided.