Closed
Bug 1347686
Opened 8 years ago
Closed 8 years ago
Running xpcshell via one-click loaner gives e10s error
Categories
(Firefox Build System :: Mach Core, enhancement)
Firefox Build System
Mach Core
Tracking
(firefox55 fixed)
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox55 | --- | fixed |
People
(Reporter: wlach, Assigned: wlach)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
When I try to run the xpcshell tests in a one click loaner, I get this weird error:
root@25dd0bbdc987:~# mach xpcshell-test devtools/server/tests/unit/test_eventlooplag_actor.js
Error running mach:
['xpcshell-test', 'devtools/server/tests/unit/test_eventlooplag_actor.js']
The error occurred in the implementation of the invoked mach command.
This should never occur and is likely a bug in the implementation of that
command. 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:
AttributeError: 'Namespace' object has no attribute 'e10s'
File "/home/worker/workspace/build/tests/xpcshell/mach_test_package_commands.py", line 65, in xpcshell
return run_xpcshell(self.context, **kwargs)
File "/home/worker/workspace/build/tests/xpcshell/mach_test_package_commands.py", line 25, in run_xpcshell
args.e10s = context.mozharness_config.get('e10s', args.e10s)
root@25dd0bbdc987:~# mach --e10s xpcshell-test devtools/server/tests/unit/test_eventlooplag_actor.js
usage: mach [global arguments] command [command arguments]
mach: error: unrecognized arguments: --e10s
It's unclear to me why e10s should even be an argument to the xpcshell tests? Doesn't that only make sense in the context of testing the full browser? Am I missing something?
Assignee | ||
Comment 1•8 years ago
|
||
Assignee | ||
Comment 2•8 years ago
|
||
Did more investigation, pretty certain the e10s option does absolutely nothing and can be taken out.
Assignee: nobody → wlachance
Comment hidden (mozreview-request) |
Comment 4•8 years ago
|
||
mozreview-review |
Comment on attachment 8847794 [details]
Bug 1347686 - Remove e10s argument from mach xpcshell test runner
https://reviewboard.mozilla.org/r/120712/#review122682
no e10s for xpcshell
Attachment #8847794 -
Flags: review?(jmaher) → review+
Pushed by wlachance@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3998666c1aa1
Remove e10s argument from mach xpcshell test runner r=jmaher
Comment 6•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•