1581814 - Re-enable mobile/android/components/extensions/test/mochitest/test_ext_tabs_executeScript_bad.html

Status: RESOLVED FIXED

(WebExtensions :: Android, task, P3)

Description

[Rob Wu [:robwu]]

I just ran the test locally, and it completed successfully without errors.

To resolve this bug, simply remove https://searchfox.org/mozilla-central/rev/7ed8e2d3d1d7a1464ba42763a33fd2e60efcaedc/mobile/android/components/extensions/test/mochitest/mochitest.ini#27

Comment 1

[Rob Wu [:robwu]]

The fix is easy, so this is a very good candidate for people who want to get familiar with contributing to extension code on Android.

To get started, see https://wiki.mozilla.org/WebExtensions/Contribution_Onramp

Then start the Android emulator with ./mach android-emulator

To check whether the patch works, run the following command (don't forget to set the MOZCONFIG environment variable if needed):

./mach mochitest mobile/android/components/extensions/test/mochitest/

Without a fix, the output will be:

 4:27.28 TEST_START: mobile/android/components/extensions/test/mochitest/test_ext_tabs_executeScript_bad.html
 4:27.28 TEST_END: SKIP

When the test is re-enabled, the test should run to completion.

Comment 2

[Harsh]

Hi,
Can I solve this if it's still open
Thanks

Comment 3

[Rob Wu [:robwu]]

Hi Harsh, this issue is still open.

You have to set up a development environment to run the test in an Android emulator, and confirm that the test passes after fixing the bug. See comment 1 for more details.

Comment 4

[Myeongjun Go]

Hello Rob :)
Could I ask you about android build?
I want to build android and run command (i.e, ./mach android-emulator).
Should I refer this page(Build Using Android Studio section)[0] for solving what I want?

[0]
https://mozilla.github.io/geckoview/contributor/geckoview-quick-start

Comment 5

[Myeongjun Go]

The reason why I ask this is because there is an up-to-date issue in the document and an error occurs when I follow the manual.

When performing the following manuals:
If the gradle sync does not automatically start, select File > Sync Project with Gradle Files.

The following error occurs:
ERROR: Building with Gradle is only supported for Fennec, i.e., MOZ_BUILD_APP == 'mobile/android'.

Comment 6

[Rob Wu [:robwu]]

I have never tried to build with Android studio, only the command line, like this:

./mach bootstrap
./mach build

Are you running into problems when you try to build Android? If so, please file a new bug in the "Firefox Build System" product, specifically the "Android Studio and Gradle integration" component. Here is the list of open bugs. If you prefer to chat (outside of Bugzilla), join us in the #addons room at chat.mozilla.org.

Comment 7

[Harsh]

Attachment: Bug 1581814 - Re-enable mobile/android/components/extensions/test/mochitest/test_ext_tabs_executeScript_bad.html . r=robwu

Comment 8

[Shivam Singhal [ :championshuttler ]]

Attachment: Bug 1581814 - Re-enable mobile/android/components/extensions/test/mochitest/test_ext_tabs_executeScript_bad.html.r?robwu

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d2b292c1241c
Re-enable mobile/android/components/extensions/test/mochitest/test_ext_tabs_executeScript_bad.html.r=robwu

Comment 10

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/d2b292c1241c