1847578 - Correct wdspec meta data for various platforms

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, task, P1)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

We haven't done a cleanup of the wdspec meta data for a while and a lot of probably no longer needed expectations exist. Lets see which we can remove per platform. I'll attach individual patches.

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1847578 - [wdspec] Correct meta data for Mac.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1847578 - [wdspec] Cleanup meta data for Linux.

Depends on D185567

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

I'm going to land the patches individually to ensure that if regressions happen we can minimize the effect.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/16499cdce160
[wdspec] Correct meta data for Mac. r=webdriver-reviewers,jdescottes

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4eeb6ee5cd26
[wdspec] Cleanup meta data for Linux. r=webdriver-reviewers,jdescottes

Comment 6

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/16499cdce160

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1847578 - [wdspec] Cleanup meta data for Windows.

Comment 8

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/4eeb6ee5cd26

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1847578 - [wdspec] Cleanup meta data for Android.

Comment 10

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1847578 - [wdspec] Cleanup meta data for remaining platforms.

Depends on D185806

Comment 11

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1847578 - [wdspec] Cleanup __dir__.ini files for WebDriver classic tests.

Depends on D185926

Comment 12

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Both the patches for MacOS and Linux were completely or partly reverted by the last WPT downstream sync. The underlying issue has been identified and will be fixed via bug 1848301.

I'll check if both the Mac and Linux patches will still apply cleanly, but I expect some merge conflicts need to be fixed.

Comment 13

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1847578 - [wdspec] Cleanup meta data for Linux.

Depends on D185567

Comment 14

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1f9641d850fc
[wdspec] Correct meta data for Mac. r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/428c548b119a
[wdspec] Cleanup meta data for Linux. r=webdriver-reviewers,jdescottes

Comment 15

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/1f9641d850fc
https://hg.mozilla.org/mozilla-central/rev/428c548b119a

Comment 16

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d8bab3ffae21
[wdspec] Cleanup meta data for Windows. r=webdriver-reviewers,Sasha
https://hg.mozilla.org/integration/autoland/rev/eaaee4cd571a
[wdspec] Cleanup meta data for Android. r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/a915064d96e6
[wdspec] Cleanup meta data for remaining platforms. r=webdriver-reviewers,jgraham
https://hg.mozilla.org/integration/autoland/rev/1318c0be0d81
[wdspec] Cleanup __dir__.ini files for WebDriver classic tests. r=webdriver-reviewers,Sasha

Comment 17

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/41436 for changes under testing/web-platform/tests

Comment 18

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Bug 1847578 - [wdspec] Remove inappropriately added actions touch scoll testcase.

Comment 19

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 20

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to Web Platform Test Sync Bot (Matrix: #interop:mozilla.org) from comment #17)

Created web-platform-tests PR
https://github.com/web-platform-tests/wpt/pull/41436 for changes under
testing/web-platform/tests

The test file got added by accident. I'm going to revert it in a follow-up patch already attached on this bug.

Comment 21

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/69e526d0c2f3
[wdspec] Remove inappropriately added actions touch scoll testcase. r=webdriver-reviewers,Sasha DONTBUILD

Comment 22

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/d8bab3ffae21
https://hg.mozilla.org/mozilla-central/rev/eaaee4cd571a
https://hg.mozilla.org/mozilla-central/rev/a915064d96e6
https://hg.mozilla.org/mozilla-central/rev/1318c0be0d81

Comment 23

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Once the final commit reaches mozilla-central the bug can be closed as fixed.

Comment 24

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 25

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/69e526d0c2f3