Removed dependency between "test" and "jacocoTestReport"

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 <[email protected]>
opensearch-project · Jul 5, 2023 · ec88d2b · ec88d2b
1 parent 4eef662
commit ec88d2b
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
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 }