Skip to content

Commit 6a4c432

Browse files
committed
update test result upload conditions and enhance summary generation logic
1 parent e448dcb commit 6a4c432

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

.github/workflows/integration-test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ jobs:
8282

8383
- name: Upload test results
8484
uses: actions/upload-artifact@v4
85-
if: always()
85+
if: failure()
8686
with:
8787
name: test-results-integration-${{ inputs.test-category }}-${{ inputs.weaviate-version }}
8888
path: ./test-results/*.trx

.github/workflows/main.yaml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ jobs:
128128
129129
- name: Upload test results
130130
uses: actions/upload-artifact@v4
131-
if: always()
131+
if: failure()
132132
with:
133133
name: test-results-unit
134134
path: ./test-results/*.trx
@@ -213,6 +213,7 @@ jobs:
213213

214214
- name: Download all test results
215215
uses: actions/download-artifact@v4
216+
continue-on-error: true
216217
with:
217218
pattern: test-results-*
218219
path: ./all-test-results
@@ -221,9 +222,12 @@ jobs:
221222
run: |
222223
if [[ "${{ needs.setup.outputs.dry-run }}" == "true" ]]; then
223224
echo "🏃 DRY-RUN MODE: Skipping test summary generation"
224-
else
225+
elif [ -d "./all-test-results" ] && [ -n "$(find ./all-test-results -name "*.trx" 2>/dev/null)" ]; then
226+
echo "📊 Generating test summary from TRX files..."
225227
dotnet tool update -g dotnet-trx
226228
find ./all-test-results -name "*.trx" -exec trx {} \;
229+
else
230+
echo "✅ No test failures - all tests passed!"
227231
fi
228232
229233
- name: Check test status

0 commit comments

Comments
 (0)