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)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Waldo, Assigned: Waldo)

References

Details

Attachments

(1 file, 1 obsolete file)

--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.
Attached patch Patch (obsolete) (deleted) — Splinter Review
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)
Attached patch Patch if bug 435138 is WFM now (deleted) — Splinter Review
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)
Attachment #322230 - Flags: review?(sayrer) → review+
Fixt in mozilla-central.
Status: NEW → RESOLVED
Closed: 16 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
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.

Attachment

General

Created:
Updated:
Size: