singhmrinal02 opened a new pull request, #40164:
URL: https://github.com/apache/beam/pull/40164
Addresses #18194.
## What
I looked into this issue and found that the JaCoCo config was only
taking `test.exec` into account.
There are some other test tasks like `needsRunnerTests` and
`validatesRunner` which can also generate `.exec` files. Since those
files weren't being picked up, their coverage wasn't included in the
final report.
So I changed it to pick up all the `.exec` files from the `build/jacoco`
directory instead of only `test.exec`.
## Testing
I ran:
- `./gradlew :buildSrc:compileGroovy` — passed
- `./gradlew :runners:direct-java:needsRunnerTests` — 997 tests completed,
17 failed, 4 skipped
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
