Closed
Bug 435376
Opened 17 years ago
Closed 16 years ago
Start making leaks fatal when running tests in setups where we don't leak
Categories
(Testing :: Mochitest, defect)
Testing
Mochitest
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Waldo, Assigned: Waldo)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
sayrer
:
review+
|
Details | Diff | Splinter Review |
--chrome tests don't leak anywhere.
Normal Mochitests don't leak on OS X.
We should enforce this so that we don't backslide in these cases, because right now we could regress and nobody would notice.
I'm sure someone is going to complain that this will turn their tinderbox orange; if you're that person, just add --leak-threshold=N to your invocation script to manually override it. The patch to be posted here only modifies the leak threshold if it wasn't set in the commandline arguments.
Assignee | ||
Comment 1•17 years ago
|
||
We could do this tinderbox side, but then people running the tests locally won't know they're running with a different setup from tinderbox, which is likely to result in more oranges. Also, as we drive down leak numbers, we'll want to make changes to these numbers, and it's vastly easier to change a file in the repository than it is to update a buildbot config files on the tinderboxen themselves. For the moment, until we hit 0 everywhere and can just make that the overall default, I think we're best off doing if-unset tweaks in the specific cases where we know we're doing well.
Attachment #322227 -
Flags: review?(sayrer)
Assignee | ||
Comment 2•17 years ago
|
||
Hm, so if bug 435138 is WFM now and the Mochitest runs on the Windows tinderbox are now doing bloat logging correctly, it seems we're at 0 leaks on normal Mochitests on Windows, too. This patch includes that restriction.
Attachment #322227 -
Attachment is obsolete: true
Attachment #322230 -
Flags: review?(sayrer)
Attachment #322227 -
Flags: review?(sayrer)
Updated•17 years ago
|
Attachment #322230 -
Flags: review?(sayrer) → review+
Assignee | ||
Comment 3•16 years ago
|
||
Fixt in mozilla-central.
Status: NEW → RESOLVED
Closed: 16 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Assignee | ||
Comment 4•16 years ago
|
||
Moving a bunch of Core :: Testing bugs to Testing :: Mochitest to clear out the former, which is obsolete now that we have more specialized categories for such bugs; filter on the string "MochitestMmMm" to delete all these notifications.
Component: Testing → Mochitest
Flags: in-testsuite+
Product: Core → Testing
QA Contact: testing → mochitest
Target Milestone: mozilla2.0 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•