Closed Bug 1833090 Opened 2 years ago Closed 1 years ago

Update bidi wpt tests chunks to be per command/event instead of per module

Categories

(Remote Protocol :: WebDriver BiDi, enhancement, P3)

enhancement
Points:
1

Tracking

(firefox116 fixed)

RESOLVED FIXED
116 Branch
Tracking Status
firefox116 --- fixed

People

(Reporter: jdescottes, Assigned: jdescottes)

References

Details

(Whiteboard: [webdriver:m7])

Attachments

(1 file)

At the moment all BiDi wdspec tests are in the same wd chunk. With Bug 1821996 we will already allow different modules to end up in different chunks.

However since BiDi wdspec tests are more nested than classic tests, it might be better to update chunking logic in order to allow individual commands or events to be in different chunks.

For reference the path to BiDi wdspec tests typically looks like:

webdriver/tests/bidi/{module_name}/{event_or_command_name}/{test_name}.py

whereas classic tests look like (after bug 1821996):

webdriver/tests/classic/{command_name}/{test_name}.py

We can increase the depth by 1 at https://searchfox.org/mozilla-central/rev/eb07633057d66ab25f9db4c5900eeb6913da7579/testing/mozbase/moztest/moztest/resolve.py#839

Summary: Update webdriver bidi CI chunks to be per command/event instead of per module → Update bidi wpt tests chunks to be per command/event instead of per module
Points: --- → 1
Priority: -- → P3
Whiteboard: [webdriver:backlog]

Note that while doing the beta patch for Bug 1821996, we more or less implemented that: https://hg.mozilla.org/releases/mozilla-beta/rev/8a4fde566abb (we add 1 depth for all bidi tests). Note that for mozilla specific tests, the path we need to use does not match the filesystem and is /_mozilla/webdriver/bidi (whereas we have /mozilla/tests/webdriver/bidi for the actual source files).

Given that the patch is written already let's just rebase and land this.

Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Pushed by jdescottes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/64ebc96e2d3f [remote] Update bidi wpt tests chunks to be per command/event instead of per module r=webdriver-reviewers,whimboo
Whiteboard: [webdriver:backlog] → [webdriver:m7]
Status: ASSIGNED → RESOLVED
Closed: 1 years ago
Resolution: --- → FIXED
Target Milestone: --- → 116 Branch
Regressions: 1836972
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: