-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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-story-player] ✅ Add e2e tests #31716
Conversation
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.
🔥 🚒
); | ||
}); | ||
|
||
it('when player becomes visible, third story starts preloading', async () => { |
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.
Hopefully we'll make this test fail soon and only prerender N+1 :))
c405f62
to
aa6f874
Compare
Owners check bot bump |
aa6f874
to
a83f435
Compare
onwers bot bump |
a83f435
to
f354051
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.
Owners bump @Enriqe
#31276
Adds e2e tests for the player.