WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
143090
run-webkit-tests should have an option to run new tests
https://bugs.webkit.org/show_bug.cgi?id=143090
Summary
run-webkit-tests should have an option to run new tests
youenn fablet
Reported
2015-03-26 03:52:02 PDT
It would be nice to run all tests modified/created through run-webkit-tests. That may help developers as well as provide limited EWS bots the ability to check a small but critical amount of tests.
Attachments
Patch
(7.73 KB, patch)
2015-03-26 04:28 PDT
,
youenn fablet
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
youenn fablet
Comment 1
2015-03-26 04:28:18 PDT
Created
attachment 249482
[details]
Patch
youenn fablet
Comment 2
2015-03-26 04:41:17 PDT
Uploaded a patch at
bug 143093
to show a potential use of --new-tests and --pass-if-no-test options.
Alexey Proskuryakov
Comment 3
2015-03-26 10:00:46 PDT
Comment on
attachment 249482
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=249482&action=review
> Tools/ChangeLog:8 > + Add a --new-tests option to run-webkit-tests which adds tests that have been changed/created compared to the repository.
Can't we figure that out outside run-webkit-tests? I'm not sure if this is a mode that engineers need to have access to. EWS can just pass the list of new tests if any.
Csaba Osztrogonác
Comment 4
2015-03-26 11:45:06 PDT
(In reply to
comment #3
)
> Comment on
attachment 249482
[details]
> Patch > > View in context: >
https://bugs.webkit.org/attachment.cgi?id=249482&action=review
> > > Tools/ChangeLog:8 > > + Add a --new-tests option to run-webkit-tests which adds tests that have been changed/created compared to the repository. > > Can't we figure that out outside run-webkit-tests? I'm not sure if this is a > mode that engineers need to have access to. > > EWS can just pass the list of new tests if any.
EWS can pass list of the tests to run-webkit-tests as command line arguments directly and in a file with --test-list option. But I'm not sure how are test expectations handled in this cases. AFAIR skipped tests are run too, but I'm not sure.
youenn fablet
Comment 5
2015-03-26 12:01:01 PDT
(In reply to
comment #4
)
> (In reply to
comment #3
) > > Comment on
attachment 249482
[details]
> > Patch > > > > View in context: > >
https://bugs.webkit.org/attachment.cgi?id=249482&action=review
> > > > > Tools/ChangeLog:8 > > > + Add a --new-tests option to run-webkit-tests which adds tests that have been changed/created compared to the repository. > > > > Can't we figure that out outside run-webkit-tests? I'm not sure if this is a > > mode that engineers need to have access to. > > > > EWS can just pass the list of new tests if any.
Combined with git, this option runs all tests added/modified by the set of commits compared to the remote origin master. Handy for branch development, rebasing -i... It might also be useful when reimporting W3C tests.
> EWS can pass list of the tests to run-webkit-tests as command > line arguments directly and in a file with --test-list option. > > But I'm not sure how are test expectations handled in this > cases. AFAIR skipped tests are run too, but I'm not sure.
Good point. It should be checked as part of integration/unit tests that should be added.
youenn fablet
Comment 6
2015-03-26 13:08:25 PDT
> > EWS can pass list of the tests to run-webkit-tests as command > > line arguments directly and in a file with --test-list option. > > > > But I'm not sure how are test expectations handled in this > > cases. AFAIR skipped tests are run too, but I'm not sure. > > Good point. > It should be checked as part of integration/unit tests that should be added.
There is the --skipped=always/default option which enables to run/skip the tests here. It all ends up whether we want to expose this option in rwt or not... I see some value to it as exposed previously. The main downsides I see with that option is that there is the need to handle within rwt the case of runs with no tests. This patch introduces the --pass-if-no-option for that purpose. But this case would probably be best handled within EWS. Thoughts?
Alexey Proskuryakov
Comment 7
2015-03-26 14:47:58 PDT
Another complication is layering. Currently, run-webkit-tests knows nothing about version control, and it would be confusing if it started to use that just for one command. If enough people want a special mode to run just the new/modified tests, webkit-patch is unclean enough to have that added too, I think, but probably not run-webkit-tests.
youenn fablet
Comment 8
2015-03-27 10:27:45 PDT
(In reply to
comment #7
)
> Another complication is layering. Currently, run-webkit-tests knows nothing > about version control, and it would be confusing if it started to use that > just for one command. > > If enough people want a special mode to run just the new/modified tests, > webkit-patch is unclean enough to have that added too, I think, but probably > not run-webkit-tests.
Then we should directly go to
bug 143093
...
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug