fix: only show test summaries on failure to reduce noise

[maxandersen]

Summary

• Changed JUnit report and Allure links steps from if: always() to if: failure() in CI workflow
• Affects both unit-test-jvm and integration-test-jvm jobs
• Eliminates empty test summary noise when tests pass

Problem

GitHub Actions workflow was generating test report summaries for every job, even when tests passed with no failures. This created noise like:

Tests	Passed ❌️	Skipped	Failed
JUnit Test Report	0 ran	0 passed	0 skipped	0 failed
View Allure Report direct link (available after first view) for macos-latest build

Solution

Modified the conditional execution of test summary steps to only run on failure:

• JUnit report annotation step: if: always() → if: failure()
• Artifact.ci links step: if: always() → if: failure()

Impact

• ✅ Cleaner job summaries when tests pass
• ✅ Debugging info still appears when tests fail
• ✅ Consistent with recent change to Gradle's add-job-summary: 'on-failure'
• ✅ No data loss - all artifacts still uploaded and accessible

Test plan

• Verify no empty summaries appear on passing test runs
• Verify JUnit reports and Allure links appear when tests fail
• Check that comprehensive test-results-summary job still provides central reference

🤖 Generated with Claude Code