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
Add failing test case for class generated by Groovy compiler #2146
Add failing test case for class generated by Groovy compiler #2146
Conversation
This modifies spotbugsTestCases/build.gradle to add Groovy support and cleans up / refactors the AbstractIntegrationTest to read .class files generated by the Groovy compiler. At the same time simplifies some logic and switches use of java.io.File over to the java.nio.file package instead.
Signed-off-by: Kengo TODA <skypencil@gmail.com>
…ss-test-case-error
Signed-off-by: Kengo TODA <skypencil@gmail.com>
Signed-off-by: Kengo TODA <skypencil@gmail.com>
de39725
to
d48728d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your PR!
I confirmed that the bug comes from 771f26f which assumes the size of field is always 1. I will wait for review from other teammates to merge this fix and your test case. I had misunderstanding about the intention of existing code, I will reconsider my fix.
Signed-off-by: Kengo TODA <skypencil@gmail.com>
Signed-off-by: Kengo TODA <skypencil@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a new fix that treats dup_x1
and dup_x2
in a better way.
This PR adds a failing test case for Groovy class that is causing Spotbugs to throw
edu.umd.cs.findbugs.ba.CFGBuilderException
. It does not fix the issue. Refer to #2145spotbugsTestCases/build.gradle
AbstractIntegrationTest
has been cleaned up / refactored to additionally search for.class
files generated by the Groovy compiler. At the same time simplifies some fall-back search path logic (e.g. presumably left over code for nonexistentjava9/
directory) and switches use ofjava.io.File
over to thejava.nio.file
package instead.Make sure these boxes are checked before submitting your PR -- thank you!
CHANGELOG.md
if you have changed SpotBugs codeNOTE: No fix in this PR