-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Issue #2934: Setup unit test coverage reports #3916
Conversation
eng/testcoverage/pom.xml
Outdated
<version>1.0.0-SNAPSHOT</version> | ||
|
||
<name>Microsoft Azure client library for Application Configuration</name> | ||
<description>This package contains the Microsoft Azure Application Configuration client library.</description> |
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.
Previous two lines are copy/paste fails.
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.
Updated
pom.client.xml
Outdated
@@ -630,6 +649,7 @@ | |||
<modules> | |||
<module>./eng/spotbugs-aggregate-report</module> | |||
<module>./eng/code-quality-reports</module> | |||
<module>./eng/testcoverage</module> |
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.
For naming consistency, can this be 'jacoco-test-coverage'
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.
Done
@@ -201,7 +201,7 @@ jobs: | |||
displayName: 'Run tests' | |||
inputs: | |||
mavenPomFile: $(pomFile) | |||
options: '$(DefaultOptions) $(ProfileFlag)' | |||
options: '$(DefaultOptions) $(ProfileFlag) -Dinclude-non-shipping-modules' |
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.
I'm not sure what this change does, will rely on @JonathanGiles insight here
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.
👍✨Ship It✨
Note: Please squash and merge this pull request.
This change is to setup JaCoCo for reporting unit test coverage for all track 2 modules.
When this is released, unit test coverage reports should be available at the link below:
https://azuresdkartifacts.blob.core.windows.net/azure-sdk-for-java/staging/test-coverage/index.html