Skip to content

Commit

Permalink
fix: Coverage report is not displayed
Browse files Browse the repository at this point in the history
Change-Id: I8ca7da707fda91a10ee265b644f67e5150db1f45
  • Loading branch information
blue928sky committed Jan 18, 2025
1 parent a0f2786 commit b2e624e
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions .github/workflows/Build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -201,14 +201,14 @@ jobs:
heap-size: 600M
script: ./gradlew connectedDemoDebugAndroidTest --daemon

- name: Run local tests (including Roborazzi) for the combined coverage report (only API 30)
if: matrix.api-level == 30
- name: Run local tests (including Roborazzi) for the combined coverage report (only API 34)
if: matrix.api-level == 34
# There is no need to verify Roborazzi tests to generate coverage.
run: ./gradlew testDemoDebugUnitTest -Proborazzi.test.verify=false # Add Prod if we ever add JVM tests for prod

# Add `createProdDebugUnitTestCoverageReport` if we ever add JVM tests for prod
- name: Generate coverage reports for Debug variants (only API 30)
if: matrix.api-level == 30
- name: Generate coverage reports for Debug variants (only API 34)
if: matrix.api-level == 34
run: ./gradlew createDemoDebugCombinedCoverageReport

- name: Upload test reports
Expand All @@ -218,8 +218,8 @@ jobs:
name: test-reports-${{ matrix.api-level }}
path: '**/build/reports/androidTests'

- name: Display local test coverage (only API 30)
if: matrix.api-level == 30
- name: Display local test coverage (only API 34)
if: matrix.api-level == 34
id: jacoco
uses: madrapps/[email protected]
with:
Expand All @@ -230,8 +230,8 @@ jobs:
${{ github.workspace }}/**/build/reports/jacoco/**/*Report.xml
token: ${{ secrets.GITHUB_TOKEN }}

- name: Upload local coverage reports (XML + HTML) (only API 30)
if: matrix.api-level == 30
- name: Upload local coverage reports (XML + HTML) (only API 34)
if: matrix.api-level == 34
uses: actions/upload-artifact@v4
with:
name: coverage-reports
Expand Down

0 comments on commit b2e624e

Please sign in to comment.