New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[import-w3c-tests] Explicitly list all directories we're not importing as skip #19642
[import-w3c-tests] Explicitly list all directories we're not importing as skip #19642
Conversation
(This builds on top of #19442, still in review.) |
EWS run on previous version of this PR (hash 5781a96) |
Not sure explicit-skip is the best name. |
Agree with @nt1m... the name could be improved but I don't have a better suggestion than what Tim proposed. |
5781a96
to
163e0e3
Compare
EWS run on previous version of this PR (hash 163e0e3) |
163e0e3
to
70ff1a4
Compare
EWS run on previous version of this PR (hash 70ff1a4) |
70ff1a4
to
4abc16a
Compare
EWS run on previous version of this PR (hash 4abc16a) |
4abc16a
to
5b14200
Compare
EWS run on previous version of this PR (hash 5b14200) |
5b14200
to
5973c45
Compare
EWS run on current version of this PR (hash 5973c45) |
β¦g as skip https://bugs.webkit.org/show_bug.cgi?id=214006 Reviewed by Jonathan Bedard. Add a new value to import-expectations.json: "skip-new-directories". "skip-new-directories" is mostly treated the same as "skip" (i.e., everything under it is skipped, unless marked as "import"). However, unlike "skip", child files are imported, and child directories are added to import-expectations.json as "skip" (thus the name). We apply this to both "web-platform-tests" and "web-platform-tests/css", to avoid importing new test suites for (presumably new) specifications. By doing this, as opposed to applying "skip" directly to those directories, we gain visibility to new test suites being added, as these now cause a "skip" line to be added when doing a full import. * LayoutTests/imported/w3c/resources/import-expectations.json: * Tools/Scripts/webkitpy/style/checkers/jsonchecker.py: (JSONImportExpectationsChecker.check): * Tools/Scripts/webkitpy/w3c/test_downloader.py: (TestDownloader.__init__): (TestDownloader._init_paths_from_expectations): (TestDownloader.update_import_expectations): * Tools/Scripts/webkitpy/w3c/test_importer.py: (TestImporter.__init__): (TestImporter.do_import): (TestImporter.should_skip_path): Canonical link: https://commits.webkit.org/272109@main
5973c45
to
e5fd5e0
Compare
Committed 272109@main (e5fd5e0): https://commits.webkit.org/272109@main Reviewed commits have been landed. Closing PR #19642 and removing active labels. |
e5fd5e0
5973c45
π§ͺ styleπ iosπ macπ wpeπ wincairoπ§ͺ bindingsπ ios-simπ mac-AS-debugπ§ͺ wpe-wk2π§ͺ webkitperlπ§ͺ ios-wk2π§ͺ api-macπ§ͺ api-wpeπ§ͺ webkitpyπ§ͺ ios-wk2-wptπ§ͺ mac-wk1π gtkπ§ͺ api-iosπ§ͺ mac-wk2π§ͺ gtk-wk2π tvπ§ͺ mac-AS-debug-wk2π§ͺ api-gtkπ tv-simπ§ͺ mac-wk2-stressπ watchπ π§ͺ unsafe-mergeπ watch-sim