update: don't fail on skipped suites

- Only upload artifacts on test failure

Author: namsnath

action.yml

@@ -117,8 +117,7 @@ runs:
         VALID_CHECK_NAME=$(echo '${{ inputs.check-name-suffix }}' | sed "s|/|__|g")
         echo "VALID_CHECK_NAME=$VALID_CHECK_NAME" | tee -a "$GITHUB_ENV"
 
-    # TODO: Figure out a generic way to get the app server logs here too
-    - if: always()
+    - if: failure()
       name: Upload artifacts
       uses: actions/upload-artifact@v4
       with:
@@ -136,7 +135,7 @@ runs:
         detailed_summary: true
         # Group the testcases by test suite in the detailed_summary
         group_suite: true
-        # Fail if no test are found.
-        require_tests: true
+        # Don't fail if no test are found.
+        require_tests: false
         # Fail the build in case of a test failure.
         fail_on_failure: true