-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Added NoSuchElementError error instead of TimeoutError when chained .find() command fails #4080
Added NoSuchElementError error instead of TimeoutError when chained .find() command fails #4080
Conversation
Status
|
Hey @dikwickley can you please check if this solution would work ? |
@subhajit20 can you share the test that you are using to test this? |
is this test meaningful? @dikwickley |
@subhajit20 Sure, but the error message is not what we are looking for. |
Hey I have changed the error message into this format as I have seen in the issue description you mentioned. |
hey @garg3133 can you review this pr? |
Fixed in #4193, thanks! |
… command fails
Added NoSuchElementError instead of TimeoutError when chained .find() command fails
fix - #4078
Thanks in advance for your contribution. Please follow the below steps in submitting a pull request, as it will help us with reviewing it quicker.
features/my-new-feature
orissue/123-my-bugfix
);