Removed dependency between "test" and "jacocoTestReport"

pawel-gudel-eliatra · pawel-gudel-eliatra · commit ec88d2bca165 · 2023-07-05T12:48:00.000+02:00
Closes issue #2913. This will allow us to not to exclude the "test" job from every other test. Unfortunately, this will break creating the coverage reports from only starting "jacocoTestReport" task in Gradle. Signed-off-by: Pawel Gudel <pawel.gudel@eliatra.com>
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -51,7 +51,6 @@ jobs:
       with:
         arguments: |
           ${{ matrix.gradle_task }} -Dbuild.snapshot=false
-          -x test
 
     - name: Coverage
       uses: codecov/codecov-action@v1
diff --git a/build.gradle b/build.gradle
@@ -390,8 +390,6 @@ tasks.withType(JavaCompile) {
 }
 
 tasks.test.finalizedBy(jacocoTestReport)  // report is always generated after tests run
-tasks.jacocoTestReport.dependsOn(test) // tests are required to run before generating the report
-
 
 allprojects {
     tasks.withType(Javadoc).all { enabled = false }

Original file line number	Diff line number	Diff line change
`@@ -390,8 +390,6 @@ tasks.withType(JavaCompile) {`
`390`	`390`	`}`
`391`	`391`
`392`	`392`	`tasks.test.finalizedBy(jacocoTestReport) // report is always generated after tests run`
`393`		`-tasks.jacocoTestReport.dependsOn(test) // tests are required to run before generating the report`
`394`		`-`
`395`	`393`
`396`	`394`	`allprojects {`
`397`	`395`	`tasks.withType(Javadoc).all { enabled = false }`