456876 - mochitests --log-file simply doesn't work when used with --test-path

Status: VERIFIED DUPLICATE of bug 508664

(Testing :: Mochitest, defect)

Description

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

User-Agent:       Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.0.1) Gecko/2008070206 Firefox/3.0.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1a2pre) Gecko/2080808184511 Minefield/3.1a2pre

when running mochitests on windows in a MINGW32 build shell window, I give it the --log-file and it translates to c:\...\...\...   When this is evaluated, the \ is not escaped causing the filepath to not exist.

Reproducible: Always

Steps to Reproduce:
1. run "python runtests.py --log-file=output.log" in a windows build environment
2. look for output.log in your working directory
3.
Actual Results:  
no file is generated

Expected Results:  
output.log to be generated and have all the log information

Comment 1

[cmtalbert]

I don't think this has to do with the windows path, because I'm now seeing the same behavior on mac OS X with a pull of source from 9/24/08.

We've done something to break this.

Repro:
python runtests.py --test-path=layout/base/tests --log-file=log.txt

Actual:
no log.txt file created

Expected:
--log-file works as usual

Comment 2

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20090114 Minefield/3.2a1pre] (home, optim default) (W2Ksp4)
(http://hg.mozilla.org/mozilla-central/rev/e0ed5852481a)

No file either, from a cmd.exe prompt, using for example
|python runtests.py --test-path=toolkit/content/tests/widgets/test_videocontrols.html --autorun --log-file=C:\471085_logs.txt|

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to comment #2)

Note that it works when running the whole suite, with:
|python runtests.py --autorun --close-when-done --log-file=..\..\..\..\..\SM-_6-Mochitest.log|

Maybe --test-path= is braking this feature ?

Comment 4

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

this is the case on all platforms when you test a single test_ file.  It is really annoying when trying to debug things and diff the results.

Comment 5

[(not currently active) Ted Mielczarek]

You filed a dupe of your own bug, man!

Comment 6

[Serge Gautherie (:sgautherie)]

V.Duplicate:
No obvious at first per (all) comments there,
but I agree the patch does look like it should fix this bug too.