Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/devtools webelementref #524

Merged
merged 6 commits into from
Aug 31, 2023
Merged

Fix/devtools webelementref #524

merged 6 commits into from
Aug 31, 2023

Conversation

Siolto
Copy link
Collaborator

@Siolto Siolto commented Aug 30, 2023

No description provided.

@Siolto Siolto requested a review from vobu August 30, 2023 16:06
@Siolto Siolto temporarily deployed to internal August 30, 2023 16:36 — with GitHub Actions Inactive
Copy link
Contributor

@vobu vobu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - the failing auth test is friendly fire from the too sophisticated (agh) gh actions env setup

@vobu vobu merged commit ece986f into main.bak Aug 31, 2023
17 of 18 checks passed
Siolto added a commit that referenced this pull request Sep 7, 2023
* fix: safeguard devtools protocol stale web element ref (#524)

* chore(release): 1.5.4

* fix: pass whole selector when renewing the webElementReference

* test: add test for controlRetrieval with whole selector

* fix: handle renewingWebElement gracefully

- update internal wdi5Control properties correctly when element could not be retrieved
- add test for failed webElement renewing

* test: refactor stale tests and invalidate cache

* fix: allControls handling with stale elements

Co-authored-by: Volker Buzek <[email protected]>

* fix: wdi5 v2 compatibility

* fix: formatting

* fix: exclude protocol tests

* chore: export element_key

* fix: make the stale element handling also work for the webdriver protocol

- adjusted the tests to webdriver compatibility
- check for webdriver stale element error message

* docs: remove devtools protocol only description

---------

Co-authored-by: Volker Buzek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants