Bug 213028 – [css-grid] Reimport WPT tests removed in r262809

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Oriol Brufau]

r261841 imported some WPT grid tests, but it was reverted in r262809.
Then these tests have been lost. Should be imported again.

[Oriol Brufau]

Created attachment 401550 [details]
Patch

[Oriol Brufau]

Created attachment 401594 [details]
Patch

[Manuel Rego Casasnovas]

Comment on attachment 401594 [details]
Patch

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

r=me, but please review the comment and change the expectations for that test accordingly.

> LayoutTests/TestExpectations:1059
> +webkit.org/b/209461 imported/w3c/web-platform-tests/css/css-grid/grid-items/grid-item-dynamic-min-contribution-001.html [ Failure ]

I think the usual way to import testharness tests is to have a -expected.txt file with several FAIL messages, instead of marking them as Failure in the TestExpectations file. As that way we'd realize when there's any single change regarding that test, while the other way around we'll only realize when the whole test starts to PASS.

[Oriol Brufau]

(In reply to Manuel Rego Casasnovas from comment #3)
> I think the usual way to import testharness tests is to have a -expected.txt
> file with several FAIL messages, instead of marking them as Failure in the
> TestExpectations file. As that way we'd realize when there's any single
> change regarding that test, while the other way around we'll only realize
> when the whole test starts to PASS.

The problem is that in a subtest, the height of the element becomes (wrongly) the height of the viewport. And the viewport has different heights in different platforms. So it would be messier.

[Oriol Brufau]

Well, maybe it's not the viewport height, but there is some variance. I was getting

FAIL Minimum size: 300% assert_equals: height expected "300px" but got "5400px"

but in an iOS emulator it was something different than 5400px.

[EWS]

Committed r262901: <https://trac.webkit.org/changeset/262901>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 401594 [details].

[Radar WebKit Bug Importer]

<rdar://problem/64251555>