Bug 151190 - Clean imported/w3c/web-platform-tests folder
Summary: Clean imported/w3c/web-platform-tests folder
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-12 06:54 PST by youenn fablet
Modified: 2015-11-19 00:01 PST (History)
8 users (show)

See Also:


Attachments
Patch (71.59 KB, patch)
2015-11-12 07:25 PST, youenn fablet
no flags Details | Formatted Diff | Diff
Patch (70.20 KB, patch)
2015-11-12 07:47 PST, youenn fablet
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews102 for mac-mavericks (711.40 KB, application/zip)
2015-11-12 08:33 PST, Build Bot
no flags Details
Archive of layout-test-results from ews106 for mac-mavericks-wk2 (781.00 KB, application/zip)
2015-11-12 08:38 PST, Build Bot
no flags Details
Archive of layout-test-results from ews116 for mac-yosemite (840.73 KB, application/zip)
2015-11-12 08:39 PST, Build Bot
no flags Details
Patch (52.85 KB, patch)
2015-11-13 07:09 PST, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for landing (54.24 KB, patch)
2015-11-16 01:28 PST, youenn fablet
no flags Details | Formatted Diff | Diff
Patch for landing (52.85 KB, patch)
2015-11-16 01:39 PST, youenn fablet
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2015-11-12 06:54:34 PST
During the previous import, tests removed in wpt repo were kept in imported/w3c/web-platform-tests WebKit folder.
We should remove these obsolete tests and their expected-txt files.
Comment 1 youenn fablet 2015-11-12 07:25:56 PST
Created attachment 265388 [details]
Patch
Comment 2 youenn fablet 2015-11-12 07:27:29 PST
(In reply to comment #0)
> During the previous import, tests removed in wpt repo were kept in
> imported/w3c/web-platform-tests WebKit folder.
> We should remove these obsolete tests and their expected-txt files.

Some tests that I thought were obsolete are actually tests that have the same filenames except for the extension which can html and xhtml for instance.
This poses an issue to get different -expected.txt files.
We might want to change filenames at the wpt repository.
Then, we might be able to remove those tests.
Comment 3 youenn fablet 2015-11-12 07:47:57 PST
Created attachment 265392 [details]
Patch
Comment 4 youenn fablet 2015-11-12 07:51:36 PST
(In reply to comment #3)
> Created attachment 265392 [details]
> Patch

Second patch does not remove empty files found in imported/w3c/web-platform-tests/dom/nodes/resources since it seems EWS has issues applying patch.
I guess the removal of these files should be landed manually.
Comment 5 Build Bot 2015-11-12 08:33:36 PST
Comment on attachment 265392 [details]
Patch

Attachment 265392 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/419476

New failing tests:
imported/w3c/web-platform-tests/dom/nodes/ParentNode-querySelector-All-xhtml.xht
Comment 6 Build Bot 2015-11-12 08:33:40 PST
Created attachment 265395 [details]
Archive of layout-test-results from ews102 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 7 Build Bot 2015-11-12 08:38:18 PST
Comment on attachment 265392 [details]
Patch

Attachment 265392 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.webkit.org/results/419480

New failing tests:
imported/w3c/web-platform-tests/dom/nodes/ParentNode-querySelector-All-xhtml.xht
Comment 8 Build Bot 2015-11-12 08:38:22 PST
Created attachment 265397 [details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 9 Build Bot 2015-11-12 08:39:25 PST
Comment on attachment 265392 [details]
Patch

Attachment 265392 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/419472

New failing tests:
imported/w3c/web-platform-tests/dom/nodes/ParentNode-querySelector-All-xhtml.xht
Comment 10 Build Bot 2015-11-12 08:39:30 PST
Created attachment 265398 [details]
Archive of layout-test-results from ews116 for mac-yosemite

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-yosemite  Platform: Mac OS X 10.10.5
Comment 11 youenn fablet 2015-11-13 07:09:37 PST
Created attachment 265480 [details]
Patch
Comment 12 Chris Dumez 2015-11-13 09:47:36 PST
Comment on attachment 265480 [details]
Patch

rs=me
Comment 13 youenn fablet 2015-11-16 01:28:45 PST
Created attachment 265577 [details]
Patch for landing
Comment 14 WebKit Commit Bot 2015-11-16 01:31:43 PST
Comment on attachment 265577 [details]
Patch for landing

Rejecting attachment 265577 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'apply-attachment', '--no-update', '--non-interactive', 265577, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
n_dom_03-expected.txt'
patching file LayoutTests/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-iframe-element/move_iframe_in_dom_04-expected.txt
rm 'LayoutTests/imported/w3c/web-platform-tests/html/semantics/embedded-content/the-iframe-element/move_iframe_in_dom_04-expected.txt'
patching file LayoutTests/imported/w3c/web-platform-tests/w3c-import.log

Failed to run "[u'/Volumes/Data/EWS/WebKit/Tools/Scripts/svn-apply', '--force']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Full output: http://webkit-queues.webkit.org/results/436123
Comment 15 youenn fablet 2015-11-16 01:39:03 PST
Created attachment 265578 [details]
Patch for landing
Comment 16 youenn fablet 2015-11-16 01:40:17 PST
(In reply to comment #15)
> Created attachment 265578 [details]
> Patch for landing

Removing removal of empty files.
Comment 17 WebKit Commit Bot 2015-11-16 02:37:47 PST
Comment on attachment 265578 [details]
Patch for landing

Clearing flags on attachment: 265578

Committed r192467: <http://trac.webkit.org/changeset/192467>
Comment 18 WebKit Commit Bot 2015-11-16 02:37:51 PST
All reviewed patches have been landed.  Closing bug.
Comment 19 youenn fablet 2015-11-16 03:01:52 PST
(In reply to comment #16)
> (In reply to comment #15)
> > Created attachment 265578 [details]
> > Patch for landing
> 
> Removing removal of empty files.

Empty files are removed as part of https://trac.webkit.org/changeset/192468