Add test summary info to jit-tests when run with --tinderbox

RESOLVED FIXED in mozilla26

Status

()

RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: dminor, Assigned: dminor)

Tracking

Trunk
mozilla26
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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

5 years ago
Created attachment 795626 [details] [diff] [review]
Add extra summary information to print_test_summary
Attachment #795626 - Flags: review?(terrence)
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+
https://hg.mozilla.org/mozilla-central/rev/1b90b1ac58a8
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
You need to log in before you can comment on or make changes to this bug.