1829950 - Add pauses to ensure event sequence in test_modifier_click

Status: RESOLVED FIXED

(Remote Protocol :: Agent, task, P1)

Description

[Julian Descottes [:jdescottes]]

See discussion at https://phabricator.services.mozilla.com/D175869#inline-976104

First we might want to add some pauses in the test_modifier_click wdspec test as well as in its bidi counterpart to make sure that key events happen before/after .

Then there is a question about input sources for an actions sequence. Should we force all input sources to provide the same number of actions?

The BiDi performActions helper adds pauses to enforce this at https://searchfox.org/mozilla-central/source/testing/web-platform/tests/tools/webdriver/webdriver/bidi/modules/input.py#227-230 but Classic does not.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1829950 - [wdspec] Add pauses in pointer_modifier tests to ensure event sequence

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2d54796666db
[wdspec] Add pauses in pointer_modifier tests to ensure event sequence r=webdriver-reviewers,jgraham

Comment 3

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

Failed to create upstream wpt PR due to merge conflicts. This requires fixup from a wpt sync admin.

Comment 4

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/2d54796666db

Comment 5

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

James, the upstream PR was never created. Can you please have a look?

Comment 6

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

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

Comment 7

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

Upstream PR merged by moz-wptsync-bot