Closed
Bug 983662
Opened 8 years ago
Closed 7 years ago
Media mochitests not running on emulator-jb due to manifest problems
Categories
(Testing :: Mochitest, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ahal, Assigned: ahal)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
987 bytes,
patch
|
armenzg
:
review+
|
Details | Diff | Splinter Review |
7.73 KB,
patch
|
jgriffin
:
review+
|
Details | Diff | Splinter Review |
On cedar the emulator-jb mochitests are failing. I investigated and found three separate manifest related problems. The first was a bad merge that caused the build to stop packaging the manifest, this is already fixed in bug 974069. The second is a wrong path being passed into mozharness, this will require a buildbot-configs change. The third is due to runtestsb2g.py not running methods in the right order (said methods just randomly modify state that affects one another in non-obvious way). Patches coming soon.
Assignee | ||
Comment 1•8 years ago
|
||
The problem here was that the path needs to be relative to the test harness (not the mozharness script). We should probably make our harnesses more robust when it comes to allowing relative paths.
Attachment #8391215 -
Flags: review?(armenzg)
Assignee | ||
Comment 2•8 years ago
|
||
The problem here was that we were calling buildURLOpts before buildTestPath, which obviously is a big no no.
Attachment #8391221 -
Flags: review?(jgriffin)
Updated•8 years ago
|
Attachment #8391215 -
Flags: review?(armenzg) → review+
Updated•8 years ago
|
Attachment #8391215 -
Attachment description: Fix erroneous manifest path → [checked-in] Fix erroneous manifest path
Comment 3•8 years ago
|
||
Live.
Updated•8 years ago
|
Attachment #8391221 -
Flags: review?(jgriffin) → review+
Assignee | ||
Comment 4•7 years ago
|
||
I don't believe this is relevant anymore. Please re-open if I'm wrong.
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•