CloudRaider
CloudRaider copied to clipboard
Fixed test coverage report issues
What Changed
Bug fixes / additions done as part of this PR:
- Compilation error due to annotation dependency
- Jacoco instrumentation error
- Jacoco code coverage is all zero percent
- Link to code coverage report in README (For this, a coverage-reports folder has been added to cloudraider-core. If only the index.html is added, then the drilldown of the classes in jacoco will not work. Hence the entire jacoco report structure is added)
Why
Todo:
- [ ] Add tests
- [ ] Add docs
- [ ] Add yourself to contributors (run
yarn contributors:add
)
- Compilation error due to annotation dependency (mvn clean install)
Before:
After:
- Jacoco instrumentation error
Before:
After:
- Jacoco code coverage is all zero percent
Before:
After:
This PR addresses https://github.com/intuit/CloudRaider/issues/38
@vaisrinivasan Please check this: https://github.com/intuit/CloudRaider/issues/38#issuecomment-1289493223
@vaisrinivasan Please check this: #38 (comment)
Have addressed the comment. https://github.com/intuit/CloudRaider/issues/38#issuecomment-1291530358
@rattandeep1998 : Just following up to check if this PR is good to be merged :)