-
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
Make test harness portable #115
Make test harness portable #115
Conversation
this AbstractIntegrationTest contains several hard-coded path which depends on structure of spotbugs project. To let plugin developers code without limitation like this, it is better to move this Test class to findbugs/src/test.
@@ -0,0 +1,81 @@ | |||
package edu.umd.cs.findbugs; |
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.
we should probably move this to a more appropriate package... maybe edu.umd.cs.findbugs.test
?
@@ -0,0 +1,40 @@ | |||
package edu.umd.cs.findbugs; |
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.
idem
@@ -0,0 +1,118 @@ | |||
package edu.umd.cs.findbugs; |
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.
idem
Currently
AbstractIntegrationTest
intest-harness
has several hard-coded value which depends on project structure of this repository, such as"../findbugsTestCases"
,"lib"
and"/build/classes/main/"
. To make this library portable, it's better to remove them.And one more problem, is that, we provide utilities as abstract class so it might be hard to use with another solution which uses the same way. Basically
Rule
is better choice for JUnit.This pull request proposes:
CountMatcher
AnalysisRunner
ExternalResource
namedSpotBugsRule
AbstractIntegrationTest
tofindbugs
subprojectSpotBugsRule
inFindUnsatisfiedObligationTest