Summary: | Second try to prevent Selenium's StaleElementReferenceException | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Test Suite | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | andrewfh, martin.renvoize, victor |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
20.11.00, 20.05.07, 19.11.13
|
|
Bug Depends on: | 26162 | ||
Bug Blocks: | |||
Attachments: |
Bug 26986: Prevent Selenium's StaleElementReferenceException
Bug 26986: Prevent Selenium's StaleElementReferenceException Bug 26986: Prevent Selenium's StaleElementReferenceException |
Description
Jonathan Druart
2020-11-10 10:37:23 UTC
Created attachment 113337 [details] [review] Bug 26986: Prevent Selenium's StaleElementReferenceException This is a follow-up bug for bug 26162 By finding the element before the click I hope to get the good element, even if the page changed in the meanwhile. Created attachment 113360 [details] [review] Bug 26986: Prevent Selenium's StaleElementReferenceException This is a follow-up bug for bug 26162 By finding the element before the click I hope to get the good element, even if the page changed in the meanwhile. Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> As for code change, it's simple and looks good. Created attachment 113688 [details] [review] Bug 26986: Prevent Selenium's StaleElementReferenceException This is a follow-up bug for bug 26162 By finding the element before the click I hope to get the good element, even if the page changed in the meanwhile. Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Run the tests in a loop for a while and couldn't trigger an error.. mind you, I couldn't when I did the same with the first attempt to fix either. Code seems sane, fingers crossed it resolves the issue more long term. Passing QA Pushed to master for 20.11, thanks to everybody involved! Pushed to 20.05.x for 20.05.07 Backported to 19.11.x branch for 19.11.13 |