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 flakes by swapping setInterval with setTimeout #16194
Conversation
hmmm, what is the root cause of |
I can only intermittently repro on the Windows Chrome platform because it's flaky : / |
return poll(`wait for attribute ${attribute} on ${element.tagName}`, () => { | ||
return element.getAttribute(attribute); | ||
}, undefined, 8000); | ||
function waitForAttribute(sandbox, element, attribute) { |
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.
Since it's an integration test, I don't know if we want a stub...
68ef479
to
5446958
Compare
@cathyxz @aghassemi PTAL : ) |
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.
yay
Flaking on master : ( |
* Revert "Shows hint overlay on up swipe like version 1.0 (#16258)" This reverts commit 56566b4. * Revert "Validator rollup (#16266)" This reverts commit 66d09a2. * Revert "throws error when using invalid component type in the config. (#16231)" This reverts commit 3693815. * Revert "Get rid of isRelayoutNeeded and refactor to onMeasureChanged (#16259)" This reverts commit 18c1db9. * Revert "Fix amp-pan-zoom zoom target bug and relayout bug (#16257)" This reverts commit 5d6d391. * Revert "Fix flakes by swapping setInterval with setTimeout (#16194)" This reverts commit 53de1af.
Fixes #16179