From 01a7a2c0f2f6dfc8e6411978af187001cded9ee1 Mon Sep 17 00:00:00 2001 From: AndiAJ Date: Mon, 24 Aug 2026 11:38:24 +0300 Subject: [PATCH] Fix hiddenVideoControlsContextMenuTest UI test --- .../assets/pages/noControlsVideoMediaPage.html | 9 +++++---- .../mozilla/reference/browser/ui/MediaPlaybackTest.kt | 2 +- .../mozilla/reference/browser/ui/robots/BrowserRobot.kt | 8 ++++++++ 3 files changed, 14 insertions(+), 5 deletions(-) diff --git a/app/src/androidTest/assets/pages/noControlsVideoMediaPage.html b/app/src/androidTest/assets/pages/noControlsVideoMediaPage.html index 639a3e90f9..27e941c141 100644 --- a/app/src/androidTest/assets/pages/noControlsVideoMediaPage.html +++ b/app/src/androidTest/assets/pages/noControlsVideoMediaPage.html @@ -1,13 +1,14 @@ - + + No_Controls_Video_Test_Page + - -

Page content: video player

+ - \ No newline at end of file + diff --git a/app/src/androidTest/java/org/mozilla/reference/browser/ui/MediaPlaybackTest.kt b/app/src/androidTest/java/org/mozilla/reference/browser/ui/MediaPlaybackTest.kt index a851861f6e..ca81e6290f 100644 --- a/app/src/androidTest/java/org/mozilla/reference/browser/ui/MediaPlaybackTest.kt +++ b/app/src/androidTest/java/org/mozilla/reference/browser/ui/MediaPlaybackTest.kt @@ -80,7 +80,7 @@ class MediaPlaybackTest { navigationToolbar {} .enterUrlAndEnterToBrowser(noControlsVideoTestPage.url) { - longClickMatchingText("test_link_video") + longClickVideoWithNoControls() verifyNoControlsVideoContextMenuItems() } } diff --git a/app/src/androidTest/java/org/mozilla/reference/browser/ui/robots/BrowserRobot.kt b/app/src/androidTest/java/org/mozilla/reference/browser/ui/robots/BrowserRobot.kt index 34bd4e4c43..2cbc50ab2c 100644 --- a/app/src/androidTest/java/org/mozilla/reference/browser/ui/robots/BrowserRobot.kt +++ b/app/src/androidTest/java/org/mozilla/reference/browser/ui/robots/BrowserRobot.kt @@ -72,6 +72,14 @@ class BrowserRobot { link.click(LONG_CLICK_DURATION) } + fun longClickVideoWithNoControls() { + mDevice.waitForWindowUpdate(packageName, waitingTime) + mDevice.findObject(UiSelector().resourceId("$packageName:id/engineView")).waitForExists(waitingTime) + mDevice.waitForIdle() + val engineView = mDevice.findObject(By.res("$packageName:id/engineView")) + engineView.click(engineView.visibleCenter, LONG_CLICK_DURATION) + } + fun longClickAndCopyText( expectedText: String, selectAll: Boolean = false,