Closed
Bug 908906
Opened 10 years ago
Closed 10 years ago
Add test summary info to jit-tests when run with --tinderbox
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla26
People
(Reporter: dminor, Assigned: dminor)
References
Details
Attachments
(1 file)
2.85 KB,
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
Mozharness looks for pass and fail counts to pass into tbpl. I would add something like the following when jit-test is run with --tinderbox to facilitate mozharness integration: log.info("Result summary:") log.info("Passed: %d" % pass_count) log.info("Failed: %d" % fail_count)
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #795626 -
Flags: review?(terrence)
Comment 2•10 years ago
|
||
Comment on attachment 795626 [details] [diff] [review] Add extra summary information to print_test_summary Review of attachment 795626 [details] [diff] [review]: ----------------------------------------------------------------- r=me ::: js/src/tests/lib/jittests.py @@ +507,5 @@ > + if options.tinderbox: > + num_failures = len(failures) if failures else 0 > + print('Result summary:') > + print('Passed: %d' % (num_tests - num_failures)) > + print('Failed: %d' % num_failures) Remove trailing whitespace.
Attachment #795626 -
Flags: review?(terrence) → review+
Assignee | ||
Comment 3•10 years ago
|
||
Thanks! https://hg.mozilla.org/integration/mozilla-inbound/rev/1b90b1ac58a8
Comment 4•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1b90b1ac58a8
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in
before you can comment on or make changes to this bug.
Description
•