[wdspec] Improve and add missing "no such element" error tests for WebDriver classic commands where needed
Categories
(Testing :: geckodriver, enhancement, P1)
Tracking
(firefox108 fixed)
Tracking | Status | |
---|---|---|
firefox108 | --- | fixed |
People
(Reporter: whimboo, Assigned: whimboo)
References
Details
(Whiteboard: [webdriver:m5], [wptsync upstream])
Attachments
(1 file, 3 obsolete files)
(deleted),
text/x-phabricator-request
|
Details |
On bug 1772484 the stale element reference
tests will be improved and a lot of new ones are getting added for commands that require the check but missed to have such a test.
This bug covers the counter part for no such element
errors. Getting it landed before bug 1692468 doesn't seem to be possible and as such delaying it looks like the better strategy here.
Assignee | ||
Comment 1•2 years ago
|
||
Before we can push possible patches on this bug we will have to wait for an agreement on bug https://github.com/w3c/webdriver/pull/1667.
Assignee | ||
Comment 2•2 years ago
|
||
Because the browsing context gets destroyed the list of known
elements will be gone and the new browsing context will not have
any known elements. As such referring to an element id from the
former browsing context has to fail with a no such element
error.
Depends on D149268
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Updated•2 years ago
|
Updated•2 years ago
|
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 6•2 years ago
|
||
The changes for bug 1692468 shouldn't affect the current behavior of our WebDriver classic implementation. As such I want to see these tests to be added first to ensure that no regression will happen.
Assignee | ||
Comment 7•2 years ago
|
||
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/77a9f0ad9a10 [wdspec] Enhance "no such element" tests for WebDriver classic commands. r=webdriver-reviewers,jdescottes
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/36761 for changes under testing/web-platform/tests
Comment 10•2 years ago
|
||
bugherder |
Upstream PR merged by moz-wptsync-bot
Updated•2 years ago
|
Description
•