Bug 113385 - [EFL] Remove a duplicated test from TestExpectations
Summary: [EFL] Remove a duplicated test from TestExpectations
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-27 03:26 PDT by Seokju Kwon
Modified: 2013-03-27 04:12 PDT (History)
5 users (show)

See Also:


Attachments
Patch (1.35 KB, patch)
2013-03-27 03:31 PDT, Seokju Kwon
no flags Details | Formatted Diff | Diff
Patch (1.55 KB, patch)
2013-03-27 03:41 PDT, Seokju Kwon
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Seokju Kwon 2013-03-27 03:26:05 PDT
--lint-test-files warnings:
LayoutTests/platform/efl/TestExpectations:1049 Duplicate or ambiguous entry lines LayoutTests/platform/efl/TestExpectations:473 and LayoutTests/platform/efl/TestExpectations:1049. editing/pasteboard/paste-table-cells.html
Comment 1 Seokju Kwon 2013-03-27 03:31:26 PDT
Created attachment 195259 [details]
Patch
Comment 2 Chris Dumez 2013-03-27 03:36:34 PDT
Comment on attachment 195259 [details]
Patch

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

> LayoutTests/platform/efl/TestExpectations:-473
> -webkit.org/b/102486 editing/pasteboard/paste-table-cells.html [ Failure ]

If you remove this one. Then the test expectation becomes "Missing". However, if I look at the LayoutTests folder, there is an expected result:
 LayoutTests/editing/pasteboard/paste-table-cells-expected.txt

Therefore, the correct expected result is probably "Failure"?
Comment 3 Seokju Kwon 2013-03-27 03:41:41 PDT
Created attachment 195262 [details]
Patch
Comment 4 WebKit Review Bot 2013-03-27 04:12:35 PDT
Comment on attachment 195262 [details]
Patch

Clearing flags on attachment: 195262

Committed r146978: <http://trac.webkit.org/changeset/146978>
Comment 5 WebKit Review Bot 2013-03-27 04:12:43 PDT
All reviewed patches have been landed.  Closing bug.