601243 - runtests.py.in should clean up its <TempDir>\tmpXYZ\mochikit.manifest

Status: RESOLVED FIXED

(Testing :: General, defect)

Description

[Serge Gautherie (:sgautherie)]

{
678     temp_file = os.path.join(tempfile.mkdtemp(), "mochikit.manifest")
}

I get one more directory and file each time I run a test (suite): my Temp dir is filling up :-(

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: stop making a secondary temp directory in runtests.py (WIP)

I have a patch that works, but appears to be leaking on linux debug chrome tests.  odd, since this is related to setup before the browser is launched.  Needless to say until I figure it out or stop seeing that leak I will wait for this patch.

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 481816 [details] [diff] [review]
stop making a secondary temp directory in runtests.py (WIP)

2 green try server runs.  Must have been something else in the tree causing those linux debug chrome leaks.

Comment 3

[cmtalbert]

Comment on attachment 481816 [details] [diff] [review]
stop making a secondary temp directory in runtests.py (WIP)

Simpler code too, we should have done this to start with ;)  Hindsight always 20/20,I guess.  Nice fix.

Comment 4

[cmtalbert]

Landed http://hg.mozilla.org/mozilla-central/rev/54a73ef4d157