From 1084d930e9f9393c414722a405a7c4f70fd97baa Mon Sep 17 00:00:00 2001 From: thsaravana Date: Sun, 8 Sep 2024 17:23:56 +0200 Subject: [PATCH 1/3] Update latest version --- README.md | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/README.md b/README.md index d38de7e..248bdd7 100644 --- a/README.md +++ b/README.md @@ -61,7 +61,7 @@ jobs: - name: Add coverage to PR id: jacoco - uses: madrapps/jacoco-report@v1.6.1 + uses: madrapps/jacoco-report@v1.7.0 with: paths: | ${{ github.workspace }}/**/build/reports/jacoco/prodNormalDebugCoverage/prodNormalDebugCoverage.xml, @@ -111,7 +111,7 @@ refer [jacoco-android-playground](https://github.com/thsaravana/jacoco-android-p ```yaml - name: Jacoco Report to PR id: jacoco - uses: madrapps/jacoco-report@v1.6.1 + uses: madrapps/jacoco-report@v1.7.0 with: paths: ${{ github.workspace }}/build/reports/jacoco/testCoverage/testCoverage.xml token: ${{ secrets.GITHUB_TOKEN }} @@ -132,7 +132,7 @@ refer [jacoco-android-playground](https://github.com/thsaravana/jacoco-android-p ```yaml - name: Jacoco Report to PR id: jacoco - uses: madrapps/jacoco-report@v1.6.1 + uses: madrapps/jacoco-report@v1.7.0 with: paths: | ${{ github.workspace }}/**/build/reports/jacoco/**/prodNormalDebugCoverage.xml, @@ -152,7 +152,7 @@ refer [jacoco-android-playground](https://github.com/thsaravana/jacoco-android-p ```yaml - name: Jacoco Report to PR id: jacoco - uses: madrapps/jacoco-report@v1.6.1 + uses: madrapps/jacoco-report@v1.7.0 with: paths: ${{ github.workspace }}/build/reports/jacoco/testCoverage/testCoverage.xml token: ${{ secrets.GITHUB_TOKEN }} @@ -162,6 +162,8 @@ refer [jacoco-android-playground](https://github.com/thsaravana/jacoco-android-p pass-emoji: ':green_circle:' fail-emoji: ':red_circle:' ``` + + ## Troubleshooting From a4f5d39babfa3356dda8150767cbc8e7c10a644f Mon Sep 17 00:00:00 2001 From: thsaravana Date: Sun, 8 Sep 2024 17:40:06 +0200 Subject: [PATCH 2/3] Update details about delta value --- README.md | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/README.md b/README.md index 248bdd7..5cb14f0 100644 --- a/README.md +++ b/README.md @@ -76,6 +76,17 @@ jobs:
multi-module screenshot +### Understanding the Coverage Report + +- The "delta" (the negative value next to the coverage) represents the percentage of newly added or modified lines of code that are not covered by unit tests. +It calculates the difference in test coverage based solely on the changes made in the current commit or pull request. +For example, if 10 lines of code are modified or added, and 8 of those lines are covered by unit tests, the "delta" +would be -20%, indicating 20% of the newly changed code is untested. However, the "delta" has limitations. It can never +be positive, meaning if you add more unit tests to cover existing, unmodified code, this additional coverage is +not reflected in the delta. The metric only considers lines directly changed in the current set of modifications, +as there is no mechanism to track improvements in coverage for pre-existing code that hasn't been altered. + + ### Example Project For a working project refer to [jacoco-playgound](https://github.com/thsaravana/jacoco-playground). Check out the PR's From 21a6ec13e34b0f51840f666a701da7f6c0d2e747 Mon Sep 17 00:00:00 2001 From: thsaravana Date: Sun, 8 Sep 2024 18:54:59 +0200 Subject: [PATCH 3/3] Fix lint issue --- README.md | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-) diff --git a/README.md b/README.md index 5cb14f0..273dc97 100644 --- a/README.md +++ b/README.md @@ -78,14 +78,13 @@ jobs: ### Understanding the Coverage Report -- The "delta" (the negative value next to the coverage) represents the percentage of newly added or modified lines of code that are not covered by unit tests. -It calculates the difference in test coverage based solely on the changes made in the current commit or pull request. -For example, if 10 lines of code are modified or added, and 8 of those lines are covered by unit tests, the "delta" -would be -20%, indicating 20% of the newly changed code is untested. However, the "delta" has limitations. It can never -be positive, meaning if you add more unit tests to cover existing, unmodified code, this additional coverage is -not reflected in the delta. The metric only considers lines directly changed in the current set of modifications, -as there is no mechanism to track improvements in coverage for pre-existing code that hasn't been altered. - +- The "delta" (the negative value next to the coverage) represents the percentage of newly added or modified lines of code that are not covered by unit tests. + It calculates the difference in test coverage based solely on the changes made in the current commit or pull request. + For example, if 10 lines of code are modified or added, and 8 of those lines are covered by unit tests, the "delta" + would be -20%, indicating 20% of the newly changed code is untested. However, the "delta" has limitations. It can never + be positive, meaning if you add more unit tests to cover existing, unmodified code, this additional coverage is + not reflected in the delta. The metric only considers lines directly changed in the current set of modifications, + as there is no mechanism to track improvements in coverage for pre-existing code that hasn't been altered. ### Example Project @@ -173,8 +172,6 @@ refer [jacoco-android-playground](https://github.com/thsaravana/jacoco-android-p pass-emoji: ':green_circle:' fail-emoji: ':red_circle:' ``` - - ## Troubleshooting