test: fix skipping: test-mic-privacy

Fix skip if no module named: usbrelay.
Add skip if no relays hardware are detected.
Add the new parameter to test:
    dsp-settle-sleep by default = 2s

Signed-off-by: Artur Wilczak <arturx.wilczak@intel.com>

Author: arikgreen

test-case/test-mic-privacy.sh

@@ -41,8 +41,6 @@ source "${TESTLIB}/relay.sh"
 ALSABAT_WAV_FILES="/tmp/mc.wav.*"
 rm -f "$ALSABAT_WAV_FILES"
 
-DSP_SETTLE_TIME=2
-
 OPT_NAME['p']='pcm_p'     	OPT_DESC['p']='pcm for playback. Example: hw:0,0'
 OPT_HAS_ARG['p']=1          OPT_VAL['p']='hw:0,0'
 
@@ -58,6 +56,9 @@ OPT_HAS_ARG['C']=1          OPT_VAL['C']='2'
 OPT_NAME['s']='sof-logger'  OPT_DESC['s']="Open sof-logger trace the data will store at $LOG_ROOT"
 OPT_HAS_ARG['s']=0          OPT_VAL['s']=1
 
+OPT_NAME['d']='dsp-settle-sleep'    OPT_DESC['d']="Waiting time to change control state"
+OPT_HAS_ARG['d']=1                  OPT_VAL['d']=2
+
 OPT_NAME['r']='rate'        OPT_DESC['r']='sample rate'
 OPT_HAS_ARG['r']=1          OPT_VAL['r']=48000
 
@@ -72,6 +73,7 @@ channel_c=${OPT_VAL['C']}
 channel_p=${OPT_VAL['N']}
 rate=${OPT_VAL['r']}
 relay=${OPT_VAL['u']}
+dsp_settle_time=${OPT_VAL['d']}
 
 dlogi "Params: pcm_p=$pcm_p, pcm_c=$pcm_c, channel_c=$channel_c, channel_p=$channel_p, rate=$rate, LOG_ROOT=$LOG_ROOT"
 
@@ -150,25 +152,26 @@ main()
 
     start_test
 
-    logger_disabled || func_lib_start_log_collect
+    dlogi "Checking usbrelay availability..."
+    command -v usbrelay || {
+        # If usbrelay package is not installed
+        skip_test "usbrelay command not found."
+    }
+
+    # display current status of relays
+    usbrelay_switch --debug || {
+        skip_test "Failed to get usbrelay status."
+    }
 
     if [ -z "$pcm_p" ] || [ -z "$pcm_c" ]; then
         skip_test "No playback or capture PCM is specified. Skip the $0 test."
     fi
-
-    # check if usbrelay tool is installed
-    command -v usbrelay || {
-        skip_test "usbrelay command not found. Please install usbrelay to control the mic privacy switch."
-    }
-
+    
     dlogi "Current DSP status is $(sof-dump-status.py --dsp_status 0)" || {
         skip_test "platform doesn't support runtime pm, skip test case"
     }
 
-    dlogi "Starting preconditions check"
-
-    # display current status of relays
-    usbrelay "--debug"
+    logger_disabled || func_lib_start_log_collect
 
     check_locale_for_alsabat
 
@@ -180,7 +183,7 @@ main()
     show_control_state
 
     # wait for the switch to settle
-    sleep "$DSP_SETTLE_TIME"
+    sleep "$dsp_settle_time"
 
     # check the PCMs before mic privacy test
     dlogi "Check playback/capture before mic privacy test"
@@ -196,14 +199,14 @@ main()
 
     dlogi "Preconditions are met, starting mic privacy test"
 
-    sleep "$DSP_SETTLE_TIME"
+    sleep "$dsp_settle_time"
 
     dlogi "===== Testing: MIC privacy ====="
     dlogi "Turn on the mic privacy switch"
     usbrelay_switch "$relay" 1
 
     # wait for the switch to settle
-    sleep "$DSP_SETTLE_TIME"
+    sleep "$dsp_settle_time"
 
     alsabat_output=$(mktemp)
     dlogc "alsabat -P$pcm_p -C$pcm_c -c 2 -r $rate"