Closed Bug 809975 Opened 12 years ago Closed 12 years ago

--failed-only should ignore expected failures

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla20

People

(Reporter: sfink, Assigned: sfink)

Details

Attachments

(1 file)

I finally tracked down why I was seeing a bunch of random test failures with or without my patch.

They were expected failures. But I've been running with -s -F (--show-cmds --failed-only), and hadn't noticed that all of these mysteriously failing files were not displaying the "REGRESSIONS:" label.

Ugh.
Only display unexpected failures.
Attachment #679766 - Flags: review?(terrence)
Comment on attachment 679766 [details] [diff] [review]
-failed-only should ignore expected failures

Review of attachment 679766 [details] [diff] [review]:
-----------------------------------------------------------------

It's a shame we need to move that complex logic above |show| selection, but I don't see a better way to do it.
Attachment #679766 - Flags: review?(terrence) → review+
Attachment #679766 - Flags: checkin+
https://hg.mozilla.org/mozilla-central/rev/5f9773707ee6
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: