Closed
Bug 974073
Opened 11 years ago
Closed 11 years ago
failed mochitests no longer log state
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla30
People
(Reporter: karlt, Assigned: froydnj)
References
Details
Attachments
(1 file)
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
<karl> are mochitest passes elided from test logs now?
<froydnj> karl: yes
<karl> i would like to know how far a test got before it timed out, but i'm not seeing progress, even for passing tests
<ted> karl: yes
<ted> karl: https://bugzilla.mozilla.org/show_bug.cgi?id=937181
<karl> would changing all the 'ok's to todos provide some progress info? or are there other options?
<karl> dump?
<ted> not sure, froydnj ^^
<karl> froydnj: are successes in a test that fails printed?
<froydnj> ted: adding || true to http://mxr.mozilla.org/mozilla-central/source/testing/mochitest/tests/SimpleTest/SimpleTest.js#330 will get you your log messages back
<karl> froydnj: i need something to run on m-c, not try
<froydnj> karl: dump is probably your best option, I think
<karl> froydnj: this is making it impossible to diagnose failed tests
<karl> froydnj: many failures happen in random tests and it is impractical to dump in all of them
<froydnj> karl: please file a bug about it, then, and make it block bug 937181
Reporter | ||
Updated•11 years ago
|
Assignee | ||
Comment 1•11 years ago
|
||
I think the most reasonable way to do this is to require the test to call:
SimpleTest.requestCompleteLog();
and that will force logging of all TEST-PASSes for the current test. Does that sound plausible, Joel?
Flags: needinfo?(jmaher)
Assignee | ||
Comment 3•11 years ago
|
||
Probably would have been simpler to just provide the patch in the first
place. :)
Attachment #8381776 -
Flags: review?(jmaher)
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → nfroyd
Updated•11 years ago
|
Attachment #8381776 -
Flags: review?(jmaher) → review+
Assignee | ||
Comment 4•11 years ago
|
||
Flags: in-testsuite-
Comment 5•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
You need to log in
before you can comment on or make changes to this bug.
Description
•