Bug 95370 - NRWT should look in mac-wk2 for a TestExpecations file
Summary: NRWT should look in mac-wk2 for a TestExpecations file
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Dirk Pranke
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-29 13:20 PDT by Jessie Berlin
Modified: 2012-08-30 13:54 PDT (History)
6 users (show)

See Also:


Attachments
Patch (3.50 KB, patch)
2012-08-29 16:21 PDT, Dirk Pranke
no flags Details | Formatted Diff | Diff
Patch (5.40 KB, patch)
2012-08-30 12:21 PDT, Dirk Pranke
no flags Details | Formatted Diff | Diff
fix mac.py (4.66 KB, patch)
2012-08-30 12:23 PDT, Dirk Pranke
ojan: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jessie Berlin 2012-08-29 13:20:55 PDT
I attempted to add a TestExpectations file for mac-wk2 today and NRWT wasn't picking it up when I ran the WK2 tests on Lion unless I passed it as the value for --additional-expectations.
Comment 1 Dirk Pranke 2012-08-29 16:21:56 PDT
Created attachment 161352 [details]
Patch
Comment 2 Jessie Berlin 2012-08-29 18:37:44 PDT
Comment on attachment 161352 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=161352&action=review

> Tools/Scripts/webkitpy/layout_tests/port/mac.py:279
> +            files.append(self._filesystem.join(self._webkit_baseline_path('mac-wk2'), 'TestExpectations'))

What about qt-wk2 gtk-wk2? Is there a way to make this generic for them and future wk2 ports?
Comment 3 Dirk Pranke 2012-08-29 18:58:00 PDT
qt has their own logic which already handles this (the qt search path is more complicated).

we could add this for gtk, efl, etc. if they wanted it but right now they haven't indicated a need. chromium doesn't :).

I can push this up into apple.py if you want the same logic across apple mac and apple win, at least.
Comment 4 Jessie Berlin 2012-08-30 10:24:38 PDT
(In reply to comment #3)
> qt has their own logic which already handles this (the qt search path is more complicated).
> 
> we could add this for gtk, efl, etc. if they wanted it but right now they haven't indicated a need. chromium doesn't :).
> 
> I can push this up into apple.py if you want the same logic across apple mac and apple win, at least.

Pushing it up into apple.py would be better, so that it just works when you get NRWT working on Windows :)
Comment 5 Dirk Pranke 2012-08-30 12:21:16 PDT
Created attachment 161529 [details]
Patch
Comment 6 Dirk Pranke 2012-08-30 12:22:19 PDT
Okay, added win-wk2 support as well.

Note that we will still only search in platform/mac and platform/mac-wk2 . Do you want to add support for platform/mac-$VERSION as well?
Comment 7 Dirk Pranke 2012-08-30 12:23:50 PDT
Created attachment 161531 [details]
fix mac.py
Comment 8 Dirk Pranke 2012-08-30 13:01:57 PDT
committed in http://trac.webkit.org/changeset/127174 .

(If you do want mac-$VERSION, let me know and I'll file a separate bug w/ a patch).
Comment 9 Jessie Berlin 2012-08-30 13:16:47 PDT
(In reply to comment #6)
> Okay, added win-wk2 support as well.
> 
> Note that we will still only search in platform/mac and platform/mac-wk2 . Do you want to add support for platform/mac-$VERSION as well?

I am not sure what you mean by mac-$VERSION. For example, do you mean mac-lion-wk2? Or just mac-lion?
Comment 10 Dirk Pranke 2012-08-30 13:21:08 PDT
(In reply to comment #9)
> (In reply to comment #6)
> > Okay, added win-wk2 support as well.
> > 
> > Note that we will still only search in platform/mac and platform/mac-wk2 . Do you want to add support for platform/mac-$VERSION as well?
> 
> I am not sure what you mean by mac-$VERSION. For example, do you mean mac-lion-wk2? Or just mac-lion?

just mac-lion.
Comment 11 Jessie Berlin 2012-08-30 13:54:37 PDT
(In reply to comment #8)
> committed in http://trac.webkit.org/changeset/127174 .
> 
> (If you do want mac-$VERSION, let me know and I'll file a separate bug w/ a patch).

For those watching at home, this broke the layout tests because no TestExpectations file existed for mac-wk2. Dirk is fixing this by adding that file.