-
-
Notifications
You must be signed in to change notification settings - Fork 18
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 default classpath to enable type resolution #154
Conversation
2d0e0d7
to
82d0ace
Compare
82d0ace
to
6740e88
Compare
@@ -15,18 +16,29 @@ import java.nio.file.Paths | |||
threadSafe = true, | |||
requiresDependencyCollection = ResolutionScope.TEST) | |||
class CheckMojo : DetektMojo() { | |||
lateinit var cliArgs: CliArgs |
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.
set to a field for testing purposes. Maybe there is another smart idea of how we can assert the expected outcome.
class CheckMojoSpec : Spek({ | ||
val invalidPackageNamingDirectoryPath by lazy { |
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.
Extracted a test factors to separate concerns, reduce duplication and improve readability
@@ -0,0 +1 @@ | |||
package `code-samples`.`valid` |
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.
Created a valid input to execute a full testrun. Maybe this is not necessary - not sure.
Codecov Report
@@ Coverage Diff @@
## main #154 +/- ##
============================================
+ Coverage 75.17% 75.33% +0.16%
- Complexity 22 31 +9
============================================
Files 8 8
Lines 145 150 +5
Branches 37 38 +1
============================================
+ Hits 109 113 +4
+ Misses 12 11 -1
- Partials 24 26 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
No description provided.