From beea88d125d2e40e8f137cf32ffdf16f74811827 Mon Sep 17 00:00:00 2001
From: Shihua Zheng <powerivq@users.noreply.github.com>
Date: Tue, 29 Oct 2024 03:44:48 +0800
Subject: [PATCH] WIP Reenable flaky video test (#40175)

---
 test/integration/test-video-players-helper.js | 13 +++++--------
 test/integration/test-video-players.js        |  6 +++---
 2 files changed, 8 insertions(+), 11 deletions(-)

diff --git a/test/integration/test-video-players-helper.js b/test/integration/test-video-players-helper.js
index e31678359f51..ef1d8f24e5d8 100644
--- a/test/integration/test-video-players-helper.js
+++ b/test/integration/test-video-players-helper.js
@@ -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({
@@ -202,8 +201,7 @@ export function runVideoPlayerIntegrationTests(
           });
       });
 
-      // TODO(#40115): fix this flaky test
-      it.skip('should trigger session analytics when a session ends', function () {
+      it('should trigger session analytics when a session ends', function () {
         let pauseButton;
 
         return getVideoPlayer({
@@ -381,8 +379,7 @@ export function runVideoPlayerIntegrationTests(
     this.timeout(timeout);
 
     describe('play/pause', () => {
-      // TODO(#40115): fix this flaky test
-      it.skip('should play when in view port initially', () => {
+      it('should play when in view port initially', () => {
         return getVideoPlayer({outsideView: false, autoplay: true}).then(
           (r) => {
             return listenOncePromise(r.video, VideoEvents_Enum.PLAYING);
@@ -593,8 +590,8 @@ export function runVideoPlayerIntegrationTests(
         video.setAttribute('id', 'myVideo');
         video.setAttribute('controls', '');
         video.setAttribute('layout', 'fixed');
-        video.setAttribute('width', '300px');
-        video.setAttribute('height', '50vh');
+        video.setAttribute('width', '300');
+        video.setAttribute('height', '300');
 
         video.style.position = 'absolute';
         video.style.top = top;
diff --git a/test/integration/test-video-players.js b/test/integration/test-video-players.js
index af88ec44bbb4..9fd1a07f6f64 100644
--- a/test/integration/test-video-players.js
+++ b/test/integration/test-video-players.js
@@ -34,7 +34,8 @@ describes.sandboxed('amp-video-iframe', {}, (env) => {
   );
 });
 
-describes.sandboxed('amp-youtube', {}, (env) => {
+// TODO(#40181): Youtube tests not working on Circle.
+describes.sandboxed.skip('amp-youtube', {}, (env) => {
   runVideoPlayerIntegrationTests(
     env,
     (fixture) => {
@@ -60,8 +61,7 @@ describes.sandboxed('amp-dailymotion', {}, (env) => {
   );
 });
 
-// TODO(#39857): Unskip when integration is fixed.
-describes.sandboxed.skip('amp-3q-player', {}, (env) => {
+describes.sandboxed('amp-3q-player', {}, (env) => {
   runVideoPlayerIntegrationTests(
     env,
     (fixture) => {