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

False positive for rescue Exception when multiple classes passed! #299

Open
OrelSokolov opened this issue Aug 30, 2017 · 0 comments
Open

Comments

@OrelSokolov
Copy link
Contributor

rescue RestClient::ExceptionWithResponse, RestClient::Exception => e

fails with Don't rescue Exception
The problem is that parser does not recognize properly last exception class:

["RestClient::ExceptionWithResponse", "const_path_ref", "RestClient", "Exception"]

So, it gives Exception instead of RestClient:Exception

BTW, another example of buggy classname.

rescue RestClient::NotFound, RestClient::Gone, RestClient::Unauthorized, RestClient::Exceptions::ReadTimeout => e
["RestClient::NotFound", "RestClient::Gone", "RestClient::Unauthorized", "const_path_ref", "RestClient::Exceptions", "ReadTimeout"]
@OrelSokolov OrelSokolov changed the title False positive for rescue Exception when multiple classed passed! False positive for rescue Exception when multiple classes passed! Aug 30, 2017
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

No branches or pull requests

1 participant