Closed Bug 1274580 Opened 3 years ago Closed 3 years ago

'./mach mochitest -f chrome' fails for Android

Categories

(Firefox for Android :: Testing, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: gbrown, Assigned: Pike)

References

Details

Attachments

(1 file)

$ ./mach mochitest -f chrome
Error running mach:

    ['mochitest', '-f', 'chrome']

The error occurred in code that was called by the mach command. This is either
a bug in the called code itself or in the way that mach is calling it.

You should consider filing a bug for this issue.

If filing a bug, please include the full output of mach, including this error
message.

The details of the failure are as follows:

Exception: A cross-directory support file path noted in a test manifest does not appear in any other manifest.
 "browser/base/content/test/general/readerModeArticle.html" must appear in another test manifest to specify an install for "!/browser/base/content/test/general/readerModeArticle.html".

  File "/home/gbrown/src/testing/mochitest/mach_commands.py", line 411, in run_mochitest_general
    driver.install_tests(tests)
  File "/home/gbrown/src/python/mozbuild/mozbuild/controller/building.py", line 675, in install_tests
    '_tests', test_objs)
  File "/home/gbrown/src/python/mozbuild/mozbuild/testing.py", line 484, in install_test_files
    _resolve_installs(install_info.deferred_installs, topobjdir, manifest)
  File "/home/gbrown/src/python/mozbuild/mozbuild/testing.py", line 424, in _resolve_installs
    'for "!/%s".' % (path, path))
See Also: → 1262006
Eitan -- It's not immediately obvious to me why the readerModeArticle.html reference was added in https://hg.mozilla.org/mozilla-central/rev/dbc96a62c1da; is it necessary?
Flags: needinfo?(eitan)
This is needed in order to pull in that file for the narrate tests. The narrate feature is not available on android ATM, and I am not sure what it would test there. That directory should probably be skipped in android builds.
Flags: needinfo?(eitan)
Unfortunately, the failure persists even if the narrate tests are skipped on android.

I don't understand the failure:

Exception: A cross-directory support file path noted in a test manifest does not appear in any other manifest.
 "browser/base/content/test/general/readerModeArticle.html" must appear in another test manifest to specify an install for "!/browser/base/content/test/general/readerModeArticle.html".

readerModeArticle.html does appear in https://hg.mozilla.org/mozilla-central/annotate/c67dc1f9fab8/browser/base/content/test/general/browser.ini#l507.

:cmanchester - Do you know what's happening here?
Flags: needinfo?(cmanchester)
I don't think skipping tests will do the trick. The browser.ini is loaded in any case. the entire directory should be excluded with moz.build.

I'm assuming the browser directory is not included in fennec.
The narrate module in toolkit might be used on Android
at some point, but currently isn't, and the tests also
depend on /browser/.

Review commit: https://reviewboard.mozilla.org/r/54626/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/54626/
Attachment #8755451 - Flags: review?(eitan)
I dove in to all-tests.json, and found that toolkit/components/narrate/test/browser.ini depends on 

  !/browser/base/content/test/general/readerModeArticle.html

and thus breaks Android.

Eitan, is this the fix you imagined? Are you the right one to review this?

Clearing the needinfo on cmanchester, as I got the tests to run locally (on a artifact build, but still).
Flags: needinfo?(cmanchester)
Filed bug 1275002 to investigate using ./mach for the automation test runs, to catch regressions like this before landing.
Comment on attachment 8755451 [details]
MozReview Request: bug 1274580, don't build narrate on Android, r?eeejay

https://reviewboard.mozilla.org/r/54626/#review51216

This looks right. I'm not a peer in that module, so my review might not count?
Attachment #8755451 - Flags: review?(eitan) → review+
Comment on attachment 8755451 [details]
MozReview Request: bug 1274580, don't build narrate on Android, r?eeejay

Let's get an additional review by Margaret.
Attachment #8755451 - Flags: review?(margaret.leibovic)
Comment on attachment 8755451 [details]
MozReview Request: bug 1274580, don't build narrate on Android, r?eeejay

https://reviewboard.mozilla.org/r/54626/#review52280

Rubber stamp! (If it works for you, it works for me)
Attachment #8755451 - Flags: review?(margaret.leibovic) → review+
Assignee: nobody → l10n
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/b71c1dcf988a
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
You need to log in before you can comment on or make changes to this bug.