Closed
Bug 1049999
Opened 11 years ago
Closed 11 years ago
Let the test run become failed when SimpleTest.js barfs on a parent.TestRunner call
Categories
(Testing :: Mochitest, defect)
Tracking
(firefox33 fixed, firefox34 fixed)
RESOLVED
FIXED
mozilla34
People
(Reporter: martijn.martijn, Assigned: martijn.martijn)
References
Details
Attachments
(1 file)
1.61 KB,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
This is more or less a follow-up from bug 928678.
When SimpleTest.js throws on parent.TestRunner, it should cause the test run to become in a failed state.
Assignee | ||
Comment 1•11 years ago
|
||
This should be tested on try first, to see if it would catch failures, but tryserver is currently closed.
Attachment #8468935 -
Flags: review?(jmaher)
Comment 2•11 years ago
|
||
Comment on attachment 8468935 [details] [diff] [review]
1049999.diff
Review of attachment 8468935 [details] [diff] [review]:
-----------------------------------------------------------------
nice!
Attachment #8468935 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Pushed to try: https://tbpl.mozilla.org/?tree=Try&rev=0cbc24350166
Assignee | ||
Comment 4•11 years ago
|
||
Tryserver is all green (the oranges don't have anything to do with this).
Keywords: checkin-needed
Comment 5•11 years ago
|
||
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Comment 7•11 years ago
|
||
Filed bug 1051775 for making this turn the run orange, since at the moment it doesn't.
Depends on: 1051775
Comment 8•11 years ago
|
||
status-firefox33:
--- → fixed
status-firefox34:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•