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
✅ amp-image-slider test speedup #17392
Conversation
66958b4
to
5e03e42
Compare
So there's good news and bad news. 👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there. 😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request. Note to project maintainer: This is a terminal state, meaning the |
5e03e42
to
51adbda
Compare
CLAs look good, thanks! |
@@ -89,7 +89,7 @@ config.run('amp-image-slider', function() { | |||
// layoutCallback is called |
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.
Suggest monitoring master red for flakes here and see if there are any platforms (e.g. Chrome on Windows) that need to be skipped due to the lower timeout.
2518b96
to
acd00d0
Compare
acd00d0
to
efd81c8
Compare
* Speed up tests by using poll with timeout * More lenient timeout with better error reporting
poll
with timeout to replacetimeout
, ensures fasterresolve
on action complete