wpt icon indicating copy to clipboard operation
wpt copied to clipboard

Add testdriver support for WebDriver BiDi

Open sadym-chromium opened this issue 1 year ago • 1 comments

Implement RFC 185: Add WebDriver BiDi support to testdriver.js. Based on https://github.com/web-platform-tests/wpt/pull/44649.

Addition to the RFC:

  • Decline and fail test on user prompt in test window.

Open question:

  • How to handle infrastructure/webdriver/bidi tests?

Blocked by:

  • http://crrev.com/c/5568670 till Tue, Jun 11, 2024, when it reaches stable. Tracking.

sadym-chromium avatar Apr 22 '24 13:04 sadym-chromium

This is now ready for reviews. @foolip @jgraham @gsnedders any thoughts?

mathiasbynens avatar Jun 24 '24 12:06 mathiasbynens

I understood that @jgraham wanted to take another look at this

Ms2ger avatar Jul 04 '24 13:07 Ms2ger

@sadym-chromium did you see this error in infastructure/ tests?

Unexpected Results
------------------
/infrastructure/expected-fail/user-prompt.html?type=confirm
  UNEXPECTED-OK /infrastructure/expected-fail/user-prompt.html?type=confirm
ERROR BiDi event processing failed: unexpected alert open (500): unexpected alert open: {Alert text : this user prompt should be dismissed}
  (Session info: chrome=128.0.6583.0)

foolip avatar Jul 15 '24 08:07 foolip

@sadym-chromium did you see this error in infastructure/ tests?

Unexpected Results
------------------
/infrastructure/expected-fail/user-prompt.html?type=confirm
  UNEXPECTED-OK /infrastructure/expected-fail/user-prompt.html?type=confirm
ERROR BiDi event processing failed: unexpected alert open (500): unexpected alert open: {Alert text : this user prompt should be dismissed}
  (Session info: chrome=128.0.6583.0)

this is related to https://crbug.com/42323841, WIP on chrtomedriver side.

sadym-chromium avatar Jul 15 '24 08:07 sadym-chromium