Bug 58635 - NRWT is missing --no-new-test-results
Summary: NRWT is missing --no-new-test-results
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-14 22:59 PDT by Adam Barth
Modified: 2011-10-12 14:32 PDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2011-04-14 22:59:00 PDT
The commit-queue uses --no-new-test-results to avoid having run-webkit-test drop junk files in its working copy.  However, that option appears to be missing from NRWT, blocking our use of it for the commit-queue.

Of somewhat lesser importance, NRWT also seems to be missing --wait-for-httpd, but I don't think we actually need that option anymore.
Comment 1 Eric Seidel (no email) 2011-04-15 06:40:23 PDT
Why don't we need --wait-for-httpd?  That moves all the httpd tests to the end and uses locking.
Comment 2 Adam Barth 2011-04-15 11:04:44 PDT
(In reply to comment #1)
> Why don't we need --wait-for-httpd?  That moves all the httpd tests to the end and uses locking.

My understanding is that we only need that if we're going to run two instances of the tests on the same box.  I don't think we have any plans to do that in the near future.
Comment 3 Dirk Pranke 2011-04-15 11:32:44 PDT
(In reply to comment #0)
> The commit-queue uses --no-new-test-results to avoid having run-webkit-test drop junk files in its working copy.  However, that option appears to be missing from NRWT, blocking our use of it for the commit-queue.
> 

NRWT doesn't generate new tests results unless asked to do so, so maybe this is not necessary? 

Alternatively, maybe NRWT should be generating new test results by default to match ORWT, and then add this?

> Of somewhat lesser importance, NRWT also seems to be missing --wait-for-httpd, but I don't think we actually need that option anymore.

NRWT locks all the time, by default.
Comment 4 Eric Seidel (no email) 2011-10-12 14:32:53 PDT
I'm pretty sure we flipped this default.

Yup, I see it in --help now:

  --no-new-test-results
                        Don't create new baselines when no expected results
                        exist