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

Stale element reference errors should report the element and/or line of code it considers stale #4184

Open
reallymello opened this issue Apr 10, 2024 · 3 comments · Fixed by #4214 · May be fixed by #4287
Open

Stale element reference errors should report the element and/or line of code it considers stale #4184

reallymello opened this issue Apr 10, 2024 · 3 comments · Fixed by #4214 · May be fixed by #4287

Comments

@reallymello
Copy link
Contributor

Description

Often when automating pages that do in place updates of the DOM or if the same selector exists between state transitions Nightwatch may report a stale element reference in the test output during test execution (depending how defensively the test is written). While the error is logged, Nightwatch does not report which element or line of code it is executing that it considers stale which makes troubleshooting and resolving the error difficult--especially in large test suites. Often I have to set breakpoints and slowly narrow in on sections of code until I find it which can be very time consuming in larger test suites.

Suggested solution

Log the selector considered stale and/or the line of code so one can quickly identify what step is encountering a stale element exception.

For newer users, including the possible cause/solution messages similar to what was added for other error types may be helpful as well.

Alternatives / Workarounds

No response

Additional Information

No response

@pujagani
Copy link
Member

Thank you for providing the details. I am looking into it. I was able to come up with a solution that logs the selector for stale element reference error. I am now trying to enhance that to add the link of code to it.

@reallymello
Copy link
Contributor Author

Amazing, thank you

@garg3133
Copy link
Member

Original PR reverted.

@garg3133 garg3133 reopened this Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants