Address PR Comments

Signed-off-by: Andre Kurait <[email protected]>

.github/workflows/CI.yml

@@ -112,12 +112,13 @@ jobs:
         with:
           name: test-reports
           path: |
+            **/build/reports/tests/
             **/reports/jacoco/mergedReport/
 
       - name: Upload to Codecov
         uses: codecov/codecov-action@v4
         with:
-          file: "./build/reports/jacoco/mergedReport/jacocoMergedReport.xml"
+          files: ${{ github.workspace }}/build/reports/jacoco/mergedReport/jacocoMergedReport.xml
           flags: gradle-test
           fail_ci_if_error: true
           disable_search: true

TrafficCapture/build.gradle

@@ -32,3 +32,16 @@ allprojects {
         }
     }
 }
+
+tasks.named('test', Test) {
+    // Memory leak tests are adding too much execution time on `test` in TrafficCapture
+    // Disabling and will test in `slowTest`
+    systemProperty 'disableMemoryLeakTests', 'true'
+}
+
+tasks.named('slowTest', Test) {
+    useJUnitPlatform {
+        // Ensure rerunning all tests to run with leak detection
+        includeTags = []
+    }
+}

build.gradle

@@ -71,7 +71,6 @@ subprojects {
             showStackTraces true
             showStandardStreams = true
         }
-        systemProperty 'disableMemoryLeakTests', 'true'
 
         // Provide way to exclude particular tests from CLI
         // e.g. ../gradlew test -PexcludeTests=**/KafkaProtobufConsumerLongTermTest*
@@ -99,7 +98,6 @@ subprojects {
         useJUnitPlatform {
             includeTags 'longTest'
         }
-        systemProperty 'disableMemoryLeakTests', 'false'
         jacoco.enabled = true
     }