RESOLVED FIXED 62751
we should only log the test names to the tests_run.txt files during layout_tests 
https://bugs.webkit.org/show_bug.cgi?id=62751
Summary we should only log the test names to the tests_run.txt files during layout_tests
Dirk Pranke
Reported 2011-06-15 13:27:36 PDT
Currently, both ORWT and NRWT log the full path to the test being run to a file in the layout-test-results directory. There's not really any need for the full path, so we should just log the test names (which are relative paths using unix-style directory separators) instead. See bug 58692 for a bit of background on the change that added the logging to ORWT (NRWT has had this forever).
Attachments
Patch (1.97 KB, patch)
2011-06-15 13:31 PDT, Dirk Pranke 		no flags
DetailsFormatted DiffDiff
add testing of the contents of tests_run*.txt per review feedback from eseidel (3.23 KB, patch)
2011-06-20 12:39 PDT, Dirk Pranke 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Dirk Pranke
Comment 1 2011-06-15 13:31:52 PDT
Created attachment 97355 [details] Patch
Eric Seidel (no email)
Comment 2 2011-06-15 15:08:04 PDT
Comment on attachment 97355 [details] Patch Seems fine. But would be nice to have a unit test.
Eric Seidel (no email)
Comment 3 2011-06-15 15:08:27 PDT
(I only care about worker.py, ORWT is deadman walking.)
Csaba Osztrogonác
Comment 4 2011-06-15 23:33:00 PDT
(In reply to comment #3) > (I only care about worker.py, ORWT is deadman walking.) rs=me for ORWT change. Thanks for the fix.
Dirk Pranke
Comment 5 2011-06-20 12:39:04 PDT
Created attachment 97840 [details] add testing of the contents of tests_run*.txt per review feedback from eseidel
WebKit Review Bot
Comment 6 2011-06-20 14:30:41 PDT
Comment on attachment 97840 [details] add testing of the contents of tests_run*.txt per review feedback from eseidel Clearing flags on attachment: 97840 Committed r89290: <http://trac.webkit.org/changeset/89290>
WebKit Review Bot
Comment 7 2011-06-20 14:30:46 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.