1812692 - Add in Device Level Screenshot Testing for GeckoView Junit Tests

Status: RESOLVED FIXED

(GeckoView :: General, enhancement, P1)

Description

[Olivia Hall [:olivia]]

Device level screenshots (that would capture the full device screen) could be beneficial when testing with GeckoView junit.

In bug 1804467, I tried adding a test in PrintDelegateTest#printPreviewRendered to test that the device print dialog opened by taking a screenshot. However, using UIAutomation to takeScreenshot seemed to cause test harness side-effects that showed up as TextInputDelegateTest timeouts. (UIAutomation was possibly interfering with the test harness' AccessibilityService?) Reproducible when running TextInputDelegateTest after PrintDelegateTest in the same test harness with UIAutomation access.

Had similar issues trying to take a screenshot using:

• InstrumentationRegistry.getInstrumentation().uiAutomation.takeScreenshot()
• Screenshot.capture() import androidx.test.runner.screenshot.Screenshot
• UIDevice.takeScreenshot() import androidx.test.uiautomator.UiDevice Dependency: androidTestImplementation 'androidx.test.uiautomator:uiautomator:2.2.0'

Goal of this bug is to find a side-effect free way to take screenshots of the entire device screen for Android junit tests and reenable/reconfigure the screenshot test in PrintDelegateTest.

Comment 1

[Olivia Hall [:olivia]]

This bug is impacting this test, printPreviewRendered, and will also impact other new printing tests that use screenshots for bug 1659818. (Bug will prevent the tests from running on CI.)

Comment 2

[Olivia Hall [:olivia]]

Attachment: Bug 1812692 - TextInputDelegateTest Timeouts r=m_kato

Reducing the number of characters in the TextInputDelegateTest content
large test prevents timeouts from occurring when using UIAutomator.
UIAutomator tests will be reactivated in bug 1659818.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1cc8016e85b7
TextInputDelegateTest Timeouts r=m_kato,geckoview-reviewers

Comment 4

[Serban Stanca [:SerbanS]]

https://hg.mozilla.org/mozilla-central/rev/1cc8016e85b7

Comment 5

[Chris Peterson [:cpeterson]]

No need to uplift this test fix to 111.