-
Notifications
You must be signed in to change notification settings - Fork 578
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 RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT #872
Comments
Are you sure that the bug detector is complaining about the My workplace uses Mockito all the time and I haven't seen this cause problems. |
Yes it is complaining about the convert() which is called with verify(). I just have another place where i have exact the same problem.
|
+1 |
+1
|
meet same with mockito.verify. do you have plans to solve it? |
Is there any update on the fix for this bug for mockito.verify use case ? |
This code leads to:
The verify method does effect the code - it breaks the unit test if the method is not called 1 time. Its more or less like an assert.
I will workaround this now with @CheckReturnValue but for me, this is a very common use case. Is it possible to fix that in spotbugs? Thanks!
The text was updated successfully, but these errors were encountered: