Skip to content

Commit

Permalink
❄️ Disable flaky video player tests (#40114)
Browse files Browse the repository at this point in the history
  • Loading branch information
danielrozenberg authored Aug 12, 2024
1 parent d384cbb commit 19c14f4
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions test/integration/test-video-players-helper.js
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,8 @@ export function runVideoPlayerIntegrationTests(
});
});

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

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

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

return getVideoPlayer({
Expand Down Expand Up @@ -379,7 +381,8 @@ export function runVideoPlayerIntegrationTests(
this.timeout(timeout);

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

0 comments on commit 19c14f4

Please sign in to comment.