Page MenuHomePhabricator

Bug 1887644 - [wdspec] Add a test for non-existent files for "input.setFiles" command.
ClosedPublic

Authored by Sasha on May 23 2024, 10:53 AM.
Referenced Files
Unknown Object (File)
Thu, Apr 30, 4:37 PM
Unknown Object (File)
Mon, Apr 27, 6:03 AM
Unknown Object (File)
Sat, Apr 25, 10:37 AM
Unknown Object (File)
Thu, Apr 23, 11:29 AM
Unknown Object (File)
Wed, Apr 22, 2:11 PM
Unknown Object (File)
Tue, Apr 21, 6:18 AM
Unknown Object (File)
Mon, Apr 20, 8:28 PM
Unknown Object (File)
Wed, Apr 15, 5:20 AM

Diff Detail

Event Timeline

phab-bot changed the visibility from "Custom Policy" to "Public (No Login Required)".
phab-bot changed the edit policy from "Custom Policy" to "Restricted Project (Project)".
phab-bot removed a project: secure-revision.
whimboo added a subscriber: whimboo.
whimboo added inline comments.
testing/web-platform/mozilla/tests/webdriver/bidi/input/set_files/invalid.py
14 ↗(On Diff #866064)

Do you remember if there will not be such an error with Chrome?

Nevertheless we could still have this as a normal wdspec test and have a dedicated assertion per browser.

This revision now requires changes to proceed.May 23 2024, 12:41 PM
Sasha requested review of this revision.May 23 2024, 2:24 PM
Sasha updated this revision to Diff 866203.
Sasha retitled this revision from Bug 1887644 - Add gecko specific test for non-existent files for "input.setFiles" command. to Bug 1887644 - [wdspec] Add a test for non-existent files for "input.setFiles" command..
Sasha added inline comments.
testing/web-platform/mozilla/tests/webdriver/bidi/input/set_files/invalid.py
14 ↗(On Diff #866064)

Chrome doesn't throw in this case.
I've created the wdspec test

whimboo added a project: testing-approved.

Thanks! Sad to see that we don't have a consistent behavior here but I assume it will not change and clients have to handle the difference.

This revision is now accepted and ready to land.May 23 2024, 4:58 PM