[py] Add test for Data URL in BiDi Network request handler

[cgoldberg]

User description

🔗 Related Issues

#16279

💥 What does this PR do?

This PR adds a new web page to the test suite that contains an image embedded as a Data URL, and a Python test that navigates to the page with a BiDi Network request handler enabled. The test is currently marked as an expected failure because it will fail in all browsers. Once browsers can handle this feature we will enable the test.

See: w3c/webdriver-bidi#727

🔄 Types of changes

• Test

---

PR Type

Tests

---

Description

• Add test for BiDi Network request handler with Data URLs

• Create HTML test page with embedded Data URL image

• Mark test as expected failure across all browsers

• Test validates Data URL handling in network requests

---

Diagram Walkthrough

flowchart LR
  A["Test File"] --> B["Add Data URL Test"]
  C["HTML Page"] --> D["Embed Data URL Image"]
  B --> E["Mark as Expected Failure"]
  D --> B

Loading

File Walkthrough

Relevant files

Tests

bidi_network_tests.py Add Data URL BiDi network test                                                      py/test/selenium/webdriver/common/bidi_network_tests.py Import pytest module for test markers Add new test function test_handler_with_data_url_request Mark test as expected failure for Chrome, Edge, and Firefox Test navigates to data URL page with BiDi network handler +14/-0    data_url.html Create Data URL test page                                                                common/src/web/data_url.html Create new HTML test page with Data URL embedded image Include base64-encoded GIF image as data URL Add proper HTML structure with meta tags and title +10/-0

---

[qodo-code-review]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review

[cgoldberg]

Thanks.. I didn't realize the exception was not on the main thread and execution continues.

I just updated the test so it will fail if continue_request() raises an exception. I left the other assertion and stuff in, because it's testing that the entire capture worked, even if it's not the specific purpose for the test.

[navin772]

This sounds good. I noticed that the test is still passing but mainly due to async timing issue, the exceptions assertion is being done before the callback has finished appending the exception, maybe because its running on a different thread and we don't wait for it to complete.
Adding a time.sleep(1) after navigation fixes the issue but it would be great if we can avoid sleep.

[cgoldberg]

I added the time.sleep(1) for now. Async and threaded code is definitely tricky to test.