Closed Bug 1475141 Opened 3 years ago Closed 2 years ago
Test Seen" string in reftests
Bug 1475141 - [reftest] Make sure self.lastTestSeen displays whether the test was in-progress or finished, r=jmaher
46 bytes, text/x-phabricator-request
|Details | Review|
+++ This bug was initially created as a clone of Bug #1472280 +++ We're also seeing bug 1471222 in a bunch of reftests, where the failure output still confusingly suggests that the timeout happened on whatever test ran last, even though the last test finished.
Andrew, any chance you can do something similar here to what you did in bug 1472280? The reftest ASan shutdown hangs in bug 1471222 have gotten pretty bad, and I'd like to update bug 1475141's patches to deal with them.
Yeah, I'll try to find some time for this in the next week or two.
Assignee: nobody → ahal
Status: NEW → ASSIGNED
Sorry for the delay on this. I have a patch up on try now: https://treeherder.mozilla.org/#/jobs?repo=try&revision=7d99b9ec5b4fbf34e7560627ccf7a92630441c39
This will also change lastTestSeen to "Last test finished" after the 'last' test in the run has completed. This will prevent us from misattributing shutdown (and between-test failures) to the unlucky test that happened to run beforehand.
Comment on attachment 9005611 [details] Bug 1475141 - [reftest] Make sure self.lastTestSeen displays whether the test was in-progress or finished, r=jmaher Joel Maher ( :jmaher PTO - back Sep 4 ) (UTC-4) has approved the revision.
Attachment #9005611 - Flags: review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/270e36de2a74 [reftest] Make sure self.lastTestSeen displays whether the test was in-progress or finished, r=jmaher
You need to log in before you can comment on or make changes to this bug.