tests: Adds test for skip-empty functionality (#4)

* Adds test for skip-empty functionality

Adds a test case to verify that the `skip-empty` option correctly prevents playlist file creation when no tests match the specified outcome filters. This ensures that the action doesn't produce empty playlist files, avoiding potential issues in downstream processes.

* Adds skip-empty option for playlist generation

Allows skipping playlist generation if no matching tests are found when using the `skip-empty` option.
This prevents errors and provides informative messages instead.

* Update

* update

* update

Author: BenjaminMichaelis

.github/workflows/test.yml

@@ -370,6 +370,25 @@ jobs:
             exit 1
           fi
 
+      # Test skip-empty: should NOT create a playlist file if no tests match
+      - name: Test Action - skip-empty prevents empty playlist
+        uses: ./
+        with:
+          trx-file-path: './TestResults/comprehensive.trx'
+          output-directory: './skip-empty-test'
+          test-outcomes: 'Pending' # No test in the TRX has this outcome
+          skip-empty: true
+          artifact-name: 'test-playlist-skipempty'
+
+      - name: Verify skip-empty prevents empty playlist
+        run: |
+          if [ -f "./skip-empty-test/comprehensive.playlist" ]; then
+            echo "❌ Playlist file should NOT have been created when skip-empty is true and no tests match."
+            exit 1
+          else
+            echo "✅ No playlist file created as expected with skip-empty enabled and no matching tests."
+          fi
+
   test-error-conditions:
     runs-on: ubuntu-latest
     steps:
@@ -386,6 +405,7 @@ jobs:
 
       - name: Verify missing TRX file error handling
         run: |
+          echo "status: ${{ steps.missing-trx-test.outcome }}"
           if [ "${{ steps.missing-trx-test.outcome }}" = "failure" ]; then
             echo "✅ Action correctly failed for missing TRX file"
           else

action.yml

@@ -80,12 +80,14 @@ runs:
 
         PLAYLIST_PATHS=()
         ARTIFACT_BASENAMES=()
+        PROCESSED_TRX_COUNT=0
 
         for ACTUAL_TRX_FILE in "${TRX_FILES[@]}"; do
           if [ ! -f "$ACTUAL_TRX_FILE" ]; then
             echo "Warning: $ACTUAL_TRX_FILE is not a file, skipping."
             continue
           fi
+          PROCESSED_TRX_COUNT=$((PROCESSED_TRX_COUNT+1))
           echo "Using TRX file: $ACTUAL_TRX_FILE"
 
           TRX_BASENAME=$(basename "$ACTUAL_TRX_FILE" .trx)
@@ -111,15 +113,25 @@ runs:
           trx-to-vsplaylist "${ARGS[@]}"
 
           if [ ! -f "$FINAL_OUTPUT" ]; then
-            echo "Error: Playlist file was not created at: $FINAL_OUTPUT"
-            exit 1
+            if [ "$SKIP_EMPTY" = "false" ]; then
+              echo "Error: Playlist file was not created at: $FINAL_OUTPUT"
+              exit 1
+            else
+              echo "Info: Playlist file was not created at: $FINAL_OUTPUT (likely due to --skip-empty and no matching tests)"
+              continue
+            fi
           fi
 
           echo "Successfully created playlist file: $FINAL_OUTPUT"
           PLAYLIST_PATHS+=("$FINAL_OUTPUT")
           ARTIFACT_BASENAMES+=("$(basename "$FINAL_OUTPUT" .playlist)")
         done
 
+        if [ "$PROCESSED_TRX_COUNT" -eq 0 ]; then
+          echo "Error: No valid TRX files were found to process."
+          exit 1
+        fi
+
         # Output all playlist paths as a colon-separated list
         echo "playlist-paths=${PLAYLIST_PATHS[*]}" | tr ' ' ':' >> $GITHUB_OUTPUT
         echo "artifact-basenames=${ARTIFACT_BASENAMES[*]}" | tr ' ' ':' >> $GITHUB_OUTPUT
@@ -130,4 +142,4 @@ runs:
       with:
         name: ${{ inputs.artifact-name || 'playlists' }}
         path: ${{ steps.convert.outputs.artifact-dir }}/*.playlist
-        if-no-files-found: 'error'
+        if-no-files-found: 'warn'