Closed Bug 1019883 Opened 5 years ago Closed 5 years ago

"expected" field of structured log messages should should not be populated for skipped tests

Categories

(Testing :: Mozbase, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla32

People

(Reporter: chmanchester, Assigned: chmanchester)

References

Details

Attachments

(1 file)

mozlog.structured populates the 'expected' field of log messages in cases status != expected, including when the test status is 'SKIP' and the expected is 'PASS'. 

This convention appears to be used to determining when a test is considered a failure by the test run, which arguably not the case for skipped tests. Either we can stop depending on this convention, or not populate the field for 'SKIP' tests. I will post a patch doing the latter.
Assignee: nobody → cmanchester
Blocks: 956739
Comment on attachment 8433586 [details] [diff] [review]
Don't populate expected field in log messages for skipped tests.;

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

Yes, I think this makes sense.
Attachment #8433586 - Flags: review?(james) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c19d5dcd5759
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.