Skip to content

Commit

Permalink
WIP Reenable flaky video test
Browse files Browse the repository at this point in the history
  • Loading branch information
powerivq committed Oct 4, 2024
1 parent 2223b4a commit d12fdce
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions test/integration/test-video-players-helper.js
Original file line number Diff line number Diff line change
Expand Up @@ -179,8 +179,7 @@ export function runVideoPlayerIntegrationTests(
});
});

// TODO(#40115): fix this flaky test
it.skip('should trigger pause analytics when the video pauses', function () {
it('should trigger pause analytics when the video pauses', function () {
let pauseButton;

return getVideoPlayer({
Expand All @@ -202,7 +201,6 @@ export function runVideoPlayerIntegrationTests(
});
});

// TODO(#40115): fix this flaky test
it.skip('should trigger session analytics when a session ends', function () {
let pauseButton;

Expand Down Expand Up @@ -381,7 +379,6 @@ export function runVideoPlayerIntegrationTests(
this.timeout(timeout);

describe('play/pause', () => {
// TODO(#40115): fix this flaky test
it.skip('should play when in view port initially', () => {
return getVideoPlayer({outsideView: false, autoplay: true}).then(
(r) => {
Expand Down

0 comments on commit d12fdce

Please sign in to comment.