feat(CONF-CPC-1413): qodana improve coverage

.github/workflows/qodana_code_quality.yml

@@ -31,16 +31,34 @@ jobs:
         run: ./gradlew build
 
       - name: Create coverage-data directory and copy contents
+        id: zipreports
+        if: success() || failure()
         run: |
           mkdir -p .qodana/code-coverage
-          cp -R */build/reports/* .qodana/code-coverage/
+          for service in $(find -maxdepth 1 -type d -name '*service*' -o -name '*gateway*' | sed 's/.\///'); do
+            if [[ -d "$service/build" && -d "$service/build/reports" ]]; then
+              cp -R "$service/build/reports" ".qodana/code-coverage/$service"
+            fi
+          done
 
       - name: Archive coverage data
+        if: success() || failure()
         uses: actions/upload-artifact@v4
         with:
-          name: gradle-coverage-data.zip
+          name: gradle-coverage-data
           path: .qodana/code-coverage
 
+      - name: Jacoco PR Comment
+        uses: madrapps/[email protected]
+        with:
+          paths: ${{ github.workspace }}/**/build/reports/jacoco/test/jacocoTestReport.xml
+          token: ${{ secrets.GITHUB_TOKEN }}
+          min-coverage-overall: 90
+          min-coverage-changed-files: 90
+          title: Jacoco Coverage Report
+          update-comment: true
+          continue-on-error: true
+
 #      - name: 'Qodana Scan'
 #        uses: JetBrains/[email protected]
 #        env:

api-gateway/build.gradle

@@ -67,6 +67,10 @@ jacocoTestReport {
             ])
         }))
     }
+
+    reports {
+        xml.required = true
+    }
 }
 
 jacocoTestCoverageVerification {

auth-service-outdated/build.gradle

@@ -86,6 +86,10 @@ jacocoTestReport {
 			])
 		}))
 	}
+
+	reports {
+		xml.required = true
+	}
 }
 
 jacocoTestCoverageVerification {

auth-service/build.gradle

@@ -75,6 +75,10 @@ jacocoTestReport {
             ])
         }))
     }
+
+    reports {
+        xml.required = true
+    }
 }
 
 jacocoTestCoverageVerification {

billing-service/build.gradle

@@ -63,6 +63,10 @@ jacocoTestReport {
 			])
 		}))
 	}
+
+	reports {
+		xml.required = true
+	}
 }
 
 jacocoTestCoverageVerification {

cart-service/build.gradle

@@ -66,6 +66,10 @@ jacocoTestReport {
             ])
         }))
     }
+
+    reports {
+        xml.required = true
+    }
 }
 
 jacocoTestCoverageVerification {

customers-service-reactive/build.gradle

@@ -48,6 +48,10 @@ jacocoTestReport {
 			])
 		}))
 	}
+
+	reports {
+		xml.required = true
+	}
 }
 
 jacocoTestCoverageVerification {

customers-service/build.gradle

@@ -49,6 +49,10 @@ jacocoTestReport {
 			])
 		}))
 	}
+
+	reports {
+		xml.required = true
+	}
 }
 
 jacocoTestCoverageVerification {

inventory-service/build.gradle

@@ -54,6 +54,10 @@ jacocoTestReport {
             ])
         }))
     }
+
+    reports {
+        xml.required = true
+    }
 }
 
 jacocoTestCoverageVerification {

products-service/build.gradle

@@ -54,6 +54,10 @@ jacocoTestReport {
 			fileTree(dir: it)
 		}))
 	}
+
+	reports {
+		xml.required = true
+	}
 }
 
 jacocoTestCoverageVerification {

vet-service/build.gradle

@@ -55,6 +55,10 @@ jacocoTestReport {
             ])
         }))
     }
+
+    reports {
+        xml.required = true
+    }
 }
 
 jacocoTestCoverageVerification {

vets-service/build.gradle

@@ -57,6 +57,10 @@ jacocoTestReport {
             ])
         }))
     }
+
+    reports {
+        xml.required = true
+    }
 }
 
 jacocoTestCoverageVerification {

visits-service-new/build.gradle

@@ -54,6 +54,10 @@ jacocoTestReport {
             ])
         }))
     }
+
+    reports {
+        xml.required = true
+    }
 }
 
 jacocoTestCoverageVerification {

visits-service-new/src/main/java/com/petclinic/visits/visitsservicenew/BusinessLayer/VisitServiceImpl.java

@@ -502,5 +502,4 @@ public Mono<VisitResponseDTO> patchVisitStatusInVisit(String visitId, String sta
                 .flatMap(entityDtoUtil::toVisitResponseDTO);
     }
 
-
 }

visits-service/build.gradle

@@ -56,6 +56,10 @@ jacocoTestReport {
 			])
 		}))
 	}
+
+	reports {
+		xml.required = true
+	}
 }
 
 jacocoTestCoverageVerification {