Bug 53954 - Move the sputnik test suite out of fast/js/
Summary: Move the sputnik test suite out of fast/js/
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: James Robinson
Depends on:
Reported: 2011-02-07 15:16 PST by James Robinson
Modified: 2011-02-07 20:02 PST (History)
5 users (show)

See Also:

Patch (40.47 KB, patch)
2011-02-07 15:21 PST, James Robinson
mitz: review+
jamesr: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Robinson 2011-02-07 15:16:22 PST
Move the sputnik test suite out of fast/js/
Comment 1 James Robinson 2011-02-07 15:21:23 PST
Created attachment 81541 [details]
Comment 2 James Robinson 2011-02-07 15:21:58 PST
I'm not sure what will happen when I try to land this.  git diff --stat shows:

 22196 files changed, 730747 insertions(+), 730728 deletions(-)
Comment 3 WebKit Review Bot 2011-02-07 15:23:15 PST
Attachment 81541 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plat..." exit_code: 1

LayoutTests/platform/chromium/test_expectations.txt:1984:  Path does not exist. sputnik  [test/expectations] [2]
Total errors found: 1 in 8 files

If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 4 James Robinson 2011-02-07 15:46:38 PST
It looks like webkit-patch upload didn't include the file move metadata, so of course the style checker isn't aware of the new patch.  I'm guessing that uploading base versions of all the moved files would explode something anyway so it's probably for the best.
Comment 5 James Robinson 2011-02-07 16:57:39 PST
Comment on attachment 81541 [details]

Thanks for the review.  cq-'ing it since this will likely take a bit of manual trickery to land.
Comment 6 James Robinson 2011-02-07 20:02:01 PST
Amazingly this just worked.  http://trac.webkit.org/changeset/77885