From ffc7c0607096458f9fdde387b6454bf03db0a1a2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Robert=20Ga=C5=82at?= <robert.galat@nordicsemi.no>
Date: Mon, 25 Mar 2024 12:40:39 +0000
Subject: [PATCH] CI: remove coverage raport from GHA
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

twister changed the way coverage is generated

Signed-off-by: Robert Gałat <robert.galat@nordicsemi.no>
---
 .github/workflows/common-build_and_run.yml | 20 -----------
 .github/workflows/on-commit.yml            |  4 ---
 .github/workflows/on-pr.yml                |  4 ---
 .github/workflows/publish_test_results.yml | 42 ----------------------
 4 files changed, 70 deletions(-)
 delete mode 100644 .github/workflows/publish_test_results.yml

diff --git a/.github/workflows/common-build_and_run.yml b/.github/workflows/common-build_and_run.yml
index c1ee0321f5..c1bf7e4169 100644
--- a/.github/workflows/common-build_and_run.yml
+++ b/.github/workflows/common-build_and_run.yml
@@ -237,10 +237,6 @@ jobs:
           source /workdir/zephyr/zephyr-env.sh
           west twister --platform native_posix --platform unit_testing --coverage --enable-ubsan --enable-lsan --enable-asan --testsuite-root /workdir/sidewalk/tests/ --inline-logs --overflow-as-errors -vvv
 
-      - name: Fix coverage report
-        run: |
-          lcov -q --remove "twister-out/coverage.info" '/**/twister-out/*' '/**/mbedtls/*' '/**/test/cmock/*' '/**/zephyr/*' -o "coverage.info"
-
       - name: Upload test results
         uses: actions/upload-artifact@v3
         if: always()
@@ -250,19 +246,3 @@ jobs:
             twister-out/twister.xml
             twister-out/**/handler.log
             twister-out/**/device.log
-
-      - name: Upload Coverage report
-        run: |
-          echo "# Coverage" >> summary
-          echo "" >> summary
-          echo "## Summary" >> summary
-          echo "" >> summary
-          echo "\`\`\`" >> summary
-          lcov --summary coverage.info --rc lcov_branch_coverage=1 >> summary
-          echo "\`\`\`" >> summary
-          echo "## File coverage" >> summary
-          echp "">> summary
-          echo "\`\`\`" >> summary
-          lcov --list coverage.info --rc lcov_branch_coverage=1 >> summary
-          echo "\`\`\`" >> summary
-          cat summary >> $GITHUB_STEP_SUMMARY
diff --git a/.github/workflows/on-commit.yml b/.github/workflows/on-commit.yml
index 8924c7a71b..99b8d85933 100644
--- a/.github/workflows/on-commit.yml
+++ b/.github/workflows/on-commit.yml
@@ -71,7 +71,3 @@ jobs:
 
   build_samples_and_tests:
     uses: ./.github/workflows/common-build_and_run.yml
-
-  publish_test_results:
-    needs: [build_samples_and_tests]
-    uses: ./.github/workflows/publish_test_results.yml
diff --git a/.github/workflows/on-pr.yml b/.github/workflows/on-pr.yml
index e8af4e0f86..0ab2f27389 100644
--- a/.github/workflows/on-pr.yml
+++ b/.github/workflows/on-pr.yml
@@ -70,7 +70,3 @@ jobs:
 
   build_samples_and_tests:
     uses: ./.github/workflows/common-build_and_run.yml
-
-  publish_test_results:
-    needs: [build_samples_and_tests]
-    uses: ./.github/workflows/publish_test_results.yml
diff --git a/.github/workflows/publish_test_results.yml b/.github/workflows/publish_test_results.yml
deleted file mode 100644
index c944ffed0b..0000000000
--- a/.github/workflows/publish_test_results.yml
+++ /dev/null
@@ -1,42 +0,0 @@
-name: Publish test results
-on:
-  workflow_call:
-
-jobs:
-  test-results:
-    name: "Publish Unit Tests Results"
-    runs-on: ubuntu-22.04
-    steps:
-      - name: Download dut test artifacts
-        uses: actions/download-artifact@v3
-        with:
-          path: artifacts/dut
-          name: tests-dut_result
-      - name: Download x86 test artifacts
-        uses: actions/download-artifact@v3
-        with:
-          path: artifacts/x86
-          name: tests-x86_result
-
-      - name: Merge Test Results
-        run: |
-          pip3 install junitparser junit2html
-          junitparser merge artifacts/*/twister.xml junit.xml
-          junit2html junit.xml junit.html
-
-      - name: Upload Unit Test Results in HTML
-        if: always()
-        uses: actions/upload-artifact@v3
-        with:
-          name: HTML Unit Test Results
-          if-no-files-found: ignore
-          path: |
-            junit.html
-
-      - name: Publish Unit Test Results
-        uses: EnricoMi/publish-unit-test-result-action@v2
-        if: always()
-        with:
-          check_name: Unit Test Results
-          files: "**/*twister.xml"
-          comment_mode: off