Closed
Bug 987398
Opened 11 years ago
Closed 11 years ago
mach mochitest commands should pass TestManifest into mochitest runner
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(firefox30 fixed, firefox31 fixed)
RESOLVED
FIXED
mozilla31
People
(Reporter: gps, Assigned: gps)
References
Details
Attachments
(2 files, 1 obsolete file)
(deleted),
patch
|
ahal
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
ahal
:
review+
|
Details | Diff | Splinter Review |
THe mach mochitest commands should be passing a TestManifest instance into the mochitest Python runner like the xpcshell do.
Will have a patch up in a moment.
Assignee | ||
Comment 1•11 years ago
|
||
The mochitest mach commands now consult the all-tests.json file for
resolving the test argument/path. The mochitest runner has been modified
to accept a manifestdestiny.TestManifest instance instead of a path to a
manifest file. The mach commands construct an in-memory TestManifest and
pass it to the mochitest runner.
While I was here, I cleaned up some old JSON serializing code.
Attachment #8395997 -
Flags: review?(ted)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → gps
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•11 years ago
|
||
Forgot to remove some now-irrelevant code.
Attachment #8396065 -
Flags: review?(ted)
Assignee | ||
Updated•11 years ago
|
Attachment #8395997 -
Attachment is obsolete: true
Attachment #8395997 -
Flags: review?(ted)
Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 8396065 [details] [diff] [review]
Run mochitests from manifests with mach
Ted is at a work week and I don't like lingering reviews :)
Attachment #8396065 -
Flags: review?(ted) → review?(ahalberstadt)
Assignee | ||
Comment 4•11 years ago
|
||
I learned that this patch may help the Addon Manager people run their bc tests that have multiple configurations from separate manifests!
However, when I learned of the existence of these multiple variations, I realized that this patch isn't complete.
While the in-memory manifest generated by mach and passed to the mochitest runner contains multiple instances of tests in toolkit/mozapps/extensions, the mochitest runner only runs each test once. I could try fixing that, but that would be more involved as it would change how tests are passed from Python to the browser (I /think/). I'm not familiar with how all this stuff works. I'm inclined to punt it to a follow-up or at least a part 2.
Assignee | ||
Comment 5•11 years ago
|
||
Looks like the multiple configs issue is with the manifest parser not knowing about install-to-subdir. See bug 977275 for the hack that is install-to-subdir. Essentially, we need to make the mochitest runner manifest aware all the way down. At that point, we can kill install-to-subdir. I believe that's the domain of bug 984670 though.
Assignee | ||
Comment 6•11 years ago
|
||
This is a gross hack but it makes both variants of the Addon Manager
tests runnable via mach.
Attachment #8396849 -
Flags: review?(ahalberstadt)
Updated•11 years ago
|
Attachment #8396065 -
Flags: review?(ahalberstadt) → review+
Updated•11 years ago
|
Attachment #8396849 -
Flags: review?(ahalberstadt) → review+
Assignee | ||
Comment 7•11 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/b151f7bd1c10
https://hg.mozilla.org/integration/fx-team/rev/dc348b447da7
Flags: in-testsuite+
Comment 8•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/b151f7bd1c10
https://hg.mozilla.org/mozilla-central/rev/dc348b447da7
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla31
Comment 9•11 years ago
|
||
Updated•11 years ago
|
status-firefox30:
--- → fixed
status-firefox31:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•