[FEATURE] add code coverage report for commons and memory modules (#1574

) * Test CI for codecov Signed-off-by: TrungBui59 <[email protected]> * Change the gradle files for codecov Signed-off-by: TrungBui59 <[email protected]> * Fixing gradle file to support for codecov Signed-off-by: TrungBui59 <[email protected]> * change the todo Signed-off-by: TrungBui59 <[email protected]> --------- Signed-off-by: TrungBui59 <[email protected]>
opensearch-project · Nov 2, 2023 · 510f50b · 510f50b
1 parent 882246c
commit 510f50b
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 15 deletions.
diff --git a/.github/workflows/CI-workflow.yml b/.github/workflows/CI-workflow.yml
@@ -164,6 +164,12 @@ jobs:
             ./gradlew integTest -Dtests.rest.cluster=localhost:9200 -Dtests.cluster=localhost:9200 -Dtests.clustername="docker-cluster"
           fi
 
+      - name: Upload Coverage Report
+        uses: codecov/codecov-action@v3
+        with:
+          flags: ml-commons
+          token: ${{ secrets.CODECOV_TOKEN }}
+
   Build-ml-windows:
     strategy:
       matrix:

diff --git a/common/build.gradle b/common/build.gradle
@@ -29,7 +29,9 @@ lombok {
 
 jacocoTestReport {
     reports {
-        html.required = true
+        xml.getRequired().set(true)
+        csv.getRequired().set(false)
+        html.getRequired().set(true)
     }
 
     dependsOn test

diff --git a/memory/build.gradle b/memory/build.gradle
@@ -46,37 +46,31 @@ test {
 
 jacocoTestReport {
     reports {
-        html.required = true
-        html.outputLocation = layout.buildDirectory.dir('jacocoHtml')
+        xml.getRequired().set(true)
+        csv.getRequired().set(false)
+        html.getRequired().set(true)
     }
 
     dependsOn test
 }
 
-List<String> jacocoExclusions = []
 
 jacocoTestCoverageVerification {
     violationRules {
         rule {
-            element = 'CLASS'
-            excludes = jacocoExclusions
             limit {
-                counter = 'BRANCH'
-                minimum = 0.7  //TODO: change this value to 0.7
+                counter = 'LINE'
+                minimum = 0.7 //TODO: increase coverage to 0.8
             }
-        }
-        rule {
-            element = 'CLASS'
-            excludes = jacocoExclusions
             limit {
-                counter = 'LINE'
-                value = 'COVEREDRATIO'
-                minimum = 0.8  //TODO: change this value to 0.8
+                counter = 'BRANCH'
+                minimum = 0.8 //TODO: increase coverage to 0.9
             }
         }
     }
     dependsOn jacocoTestReport
 }
+check.dependsOn jacocoTestCoverageVerification
 
 spotless {
     java {