[Enchancement] Removed dependency between "test" and "jacocoTestRepor…

…t" (opensearch-project#2935)

As a part of discussions in opensearch-project#2861 there's been determined that CI
workflow in Github Actions is excluding a lot of unnecessary task.
@nibix created opensearch-project#2913 to address this problem. Most of the unneeded tasks
has been already removed in opensearch-project#2861. This PR removes last existing part.
@peternied's confirmed removal of dependency in his
[comment](opensearch-project#2913 (comment)).

This change removes the dependency between "_jacocoTestReport_" and
"_test_" tasks. Specifically the dependency that forces to start
"_test_" task always when "_jacocoTestReport_" is called. This allows us
to always generate coverage report in the end of every task run in "CI"
workflow without having to exclude "_test_" task in every single one of
them. Unfortunately this will break functionality to create code
coverage by starting only "_jacocoTestReport_" task. It looks like this
was not used widely and was certainly not used in any of Github Actions
Workflows. This will not break creating coverage reports in the end of
"_test_" task.


Signed-off-by: Pawel Gudel <[email protected]>

.github/workflows/ci.yml

@@ -58,7 +58,6 @@ jobs:
         cache-disabled: true
         arguments: |
           ${{ matrix.gradle_task }} -Dbuild.snapshot=false
-          -x test
 
     - name: Coverage
       uses: codecov/codecov-action@v3

build.gradle

@@ -398,8 +398,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 }