Skip to content

Commit a4c9967

Browse files
committed
Improve Github Actions test reporting.
1 parent 0d6bb0e commit a4c9967

2 files changed

Lines changed: 11 additions & 16 deletions

File tree

.circleci/config.yml

Whitespace-only changes.

.github/workflows/reusable-workflow.yaml

Lines changed: 11 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,7 @@ jobs:
3737
cordova telemetry off
3838
- name: React Native Dependencies
3939
if: ${{ inputs.lib == 'SalesforceReact' }}
40-
run: |
41-
npm install -g typescript
40+
run: npm install -g typescript
4241
- uses: actions/setup-java@v4
4342
with:
4443
distribution: 'zulu'
@@ -47,11 +46,12 @@ jobs:
4746
uses: android-actions/setup-android@v3
4847
- uses: gradle/actions/setup-gradle@v4
4948
with:
50-
gradle-version: 8.7
49+
gradle-version: "8.7"
50+
add-job-summary: on-failure
51+
add-job-summary-as-pr-comment: on-failure
5152
- name: Run Lint
5253
if: ${{ inputs.is_pr }}
53-
run: |
54-
./gradlew libs:${{ inputs.lib }}:lint
54+
run: ./gradlew libs:${{ inputs.lib }}:lint
5555
- uses: ruby/setup-ruby@v1
5656
if: ${{ inputs.is_pr }}
5757
with:
@@ -138,7 +138,6 @@ jobs:
138138
139139
# Move one result to the directory expected for code coverge.
140140
mv firebase_${PR_API_VERSION} firebase
141-
# Use this reporter for PRs because it correctly displays flapping/retried tests.
142141
- name: Test Report
143142
uses: mikepenz/action-junit-report@v5
144143
if: ${{ inputs.is_pr }}
@@ -150,19 +149,15 @@ jobs:
150149
flaky_summary: true
151150
fail_on_failure: true
152151
detailed_summary: true
152+
group_reports: false
153+
include_passed: true
154+
skip_success_summary: true
155+
include_empty_in_summary: false
156+
simplified_summary: true
153157
report_paths: 'firebase_results/**.xml'
154-
# Use this reporter for Nightly tests because it displays results per API level.
155-
- name: Test Report
156-
uses: dorny/test-reporter@v1
157-
if: ${{ ! inputs.is_pr }}
158-
with:
159-
name: ${{ inputs.lib }} Test Results
160-
path: firebase_results/**.xml
161-
reporter: java-junit
162158
- name: Convert Code Coverage
163159
if: success() || failure()
164-
run: |
165-
./gradlew libs:${{ inputs.lib }}:convertCodeCoverage
160+
run: ./gradlew libs:${{ inputs.lib }}:convertCodeCoverage
166161
- uses: codecov/codecov-action@v5
167162
if: success() || failure()
168163
with:

0 commit comments

Comments
 (0)