-
Notifications
You must be signed in to change notification settings - Fork 173
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
refactor (gradle analysis) add a test suite and clean up code. #425
Conversation
Codecov Report
@@ Coverage Diff @@
## master #425 +/- ##
==========================================
+ Coverage 42.98% 45.84% +2.85%
==========================================
Files 81 81
Lines 4234 4234
==========================================
+ Hits 1820 1941 +121
+ Misses 2243 2110 -133
- Partials 171 183 +12
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #425 +/- ##
==========================================
+ Coverage 42.98% 45.84% +2.85%
==========================================
Files 81 81
Lines 4234 4234
==========================================
+ Hits 1820 1941 +121
+ Misses 2243 2110 -133
- Partials 171 183 +12
Continue to review full report at Codecov.
|
Gradle analysis has been growing by small PR's for a while and needed to be refactored in order to create a robust testing method. This PR introduced testing for
analyzer/gradle
Discovery and Analysis, and manybuildtool/gradle
functions.This PR also includes a fair amount of renaming within
buildtools/gradle
in order to clarify what the functions are doing and ensure that production vs testing paths are more explicit.This PR fixes the gradle test flake on circle ci.