Added shellcheck disable=SC2317 for the cleanup function in all gst mm runner scripts, added help section and updated readme files for audio

Signed-off-by: Nitin Nakka <nitinn@qti.qualcomm.com>

Author: nitinn22

Runner/suites/Multimedia/GSTreamer/Audio/Audio_Record_Playback/README.md

@@ -58,7 +58,7 @@ At a high level, the test:
 
 ## Test Cases
 
-By default, the test runs **8 test cases** with 10 second duration:
+By default, the test runs **10 test cases** with 10 second duration:
 
 ### ENCODE PHASE (4 tests)
 
@@ -70,7 +70,7 @@ By default, the test runs **8 test cases** with 10 second duration:
 3. **record_pulsesrc_wav** - Record WAV format using pulsesrc → audioconvert → wavenc
 4. **record_pulsesrc_flac** - Record FLAC format using pulsesrc → audioconvert → flacenc
 
-### DECODE PHASE (4 tests)
+### DECODE PHASE (6 tests)
 
 #### Playback Tests - audiotestsrc recordings
 5. **playback_wav** - Playback WAV file using filesrc → wavparse → audioconvert → pulsesink
@@ -80,20 +80,28 @@ By default, the test runs **8 test cases** with 10 second duration:
 7. **playback_pulsesrc_wav** - Playback WAV file using filesrc → wavparse → audioconvert → pulsesink
 8. **playback_pulsesrc_flac** - Playback FLAC file using filesrc → flacparse → flacdec → pulsesink
 
-**Total: 8 test cases** (4 encode + 4 decode)
+#### Playback Tests - External Test Files (OGG/MP3)
+9. **playback_sample_ogg** - Playback OGG file using filesrc → oggdemux → vorbisdec → pulsesink
+10. **playback_sample_mp3** - Playback MP3 file using filesrc → mpegaudioparse → mpg123audiodec → pulsesink
+
+**Total: 10 test cases** (4 encode + 6 decode)
+
+**Note:** OGG/MP3 playback tests require external test files (downloaded from URL or provided via `--clip-path`). If test files are not available, these tests will be skipped.
 
 ---
 
 ## PASS / FAIL / SKIP criteria
 
 ### PASS
 - **Recording**: Output file is created and has size > 1000 bytes
-- **Playback**: Pipeline completes successfully (exit code 0, 124, or 143)
+- **Playback**: Pipeline completes successfully with exit code 0 (clean completion)
+  - **Note**: Timeout (exit code 124) or termination (exit code 143) are treated as **FAIL** for controlled-duration recordings
+  - This ensures playback completed successfully rather than being interrupted
 - **Overall**: At least one test passes and no tests fail
 
 ### FAIL
 - **Recording**: No output file created or file size too small
-- **Playback**: Pipeline fails or GStreamer errors detected
+- **Playback**: Pipeline exits with non-zero code (including timeout 124 or termination 143) or GStreamer errors detected
 - **Overall**: One or more tests fail
 
 ### SKIP

Runner/suites/Multimedia/GSTreamer/Audio/Audio_Record_Playback/run.sh

@@ -127,6 +127,7 @@ for param in AUDIO_DURATION AUDIO_GST_DEBUG GST_DEBUG_LEVEL; do
   fi
 done
 
+# shellcheck disable=SC2317
 cleanup() {
   # Best-effort: try to kill only children first; fall back to name-based kill
   if ! pkill -P "$$" -x gst-launch-1.0 >/dev/null 2>&1; then
@@ -718,7 +719,7 @@ run_playback_ogg_mp3_test() {
   fi
 
   # Check for successful completion
-  if [ "$gstRc" -eq 0 ] || [ "$gstRc" -eq 124 ] || [ "$gstRc" -eq 143 ]; then
+  if [ "$gstRc" -eq 0 ]; then
     log_pass "$testname: PASS"
     pass_count=$((pass_count + 1))
     return 0
@@ -769,9 +770,10 @@ provision_test_files() {
         log_warn "Test file download failed (offline or URL issue)"
       fi
     fi
+    
   fi
 
-  # Check what we have
+  # Check what we have AFTER all provisioning attempts
   have_ogg=0
   have_mp3=0
 
@@ -791,6 +793,7 @@ provision_test_files() {
     fi
   fi
 
+  # Only warn if BOTH files are missing after all attempts
   if [ "$have_ogg" -eq 0 ] && [ "$have_mp3" -eq 0 ]; then
     log_warn "No Test files (OGG/MP3) available for playback tests"
   fi

Runner/suites/Multimedia/GSTreamer/Display/Waylandsink_Playback/run.sh

@@ -92,6 +92,7 @@ height="${VIDEO_HEIGHT:-1080}"
 framerate="${VIDEO_FRAMERATE:-30}"
 gstDebugLevel="${VIDEO_GST_DEBUG:-${GST_DEBUG_LEVEL:-2}}"
 
+# shellcheck disable=SC2317
 cleanup() {
   # Best-effort: try to kill only children first; fall back to name-based kill
   if ! pkill -P "$$" -x gst-launch-1.0 >/dev/null 2>&1; then

Runner/suites/Multimedia/GSTreamer/Video/Video_Encode_Decode/run.sh

@@ -103,6 +103,7 @@ for param in VIDEO_DURATION RUNTIMESEC VIDEO_FRAMERATE VIDEO_GST_DEBUG GST_DEBUG
   fi
 done
 
+# shellcheck disable=SC2317
 cleanup() {
   # Best-effort: try to kill only children first; fall back to name-based kill
   if ! pkill -P "$$" -x gst-launch-1.0 >/dev/null 2>&1; then