Closed
Bug 1141029
Opened 11 years ago
Closed 11 years ago
Parity disable B2G failing mochitests for Mulet on TaskCluster
Categories
(Firefox OS Graveyard :: Runtime, defect)
Tracking
(firefox39 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox39 | --- | fixed |
People
(Reporter: gerard-majax, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
7.50 KB,
patch
|
gerard-majax
:
review+
|
Details | Diff | Splinter Review |
We lack some parity on Mulet for B2G skipped tests.
Reporter | ||
Comment 1•11 years ago
|
||
Disabling a couple: https://treeherder.allizom.org/#/jobs?repo=try&revision=6ec85f75e472&exclusion_profile=false
We get M1 and M4 green
Reporter | ||
Comment 2•11 years ago
|
||
Reporter | ||
Comment 3•11 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #2)
> More: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d8a38d9b3f39
https://treeherder.allizom.org/#/jobs?repo=try&revision=d8a38d9b3f39&exclusion_profile=false
Reporter | ||
Comment 4•11 years ago
|
||
Reporter | ||
Comment 5•11 years ago
|
||
Reporter | ||
Comment 6•11 years ago
|
||
Reporter | ||
Comment 7•11 years ago
|
||
It looks like all other failures we can see on push linked on bug 1141027 are not tests already skipped on b2g desktop.
Reporter | ||
Comment 8•11 years ago
|
||
Comment on attachment 8575903 [details] [diff] [review]
Disabling mochitests on Mulet with parity to B2G Desktop for taskcluster r=...
Andrew, can you review this ? Thanks!
Attachment #8575903 -
Flags: review?(ahalberstadt)
Reporter | ||
Updated•11 years ago
|
Reporter | ||
Comment 9•11 years ago
|
||
I'm also wondering if the test skip at https://hg.mozilla.org/try/rev/54c3e520c7ce#l2.13 should not also be included in the present patch, since as far as I understand the condition, it's being skipped on b2g.
Flags: needinfo?(ahalberstadt)
Comment 10•11 years ago
|
||
Comment on attachment 8575903 [details] [diff] [review]
Disabling mochitests on Mulet with parity to B2G Desktop for taskcluster r=...
Review of attachment 8575903 [details] [diff] [review]:
-----------------------------------------------------------------
Lgtm!
Attachment #8575903 -
Flags: review?(ahalberstadt) → review+
Comment 11•11 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #9)
> I'm also wondering if the test skip at
> https://hg.mozilla.org/try/rev/54c3e520c7ce#l2.13 should not also be
> included in the present patch, since as far as I understand the condition,
> it's being skipped on b2g.
I'm not quite sure what the question is. But in general just because it is skipped on B2G doesn't mean it isn't supposed to run there. When we first stood up tests we just disabled everything that wasn't working.
Flags: needinfo?(ahalberstadt)
Reporter | ||
Comment 12•11 years ago
|
||
Reporter | ||
Updated•11 years ago
|
Attachment #8575903 -
Attachment is obsolete: true
Reporter | ||
Comment 13•11 years ago
|
||
Comment on attachment 8576029 [details] [diff] [review]
Disabling mochitests on Mulet with parity to B2G Desktop for taskcluster r=ahal
Carrying r+, updating bug title
Attachment #8576029 -
Flags: review+
Reporter | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 14•11 years ago
|
||
Flags: in-testsuite-
Keywords: checkin-needed
Comment 15•11 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•