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-bind: Move integration test from test-bind-impl to test-bind-integrations #8010
Conversation
90e06e0
to
8ab922d
Compare
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.
Thanks for following up on this.
expect(vid.getAttribute('src')).to | ||
.equal('https://www.google.com/unbound.webm'); | ||
}); | ||
}); |
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.
Why remove this test?
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.
I'm not sure. Might have been a badly-resolved merge conflict? I've reverted this.
bd35b0d
to
b9c6004
Compare
@choumx good to go? |
@@ -64,6 +64,39 @@ describe.configure().retryOnSaucelabs().run('amp-bind', function() { | |||
waitForEvent('amp:bind:setState')); | |||
} | |||
|
|||
function waitForAllMutations() { |
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.
Nit: Add /** @return {!Promise} */
Moved an integration test to the new test-bind-integrations test.
/to @choumx @jridgewell