Bug 143378 - Move Mac-specific ruby tests to platform/mac
Summary: Move Mac-specific ruby tests to platform/mac
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Myles C. Maxfield
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-04-03 09:31 PDT by Myles C. Maxfield
Modified: 2015-05-27 11:31 PDT (History)
5 users (show)

See Also:


Attachments
Patch (51.83 KB, patch)
2015-04-03 09:32 PDT, Myles C. Maxfield
bfulgham: review+
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews101 for mac-mavericks (733.67 KB, application/zip)
2015-04-03 10:20 PDT, Build Bot
no flags Details
Archive of layout-test-results from ews106 for mac-mavericks-wk2 (769.24 KB, application/zip)
2015-04-03 10:25 PDT, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Myles C. Maxfield 2015-04-03 09:31:42 PDT
Move Mac-specific ruby tests to platform/mac
Comment 1 Myles C. Maxfield 2015-04-03 09:32:12 PDT
Created attachment 250080 [details]
Patch
Comment 2 Brent Fulgham 2015-04-03 10:00:09 PDT
Comment on attachment 250080 [details]
Patch

Did you use "svn mv" on these files? It doesn't look like it, which means we will lose the (brief) history of the test files.
Comment 3 Build Bot 2015-04-03 10:20:15 PDT
Comment on attachment 250080 [details]
Patch

Attachment 250080 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5544167048478720

New failing tests:
platform/mac/fast/ruby/ruby-expansion-cjk-5.html
platform/mac/fast/ruby/ruby-expansion-cjk-2.html
platform/mac/fast/ruby/ruby-expansion-cjk-3.html
platform/mac/fast/ruby/ruby-expansion-cjk.html
platform/mac/fast/ruby/ruby-expansion-cjk-4.html
Comment 4 Build Bot 2015-04-03 10:20:18 PDT
Created attachment 250087 [details]
Archive of layout-test-results from ews101 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews101  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 5 Brent Fulgham 2015-04-03 10:23:11 PDT
(In reply to comment #4)
> Created attachment 250087 [details]
> Archive of layout-test-results from ews101 for mac-mavericks
> 
> The attached test failures were seen while running run-webkit-tests on the
> mac-ews.
> Bot: ews101  Port: mac-mavericks  Platform: Mac OS X 10.9.5

Are these tests only valid on Yosemite or newer? Might need to be skipped in platform/mac-yosemite...
Comment 6 Brent Fulgham 2015-04-03 10:23:30 PDT
(In reply to comment #5)
> (In reply to comment #4)
> > Created attachment 250087 [details]
> > Archive of layout-test-results from ews101 for mac-mavericks
> > 
> > The attached test failures were seen while running run-webkit-tests on the
> > mac-ews.
> > Bot: ews101  Port: mac-mavericks  Platform: Mac OS X 10.9.5
> 
> Are these tests only valid on Yosemite or newer? Might need to be skipped in
> platform/mac-yosemite...

... I mean "platform/mac-mavericks".
Comment 7 Build Bot 2015-04-03 10:25:49 PDT
Comment on attachment 250080 [details]
Patch

Attachment 250080 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/6312195449159680

New failing tests:
platform/mac/fast/ruby/ruby-expansion-cjk-5.html
platform/mac/fast/ruby/ruby-expansion-cjk-2.html
platform/mac/fast/ruby/ruby-expansion-cjk-3.html
platform/mac/fast/ruby/ruby-expansion-cjk.html
platform/mac/fast/ruby/ruby-expansion-cjk-4.html
Comment 8 Build Bot 2015-04-03 10:25:52 PDT
Created attachment 250088 [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 Alexey Proskuryakov 2015-04-03 12:18:22 PDT
In fact, I'd prefer to not have any tests in platform directories, that's super confusing. I think that we should only have results there.
Comment 10 Myles C. Maxfield 2015-04-03 15:47:23 PDT
The tests should work on both Mac platforms. I'll have to figure out why they're not.

Currently I don't believe there is a way to make reftests look for expected results in platform directories. That's why I had to move the tests themselves.
Comment 11 Alexey Proskuryakov 2015-04-03 15:53:15 PDT
You don't have to move the tests, you can just add expectations for other platforms (skip or failure, whichever is more appropriate).
Comment 12 Myles C. Maxfield 2015-04-03 16:10:27 PDT
(In reply to comment #11)
> You don't have to move the tests, you can just add expectations for other
> platforms (skip or failure, whichever is more appropriate).

That is the current state. Sounds like I should do nothing.
Comment 13 Daniel Bates 2015-05-27 11:31:17 PDT
For completeness, we moved the the ruby tests from LayoutTests/fast/ruby to LayoutTests/platform/mac in <http://trac.webkit.org/changeset/184899> (bug #145336).