Skip to content

Commit 0ff232c

Browse files
committed
injected: fixing flaky duckplayer tests
1 parent e424b1c commit 0ff232c

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

injected/integration-test/duckplayer-mobile-drawer.spec.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ import { DuckplayerOverlays } from './page-objects/duckplayer-overlays.js';
33

44
test.describe('Duck Player - Drawer UI variant', () => {
55
test.describe('Video Player overlays', () => {
6-
test("Selecting 'watch here' on mobile", async ({ page }, workerInfo) => {
6+
test("Selecting 'watch here' on mobile abc", async ({ page }, workerInfo) => {
77
const overlays = DuckplayerOverlays.create(page, workerInfo);
88
await overlays.reducedMotion();
99

@@ -15,6 +15,7 @@ test.describe('Duck Player - Drawer UI variant', () => {
1515
await overlays.gotoPlayerPage();
1616

1717
// watch here = overlays removed
18+
await overlays.mobile.drawerIsPresented();
1819
await overlays.mobile.choosesWatchHere();
1920
await overlays.mobile.overlayIsRemoved();
2021
await overlays.pixels.sendsPixels([
@@ -34,6 +35,7 @@ test.describe('Duck Player - Drawer UI variant', () => {
3435
await overlays.gotoPlayerPage();
3536

3637
// watch here = overlays removed
38+
await overlays.mobile.drawerIsPresented();
3739
await overlays.mobile.selectsRemember();
3840
await overlays.mobile.choosesWatchHere();
3941
await overlays.mobile.overlayIsRemoved();
@@ -54,6 +56,7 @@ test.describe('Duck Player - Drawer UI variant', () => {
5456
await overlays.userSettingIs('always ask');
5557
await overlays.gotoPlayerPage();
5658

59+
await overlays.mobile.drawerIsPresented();
5760
await overlays.mobile.choosesDuckPlayer();
5861
await overlays.pixels.sendsPixels([
5962
{ pixelName: 'overlay', params: {} },
@@ -72,6 +75,7 @@ test.describe('Duck Player - Drawer UI variant', () => {
7275
await overlays.userSettingIs('always ask');
7376
await overlays.gotoPlayerPage();
7477

78+
await overlays.mobile.drawerIsPresented();
7579
await overlays.mobile.selectsRemember();
7680
await overlays.mobile.choosesDuckPlayer();
7781
await overlays.pixels.sendsPixels([

injected/integration-test/page-objects/duckplayer-overlays.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -542,6 +542,10 @@ class DuckplayerOverlaysMobile {
542542
this.overlays = overlays;
543543
}
544544

545+
async drawerIsPresented() {
546+
const { page } = this.overlays;
547+
await page.locator('ddg-video-drawer-mobile').waitFor({ state: 'visible', timeout: 2000 });
548+
}
545549
async choosesWatchHere() {
546550
const { page } = this.overlays;
547551
await page.getByRole('button', { name: 'No Thanks' }).click();

0 commit comments

Comments
 (0)