Enable JaCoCo by default

junit-team · Nov 18, 2024 · a9b0a12 · a9b0a12
1 parent 6082336
commit a9b0a12
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/.github/workflows/cross-version.yml b/.github/workflows/cross-version.yml
@@ -62,6 +62,7 @@ jobs:
         with:
           encryptionKey: ${{ secrets.GRADLE_ENCRYPTION_KEY }}
           arguments: |
+            -Ptesting.enableJaCoCo=false \
             -PjavaToolchain.version=${{ matrix.jdk.version }} \
             -Dscan.tag.JDK_${{ matrix.jdk.version }} \
             build \
@@ -101,6 +102,7 @@ jobs:
         with:
           encryptionKey: ${{ secrets.GRADLE_ENCRYPTION_KEY }}
           arguments: |
+            -Ptesting.enableJaCoCo=false \
             -PjavaToolchain.version=${{ matrix.jdk }} \
             -PjavaToolchain.implementation=j9 \
             -Dscan.tag.JDK_${{ matrix.jdk }} \

diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -34,7 +34,6 @@ jobs:
       with:
         encryptionKey: ${{ secrets.GRADLE_ENCRYPTION_KEY }}
         arguments: |
-          -Ptesting.enableJaCoCo \
           :platform-tooling-support-tests:test \
           build \
           jacocoRootReport \

diff --git a/README.md b/README.md
@@ -49,7 +49,7 @@ builds of the next OpenJDK.
 Code coverage using [JaCoCo] for the latest build is available on [Codecov].
 
 A code coverage report can also be generated locally via the [Gradle Wrapper] by
-executing `./gradlew -Ptesting.enableJaCoCo clean jacocoRootReport`. The results will be available
+executing `./gradlew clean jacocoRootReport`. The results will be available
 in `build/reports/jacoco/jacocoRootReport/html/index.html`.
 
 ## Develocity

diff --git a/gradle/plugins/build-parameters/build.gradle.kts b/gradle/plugins/build-parameters/build.gradle.kts
@@ -67,7 +67,7 @@ buildParameters {
 		description = "Testing related parameters"
 		bool("enableJaCoCo") {
 			description = "Enables JaCoCo test coverage reporting"
-			defaultValue = false
+			defaultValue = true
 		}
 		bool("enableJFR") {
 			description = "Enables Java Flight Recorder functionality"