Skip to content

Commit 5e70414

Browse files
committed
fix: only post summary on failure. have .ci and direct links in summary
1 parent f1346ce commit 5e70414

File tree

2 files changed

+21
-19
lines changed

2 files changed

+21
-19
lines changed

.github/actions/shared-build-setup/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ inputs:
55
add-job-summary:
66
description: Add the job summary to the output
77
required: false
8-
default: 'always'
8+
default: 'on-failure'
99
java-version:
1010
description: The Java version to use
1111
required: false

.github/workflows/step-ci-build.yml

Lines changed: 20 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ jobs:
292292

293293
test-results-summary:
294294
if: always() && !inputs.skip_tests
295-
needs: [unit-test-jvm, integration-test-jvm, build-test-native-image, merge-test-reports]
295+
needs: [merge-test-reports]
296296
runs-on: ubuntu-latest
297297
steps:
298298
- uses: actions/checkout@93cb6efe18208431cddfb8368fd83d5badbf9bfd # v5
@@ -301,44 +301,46 @@ jobs:
301301
- name: Create test results summary
302302
run: |
303303
SHA_PREFIX="${{ steps.shared-build.outputs.github-short-sha }}"
304+
REPO="${{ github.repository }}"
305+
RUN="${{ github.run_id }}.${{ github.run_attempt}}"
304306
cat >> $GITHUB_STEP_SUMMARY <<EOF
305307
# 🧪 Test Results Summary
306308
307309
## 📊 Combined Reports (All Tests Merged)
308310
309-
- **[View Merged Allure Report](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-merged-allurereport-allure-report/build/reports/allure-report/allureReport/index.html)** - All unit and integration tests across all platforms
311+
- **Merged Allure Report**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-merged-allurereport-allure-report) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-merged-allurereport-allure-report/index.html) - All unit and integration tests across all platforms
310312
311313
## 🖥️ Unit Test Reports (by Platform)
312314
313-
- [Ubuntu Latest](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-unit-test-jvm-allure-report-ubuntu-latest/build/reports/allure-report/allureReport/index.html)
314-
- [macOS Latest](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-unit-test-jvm-allure-report-macos-latest/build/reports/allure-report/allureReport/index.html)
315-
- [Windows Latest](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-unit-test-jvm-allure-report-windows-latest/build/reports/allure-report/allureReport/index.html)
315+
- **Ubuntu Latest**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-unit-test-jvm-allure-report-ubuntu-latest) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-unit-test-jvm-allure-report-ubuntu-latest/index.html)
316+
- **macOS Latest**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-unit-test-jvm-allure-report-macos-latest) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-unit-test-jvm-allure-report-macos-latest/index.html)
317+
- **Windows Latest**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-unit-test-jvm-allure-report-windows-latest) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-unit-test-jvm-allure-report-windows-latest/index.html)
316318
317319
## 🔧 Integration Test Reports (by Platform)
318320
319321
### Ubuntu
320-
- [Java 8](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-8/build/reports/allure-report/allureReport/index.html)
321-
- [Java 11](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-11/build/reports/allure-report/allureReport/index.html)
322-
- [Java 17](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-17/build/reports/allure-report/allureReport/index.html)
323-
- [Java 21](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-21/build/reports/allure-report/allureReport/index.html)
324-
- [Java 25](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-25/build/reports/allure-report/allureReport/index.html)
322+
- **Java 8**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-8) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-8/index.html)
323+
- **Java 11**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-11) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-11/index.html)
324+
- **Java 17**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-17) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-17/index.html)
325+
- **Java 21**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-21) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-21/index.html)
326+
- **Java 25**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-25) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-ubuntu-latest-25/index.html)
325327
326328
### macOS
327-
- [Java 11](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-macos-latest-11/build/reports/allure-report/allureReport/index.html)
328-
- [Java 25](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-macos-latest-25/build/reports/allure-report/allureReport/index.html)
329+
- **Java 11**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-macos-latest-11) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-macos-latest-11/index.html)
330+
- **Java 25**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-macos-latest-25) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-macos-latest-25/index.html)
329331
330332
### Windows
331-
- [Java 11](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-windows-latest-11/build/reports/allure-report/allureReport/index.html)
332-
- [Java 25](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-integration-test-jvm-allure-report-windows-latest-25/build/reports/allure-report/allureReport/index.html)
333+
- **Java 11**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-windows-latest-11) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-windows-latest-11/index.html)
334+
- **Java 25**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-windows-latest-25) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-integration-test-jvm-allure-report-windows-latest-25/index.html)
333335
334336
## 🚀 Native Image Test Reports
335337
336-
- [Ubuntu Latest](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-test-native-image-allure-report-ubuntu-latest/build/reports/allure-report/allureReport/index.html)
337-
- [macOS Latest](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-test-native-image-allure-report-macos-latest/build/reports/allure-report/allureReport/index.html)
338-
- [Windows Latest](https://www.artifact.ci/artifact/view/${{ github.repository }}/run/${{ github.run_id }}.${{ github.run_attempt}}/${SHA_PREFIX}-test-native-image-allure-report-windows-latest/build/reports/allure-report/allureReport/index.html)
338+
- **Ubuntu Latest**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-test-native-image-allure-report-ubuntu-latest) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-test-native-image-allure-report-ubuntu-latest/index.html)
339+
- **macOS Latest**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-test-native-image-allure-report-macos-latest) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-test-native-image-allure-report-macos-latest/index.html)
340+
- **Windows Latest**: [artifact.ci](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-test-native-image-allure-report-windows-latest) | [direct](https://www.artifact.ci/artifact/view/\${REPO}/run/\${RUN}/${SHA_PREFIX}-test-native-image-allure-report-windows-latest/index.html)
339341
340342
---
341-
*Note: artifact.ci links require viewing once before the direct link becomes available*
343+
*Note: Visit the artifact.ci link first, then the direct link will work*
342344
EOF
343345
344346

0 commit comments

Comments
 (0)