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

Attempt to remove StaleElementReferenceException errors from pipeline #1312

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

johanib
Copy link
Contributor

@johanib johanib commented Nov 12, 2024

Prior to this change, the webtests would randomly fail with
StaleElementReferenceException errors.
This change attempts to work around those errors by adding a retry.
Most of the time, the error is thrown on the consent check.

@johanib johanib force-pushed the feature/experiment_panther_tests branch 3 times, most recently from 5512ba0 to 5d2f4fb Compare November 18, 2024 09:22
Prior to this change, the webtests would randomly fail with
StaleElementReferenceException errors.
This change attempts to work around those errors by adding a retry.
Most of the time, the error is thrown on the consent check.
@johanib johanib force-pushed the feature/experiment_panther_tests branch from 5d2f4fb to f6ca504 Compare November 18, 2024 09:35
@johanib johanib changed the title Feature/experiment panther tests Attempt to remove StaleElementReferenceException errors from pipeline Nov 18, 2024
@johanib johanib marked this pull request as ready for review November 18, 2024 09:45
@johanib johanib requested a review from MKodde November 18, 2024 09:45
@MKodde
Copy link
Contributor

MKodde commented Nov 18, 2024

I remember trying several things to get this test to pass on every build. If this fixes the issue, I'm all for it!

@johanib
Copy link
Contributor Author

johanib commented Nov 18, 2024

I remember trying several things to get this test to pass on every build. If this fixes the issue, I'm all for it!

If it doesn't, Ill remove it straight away. Other things didn't work.

@johanib johanib merged commit cbe71a9 into main Nov 18, 2024
3 checks passed
@johanib johanib deleted the feature/experiment_panther_tests branch November 18, 2024 12:22
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