Closed Bug 1942644 Opened 25 days ago Closed 24 days ago

[wpt-sync] Sync PR 50177 - [WebDriver BiDi] Fix file test

Categories

(Remote Protocol :: WebDriver BiDi, task, P4)

task

Tracking

(firefox136 fixed)

RESOLVED FIXED
136 Branch
Tracking Status
firefox136 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 50177 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/50177
Details from upstream follow.

Nikolay Vitkov <nvitkov@chromium.org> wrote:

[WebDriver BiDi] Fix file test

I took a look at this failing test and it seems that the test itself is the issue:
From the long we sent two different files:
Files in the first input.setFiles command:

"files": [ "/tmp/wdspec-t_9y6_4n/pytest/wdspec-0/path/to/noop.txt" ]

Files in second input.setFiles command:

 "files": [ "/tmp/wdspec-t_9y6_4n/pytest/wdspec-1/path/to/noop.txt" ]

This means that they will be treated as separate files.

This is happening because https://github.com/web-platform-tests/wpt/blob/fix-file-test/webdriver/tests/bidi/input/set_files/conftest.py#L18 we create a new temp folder in each invocation of the fixture.

Component: web-platform-tests → WebDriver BiDi
Product: Testing → Remote Protocol

CI Results

Ran 12 Firefox configurations based on mozilla-central, and Firefox, and Chrome on GitHub CI

Total 1 tests and 9 subtests

Status Summary

Firefox

OK : 1
PASS: 8
FAIL: 1

Chrome

OK : 1
PASS: 9

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

  • /webdriver/tests/bidi/input/set_files/files.py [wpt.fyi]: OK [Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux2204-64-wayland-debug, Gecko-linux2204-64-wayland-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub], SKIP [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview] (Chrome: OK)
    • test_set_files_twice_same_in_different_folders: FAIL (Chrome: PASS)

Tests Disabled in Gecko Infrastructure

  • /webdriver/tests/bidi/input/set_files/files.py [wpt.fyi]: OK [Gecko-linux1804-64-qr-debug, Gecko-linux1804-64-qr-opt, Gecko-linux2204-64-wayland-debug, Gecko-linux2204-64-wayland-opt, Gecko-windows11-32-2009-qr-debug, Gecko-windows11-32-2009-qr-opt, Gecko-windows11-64-2009-qr-debug, Gecko-windows11-64-2009-qr-opt, GitHub], SKIP [Gecko-android-em-7.0-x86_64-lite-qr-opt-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-geckoview, Gecko-android-em-7.0-x86_64-qr-debug-isolated-process-geckoview, Gecko-android-em-7.0-x86_64-qr-opt-geckoview] (Chrome: OK)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ad22ce147a3b [wpt PR 50177] - [WebDriver BiDi] Fix file test, a=testonly
Status: NEW → RESOLVED
Closed: 24 days ago
Resolution: --- → FIXED
Target Milestone: --- → 136 Branch
You need to log in before you can comment on or make changes to this bug.