From 7bf416e5d78a0756e4cd61bcb0628f8ffc72c12f Mon Sep 17 00:00:00 2001 From: thsaravana Date: Mon, 22 Mar 2021 20:59:58 +0530 Subject: [PATCH 01/14] Test large file --- .github/workflows/coverage.yml | 9 +++++++++ testPlaystoreProductionDebugUnitTestCoverage.xml | 1 + 2 files changed, 10 insertions(+) create mode 100755 testPlaystoreProductionDebugUnitTestCoverage.xml diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 78618e9..fcf464b 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -40,3 +40,12 @@ jobs: run: | echo "Total coverage ${{ steps.jacoco.outputs.coverage-overall }}" echo "Changed Files coverage ${{ steps.jacoco.outputs.coverage-changed-files }}" + + - name: Jacoco Report to PR 1 + id: jacoco1 + uses: madrapps/jacoco-report@v1 + with: + path: ${{ github.workspace }}/testPlaystoreProductionDebugUnitTestCoverage.xml + token: ${{ secrets.GITHUB_TOKEN }} + min-coverage-overall: 11 + min-coverage-changed-files: 70 diff --git a/testPlaystoreProductionDebugUnitTestCoverage.xml b/testPlaystoreProductionDebugUnitTestCoverage.xml new file mode 100755 index 0000000..08cab98 --- /dev/null +++ b/testPlaystoreProductionDebugUnitTestCoverage.xml @@ -0,0 +1 @@ + \ No newline at end of file From dc736a90572e36ead1d6e6f2df0ce9269365ee4f Mon Sep 17 00:00:00 2001 From: thsaravana Date: Mon, 22 Mar 2021 22:15:01 +0530 Subject: [PATCH 02/14] Adding debug-mode --- .github/workflows/coverage.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index fcf464b..f332b69 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -49,3 +49,4 @@ jobs: token: ${{ secrets.GITHUB_TOKEN }} min-coverage-overall: 11 min-coverage-changed-files: 70 + debug-mode: true From afeb2dfcecf906746abbe83d29083c5ce534d446 Mon Sep 17 00:00:00 2001 From: thsaravana Date: Mon, 22 Mar 2021 22:19:10 +0530 Subject: [PATCH 03/14] Pointing to v1.1 --- .github/workflows/coverage.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index f332b69..69a8252 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -29,7 +29,7 @@ jobs: - name: Jacoco Report to PR id: jacoco - uses: madrapps/jacoco-report@v1 + uses: madrapps/jacoco-report@v1.1 with: path: ${{ github.workspace }}/build/reports/jacoco/testCoverage/testCoverage.xml token: ${{ secrets.GITHUB_TOKEN }} @@ -43,7 +43,7 @@ jobs: - name: Jacoco Report to PR 1 id: jacoco1 - uses: madrapps/jacoco-report@v1 + uses: madrapps/jacoco-report@v1.1 with: path: ${{ github.workspace }}/testPlaystoreProductionDebugUnitTestCoverage.xml token: ${{ secrets.GITHUB_TOKEN }} From 7b4da8b83af6055e39728a13a1022c51210d03a4 Mon Sep 17 00:00:00 2001 From: thsaravana Date: Tue, 23 Mar 2021 00:05:18 +0530 Subject: [PATCH 04/14] Reverting the code to older version --- .github/workflows/coverage.yml | 11 +---------- testPlaystoreProductionDebugUnitTestCoverage.xml | 1 - 2 files changed, 1 insertion(+), 11 deletions(-) delete mode 100755 testPlaystoreProductionDebugUnitTestCoverage.xml diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 69a8252..a24a65a 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -35,18 +35,9 @@ jobs: token: ${{ secrets.GITHUB_TOKEN }} min-coverage-overall: 40 min-coverage-changed-files: 60 + debug-mode: false - name: Get the Coverage info run: | echo "Total coverage ${{ steps.jacoco.outputs.coverage-overall }}" echo "Changed Files coverage ${{ steps.jacoco.outputs.coverage-changed-files }}" - - - name: Jacoco Report to PR 1 - id: jacoco1 - uses: madrapps/jacoco-report@v1.1 - with: - path: ${{ github.workspace }}/testPlaystoreProductionDebugUnitTestCoverage.xml - token: ${{ secrets.GITHUB_TOKEN }} - min-coverage-overall: 11 - min-coverage-changed-files: 70 - debug-mode: true diff --git a/testPlaystoreProductionDebugUnitTestCoverage.xml b/testPlaystoreProductionDebugUnitTestCoverage.xml deleted file mode 100755 index 08cab98..0000000 --- a/testPlaystoreProductionDebugUnitTestCoverage.xml +++ /dev/null @@ -1 +0,0 @@ - \ No newline at end of file From 58f3f237a1aca9c54c15fc94eee524a37510201c Mon Sep 17 00:00:00 2001 From: thsaravana Date: Fri, 30 Apr 2021 18:35:15 +0530 Subject: [PATCH 05/14] Add reviewers yml --- .github/workflows/add-reviewers.yml | 29 +++++++++++++++++++ .../com/madrapps/jacoco/StringOperation.java | 7 +++++ .../madrapps/jacoco/operation/StringOp.java | 6 +++- src/main/kotlin/com/madrapps/jacoco/Math.kt | 10 +++---- .../com/madrapps/jacoco/MathOperation.kt | 12 ++++++++ 5 files changed, 58 insertions(+), 6 deletions(-) create mode 100644 .github/workflows/add-reviewers.yml create mode 100644 src/main/java/com/madrapps/jacoco/StringOperation.java create mode 100644 src/main/kotlin/com/madrapps/jacoco/MathOperation.kt diff --git a/.github/workflows/add-reviewers.yml b/.github/workflows/add-reviewers.yml new file mode 100644 index 0000000..56f356d --- /dev/null +++ b/.github/workflows/add-reviewers.yml @@ -0,0 +1,29 @@ +name: Measure coverage + +on: + push: + branches: [ main ] + pull_request: + +jobs: + build: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v2 + - name: Set up JDK 1.8 + uses: actions/setup-java@v1 + with: + java-version: 1.8 + + - name: Grant execute permission for gradlew + run: chmod +x gradlew + + - name: Run Coverage + run: ./gradlew testCoverage + + - name: Add Reviewers to PR + uses: madrapps/add-reviewers@main + with: + token: ${{ secrets.GITHUB_TOKEN }} + reviewers: saravana-thiyagaraj,instrap + debug-mode: true diff --git a/src/main/java/com/madrapps/jacoco/StringOperation.java b/src/main/java/com/madrapps/jacoco/StringOperation.java new file mode 100644 index 0000000..df85f61 --- /dev/null +++ b/src/main/java/com/madrapps/jacoco/StringOperation.java @@ -0,0 +1,7 @@ +package com.madrapps.jacoco; + +public interface StringOperation { + boolean endsWith(String source, String chars); + + boolean startsWith(String source, String chars); +} diff --git a/src/main/java/com/madrapps/jacoco/operation/StringOp.java b/src/main/java/com/madrapps/jacoco/operation/StringOp.java index be10d58..86d0730 100644 --- a/src/main/java/com/madrapps/jacoco/operation/StringOp.java +++ b/src/main/java/com/madrapps/jacoco/operation/StringOp.java @@ -1,11 +1,15 @@ package com.madrapps.jacoco.operation; -public class StringOp { +import com.madrapps.jacoco.StringOperation; +public class StringOp implements StringOperation { + + @Override public boolean endsWith(String source, String chars) { return source.endsWith(chars); } + @Override public boolean startsWith(String source, String chars) { return source.startsWith(chars); } diff --git a/src/main/kotlin/com/madrapps/jacoco/Math.kt b/src/main/kotlin/com/madrapps/jacoco/Math.kt index 9352ca2..e798382 100644 --- a/src/main/kotlin/com/madrapps/jacoco/Math.kt +++ b/src/main/kotlin/com/madrapps/jacoco/Math.kt @@ -1,20 +1,20 @@ package com.madrapps.jacoco -class Arithmetic { +class Arithmetic : MathOperation { - fun add(a: Int, b: Int): Int { + override fun add(a: Int, b: Int): Int { return a + b } - fun subtract(a: Int, b: Int): Int { + override fun subtract(a: Int, b: Int): Int { return a - b } - fun multiply(a: Int, b: Int): Int { + override fun multiply(a: Int, b: Int): Int { return a * b } - fun divide(a: Int, b: Int): Int { + override fun divide(a: Int, b: Int): Int { return a / b } diff --git a/src/main/kotlin/com/madrapps/jacoco/MathOperation.kt b/src/main/kotlin/com/madrapps/jacoco/MathOperation.kt new file mode 100644 index 0000000..dc28882 --- /dev/null +++ b/src/main/kotlin/com/madrapps/jacoco/MathOperation.kt @@ -0,0 +1,12 @@ +package com.madrapps.jacoco + +interface MathOperation { + fun add(a: Int, b: Int): Int + fun subtract(a: Int, b: Int): Int + fun multiply(a: Int, b: Int): Int + fun divide(a: Int, b: Int): Int + + interface MathOp { + + } +} \ No newline at end of file From 46959ef6fddb2d7d40ef21deefe44fe3a88944d0 Mon Sep 17 00:00:00 2001 From: thsaravana Date: Fri, 30 Apr 2021 18:46:48 +0530 Subject: [PATCH 06/14] Changing name of the workflow --- .github/workflows/add-reviewers.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/add-reviewers.yml b/.github/workflows/add-reviewers.yml index 56f356d..038875d 100644 --- a/.github/workflows/add-reviewers.yml +++ b/.github/workflows/add-reviewers.yml @@ -1,4 +1,4 @@ -name: Measure coverage +name: Add Requester on: push: From edc9d418c1999eac6c9769c62e537daab201c42b Mon Sep 17 00:00:00 2001 From: thsaravana Date: Fri, 30 Apr 2021 18:58:40 +0530 Subject: [PATCH 07/14] Remove coverage step --- .github/workflows/add-reviewers.yml | 3 --- 1 file changed, 3 deletions(-) diff --git a/.github/workflows/add-reviewers.yml b/.github/workflows/add-reviewers.yml index 038875d..852b825 100644 --- a/.github/workflows/add-reviewers.yml +++ b/.github/workflows/add-reviewers.yml @@ -18,9 +18,6 @@ jobs: - name: Grant execute permission for gradlew run: chmod +x gradlew - - name: Run Coverage - run: ./gradlew testCoverage - - name: Add Reviewers to PR uses: madrapps/add-reviewers@main with: From f0db3e907cdb826be11efaae15dc738a9b991d75 Mon Sep 17 00:00:00 2001 From: thsaravana Date: Fri, 30 Apr 2021 19:09:01 +0530 Subject: [PATCH 08/14] Try out --- .github/workflows/add-reviewers.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/add-reviewers.yml b/.github/workflows/add-reviewers.yml index 852b825..a835b71 100644 --- a/.github/workflows/add-reviewers.yml +++ b/.github/workflows/add-reviewers.yml @@ -18,7 +18,7 @@ jobs: - name: Grant execute permission for gradlew run: chmod +x gradlew - - name: Add Reviewers to PR + - name: Add Reviewers uses: madrapps/add-reviewers@main with: token: ${{ secrets.GITHUB_TOKEN }} From 3bff9f56b7ee66c301dc48a0fbf2b93fcd3968fe Mon Sep 17 00:00:00 2001 From: thsaravana Date: Fri, 30 Apr 2021 19:20:58 +0530 Subject: [PATCH 09/14] Adding more reviewers --- .github/workflows/add-reviewers.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/add-reviewers.yml b/.github/workflows/add-reviewers.yml index a835b71..7401073 100644 --- a/.github/workflows/add-reviewers.yml +++ b/.github/workflows/add-reviewers.yml @@ -22,5 +22,5 @@ jobs: uses: madrapps/add-reviewers@main with: token: ${{ secrets.GITHUB_TOKEN }} - reviewers: saravana-thiyagaraj,instrap + reviewers: saravana-thiyagaraj,instrap,muralikrish91,thsaravana debug-mode: true From caf4dbd62c6d540b16df09fb3fa8bc5112a33c5b Mon Sep 17 00:00:00 2001 From: thsaravana Date: Thu, 6 Jan 2022 16:24:36 +0530 Subject: [PATCH 10/14] Testing Jacoco Report missing SHA --- .github/workflows/coverage.yml | 4 ++-- src/main/java/com/madrapps/jacoco/operation/StringOp.java | 4 ++++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index a24a65a..63c46a4 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -29,13 +29,13 @@ jobs: - name: Jacoco Report to PR id: jacoco - uses: madrapps/jacoco-report@v1.1 + uses: madrapps/jacoco-report@v1.3_baseSHA with: path: ${{ github.workspace }}/build/reports/jacoco/testCoverage/testCoverage.xml token: ${{ secrets.GITHUB_TOKEN }} min-coverage-overall: 40 min-coverage-changed-files: 60 - debug-mode: false + debug-mode: true - name: Get the Coverage info run: | diff --git a/src/main/java/com/madrapps/jacoco/operation/StringOp.java b/src/main/java/com/madrapps/jacoco/operation/StringOp.java index 86d0730..a1e92d7 100644 --- a/src/main/java/com/madrapps/jacoco/operation/StringOp.java +++ b/src/main/java/com/madrapps/jacoco/operation/StringOp.java @@ -13,4 +13,8 @@ public boolean endsWith(String source, String chars) { public boolean startsWith(String source, String chars) { return source.startsWith(chars); } + + public void sampleMethod() { + + } } From 856000aeafd25a459611d076ec04e91d68e9f3b3 Mon Sep 17 00:00:00 2001 From: thsaravana Date: Thu, 6 Jan 2022 16:29:20 +0530 Subject: [PATCH 11/14] Testing Jacoco Report missing SHA --- .github/workflows/coverage.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 63c46a4..188319e 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -29,7 +29,7 @@ jobs: - name: Jacoco Report to PR id: jacoco - uses: madrapps/jacoco-report@v1.3_baseSHA + uses: madrapps/jacoco-report@v1.2 with: path: ${{ github.workspace }}/build/reports/jacoco/testCoverage/testCoverage.xml token: ${{ secrets.GITHUB_TOKEN }} From 7fb8fbd05950afa4887e71d580e211e35193a3af Mon Sep 17 00:00:00 2001 From: thsaravana Date: Thu, 6 Jan 2022 16:31:09 +0530 Subject: [PATCH 12/14] Testing Jacoco Report missing SHA --- .github/workflows/coverage.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 188319e..1c5fc7d 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -29,9 +29,9 @@ jobs: - name: Jacoco Report to PR id: jacoco - uses: madrapps/jacoco-report@v1.2 + uses: madrapps/jacoco-report@v1.3_baseSHA with: - path: ${{ github.workspace }}/build/reports/jacoco/testCoverage/testCoverage.xml + paths: ${{ github.workspace }}/build/reports/jacoco/testCoverage/testCoverage.xml token: ${{ secrets.GITHUB_TOKEN }} min-coverage-overall: 40 min-coverage-changed-files: 60 From 5feb41d9631c2740e794d186bd3717fc50aeebf1 Mon Sep 17 00:00:00 2001 From: thsaravana Date: Thu, 6 Jan 2022 16:36:57 +0530 Subject: [PATCH 13/14] Testing Jacoco Report push Event --- .github/workflows/coverage.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/coverage.yml b/.github/workflows/coverage.yml index 1c5fc7d..8ee67f2 100644 --- a/.github/workflows/coverage.yml +++ b/.github/workflows/coverage.yml @@ -2,7 +2,6 @@ name: Measure coverage on: push: - branches: [ main ] pull_request: jobs: From 95845e0dd32ea6d8e80f5218fd084e0049c4fc03 Mon Sep 17 00:00:00 2001 From: thsaravana Date: Thu, 6 Jan 2022 17:01:05 +0530 Subject: [PATCH 14/14] Testing Jacoco Report payload --- src/main/java/com/madrapps/jacoco/operation/StringOp.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/src/main/java/com/madrapps/jacoco/operation/StringOp.java b/src/main/java/com/madrapps/jacoco/operation/StringOp.java index a1e92d7..86d0730 100644 --- a/src/main/java/com/madrapps/jacoco/operation/StringOp.java +++ b/src/main/java/com/madrapps/jacoco/operation/StringOp.java @@ -13,8 +13,4 @@ public boolean endsWith(String source, String chars) { public boolean startsWith(String source, String chars) { return source.startsWith(chars); } - - public void sampleMethod() { - - } }