Skip to content

Commit b77262a

Browse files
ekurdybxekurdybx
committed
fix: tests: SRC tests catch aplay warnings and refactor
Catch aplay incorrect sample rate warnings and refactor tests Signed-off-by: Emilia Kurdybelska <emiliax.kurdybelska@intel.com>
1 parent 4bc7c24 commit b77262a

6 files changed

Lines changed: 143 additions & 57 deletions

File tree

‎case-lib/lib.sh‎

Lines changed: 56 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1534,47 +1534,85 @@ restore_topology() {
15341534
check_topology
15351535
}
15361536

1537+
# Check aplay output for warnings about incorrect sample rate
1538+
# Arguments: 1-aplay output
1539+
check_for_aplay_warnings()
1540+
{
1541+
dlogi "$1"
1542+
if echo "$1" | grep -q "Warning:"; then
1543+
dlogw "Caught aplay warning!"
1544+
return 1
1545+
fi
1546+
return 0
1547+
}
1548+
15371549
# Play sound and record it
15381550
# Arguments: 1-arecord options 2-aplay options
15391551
play_and_record()
15401552
{
1553+
errors=0
15411554
dlogi "Play [aplay $SOF_ALSA_OPTS $SOF_APLAY_OPTS $2] and capture sound [arecord $1]"
15421555
# shellcheck disable=SC2086
15431556
arecord $SOF_ALSA_OPTS $SOF_ARECORD_OPTS $1 & PID=$!
1557+
if [ $? -ne 0 ]; then
1558+
errors=$((errors+1))
1559+
fi
15441560
# shellcheck disable=SC2086
1545-
aplay $SOF_ALSA_OPTS $SOF_APLAY_OPTS $2
1561+
aplay_output=$(aplay $SOF_ALSA_OPTS $SOF_APLAY_OPTS $2 2>&1)
1562+
if [ $? -ne 0 ]; then
1563+
errors=$((errors+1))
1564+
fi
15461565
wait $PID
15471566
sleep 1
1567+
check_for_aplay_warnings "$aplay_output"
1568+
if [ $? -ne 0 ]; then
1569+
errors=$((errors+1))
1570+
fi
1571+
1572+
if [ "$errors" = 0 ]; then
1573+
return 0
1574+
fi
1575+
return 1
15481576
}
15491577

1550-
# Analyze files to look for glitches.
1578+
# Analyze file to look for glitches.
15511579
# Returns exit code 0 if there are no glitches, 1 if there are.
1552-
# Arguments: the list of filenames
1580+
# Arguments: filename
15531581
check_soundfile_for_glitches()
15541582
{
1555-
glitched_files=0
1583+
if [ -f "$result_filename" ]; then
1584+
dlogi "Analyzing $result_filename file..."
1585+
if python3 "$SCRIPT_HOME"/tools/analyze-wav.py "$result_filename"; then
1586+
dlogi "$result_filename file is correct"
1587+
else
1588+
dlogw "GLITCHED FILE: $result_filename"
1589+
return 1
1590+
fi
1591+
else
1592+
dlogw "MISSING FILE: $result_filename"
1593+
return 1
1594+
fi
1595+
}
1596+
1597+
# Check list of soundfiles for glitches
1598+
# Returns exit code 0 if there are no glitches, 1 if there are.
1599+
# Arguments: list of filenames
1600+
check_soundfiles_for_glitches()
1601+
{
1602+
failures=0
15561603
# shellcheck disable=SC2154
15571604
for result_filename in "${all_result_files[@]}"
15581605
do
1559-
if [ -f "$result_filename" ]; then
1560-
dlogi "Analyzing $result_filename file..."
1561-
if python3 "$SCRIPT_HOME"/tools/analyze-wav.py "$result_filename"; then
1562-
dlogi "$result_filename file is correct"
1563-
else
1564-
dlogw "Found issues in $result_filename file"
1565-
glitched_files=$((glitched_files+1))
1566-
fi
1567-
else
1568-
dlogw "$result_filename file not found, check for previous errors"
1569-
glitched_files=$((glitched_files+1))
1606+
if ! check_soundfile_for_glitches "$result_filename"; then
1607+
failures=$((failures+1))
15701608
fi
15711609
done
15721610

1573-
if [ $glitched_files -eq 0 ]; then
1574-
dlogi "Analysis finished, no issues found"
1611+
if [ "$failures" = 0 ]; then
1612+
dlogi "All files correct"
15751613
return 0
15761614
else
1577-
dlogi "$glitched_files files corrupted"
1615+
dlogw "Found $failures corrupted files"
15781616
return 1
15791617
fi
15801618
}

‎test-case/check-8bit-play-rec.sh‎

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@ source "$(dirname "${BASH_SOURCE[0]}")"/../case-lib/lib.sh
3131
OPT_NAME['t']='tplg' OPT_DESC['t']='tplg file, default value is env TPLG: $''TPLG'
3232
OPT_HAS_ARG['t']=1 OPT_VAL['t']="$TPLG"
3333

34-
OPT_NAME['p']='playback_device' OPT_DESC['p']='ALSA pcm playback device. Example: hw:0,1'
35-
OPT_HAS_ARG['p']=1 OPT_VAL['p']=''
34+
OPT_NAME['p']='playback_device' OPT_DESC['p']='ALSA pcm playback device. Default: hw:0,0'
35+
OPT_HAS_ARG['p']=1 OPT_VAL['p']='hw:0,0'
3636

37-
OPT_NAME['c']='capture_device' OPT_DESC['c']='ALSA pcm capture device. Example: hw:0,1'
38-
OPT_HAS_ARG['c']=1 OPT_VAL['c']=''
37+
OPT_NAME['c']='capture_device' OPT_DESC['c']='ALSA pcm capture device. Default: hw:0,0'
38+
OPT_HAS_ARG['c']=1 OPT_VAL['c']='hw:0,0'
3939

4040
OPT_NAME['s']='sof-logger' OPT_DESC['s']="Open sof-logger trace the data will store at $LOG_ROOT"
4141
OPT_HAS_ARG['s']=0 OPT_VAL['s']=1
@@ -87,7 +87,6 @@ run_tests()
8787
{
8888
generate_chirps
8989

90-
set +e
9190
play_and_record "-D$capture_dev $rec_opt $u8_play_filename" "-D$playback_dev $play_opt -t wav $chirp_u8_filename"
9291
play_and_record "-D$capture_dev $rec_opt $alaw_play_filename" "-D$playback_dev $play_opt -t raw -f A_LAW $chirp_alaw_filename"
9392
play_and_record "-D$capture_dev $rec_opt $mulaw_play_filename" "-D$playback_dev $play_opt -t raw -f MU_LAW $chirp_mulaw_filename"
@@ -98,9 +97,8 @@ run_tests()
9897

9998
sox --encoding a-law -r 48000 -c 2 tmp1.raw "$alaw_rec_filename"
10099
sox --encoding u-law -r 48000 -c 2 tmp2.raw "$mulaw_rec_filename"
101-
set -e
102100

103-
if check_soundfile_for_glitches "${all_result_files[@]}"; then
101+
if check_soundfiles_for_glitches "${all_result_files[@]}"; then
104102
dlogi "All files correct"
105103
else
106104
die "Detected corrupted files!"

‎test-case/check-float-play-rec.sh‎

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@ source "$(dirname "${BASH_SOURCE[0]}")"/../case-lib/lib.sh
3131
OPT_NAME['t']='tplg' OPT_DESC['t']='tplg file, default value is env TPLG: $''TPLG'
3232
OPT_HAS_ARG['t']=1 OPT_VAL['t']="$TPLG"
3333

34-
OPT_NAME['p']='playback_device' OPT_DESC['p']='ALSA pcm playback device. Example: hw:0,1'
35-
OPT_HAS_ARG['p']=1 OPT_VAL['p']=''
34+
OPT_NAME['p']='playback_device' OPT_DESC['p']='ALSA pcm playback device. Default: hw:0,0'
35+
OPT_HAS_ARG['p']=1 OPT_VAL['p']='hw:0,0'
3636

37-
OPT_NAME['c']='capture_device' OPT_DESC['c']='ALSA pcm capture device. Example: hw:0,1'
38-
OPT_HAS_ARG['c']=1 OPT_VAL['c']=''
37+
OPT_NAME['c']='capture_device' OPT_DESC['c']='ALSA pcm capture device. Default: hw:0,0'
38+
OPT_HAS_ARG['c']=1 OPT_VAL['c']='hw:0,0'
3939

4040
OPT_NAME['s']='sof-logger' OPT_DESC['s']="Open sof-logger trace the data will store at $LOG_ROOT"
4141
OPT_HAS_ARG['s']=0 OPT_VAL['s']=1
@@ -70,12 +70,10 @@ run_tests()
7070
{
7171
generate_chirps
7272

73-
set +e
7473
play_and_record "-D$capture_dev $rec_opt -f S32_LE $rec_play_filename" "-D$playback_dev $chirp_float_filename"
7574
play_and_record "-D$capture_dev $rec_opt -f FLOAT_LE $rec_filename" "-D$playback_dev $chirp_s32_filename"
76-
set -e
7775

78-
if check_soundfile_for_glitches "${all_result_files[@]}"; then
76+
if check_soundfiles_for_glitches "${all_result_files[@]}"; then
7977
dlogi "All files correct"
8078
else
8179
die "Detected corrupted files!"

‎test-case/check-selector-play.sh‎

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,15 +81,27 @@ prepare_test_soundfiles()
8181

8282
run_tests()
8383
{
84+
failures=0
8485
set +e
8586
for ch_nr in "${channels_to_test[@]}"
8687
do
88+
test_pass=true
8789
test_filename="$HOME/Music/${ch_nr}_channels_test.wav"
8890
result_filename="$LOG_ROOT/rec_${ch_nr}ch.wav"
91+
dlogi "--------------- PLAY $ch_nr CHANNELS, RECORD ON 2 channels ---------------"
8992

9093
play_and_record "-D$capture_dev $rec_opt -d 25 $result_filename" "-Dplug$playback_dev $test_filename"
91-
94+
if [ $? -eq 1 ]; then
95+
test_pass=false
96+
dlogi "TEST $ch_nr channels FAIL: aplay/arecord failed, look for previous errors"
97+
fi
9298
if ! analyze_mixed_sound "$result_filename" "$ch_nr"; then
99+
test_pass=false
100+
dlogi "TEST $ch_nr channels FAIL: Recording has incorrect nr of channels"
101+
fi
102+
if [ "$test_pass" = true ]; then
103+
dlogi "TEST $ch_nr channels PASS: No issues found."
104+
else
93105
failures=$((failures+1))
94106
fi
95107
done

‎test-case/check-src-play.sh‎

Lines changed: 36 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@ source "$(dirname "${BASH_SOURCE[0]}")"/../case-lib/lib.sh
3131
OPT_NAME['t']='tplg' OPT_DESC['t']='tplg file, default value is env TPLG: $''TPLG'
3232
OPT_HAS_ARG['t']=1 OPT_VAL['t']="$TPLG"
3333

34-
OPT_NAME['p']='playback_device' OPT_DESC['p']='ALSA pcm playback device. Example: hw:0,1'
35-
OPT_HAS_ARG['p']=1 OPT_VAL['p']=''
34+
OPT_NAME['p']='playback_device' OPT_DESC['p']='ALSA pcm playback device. Default: hw:0,2'
35+
OPT_HAS_ARG['p']=1 OPT_VAL['p']='hw:0,2'
3636

37-
OPT_NAME['c']='capture_device' OPT_DESC['c']='ALSA pcm capture device. Example: hw:0,1'
38-
OPT_HAS_ARG['c']=1 OPT_VAL['c']=''
37+
OPT_NAME['c']='capture_device' OPT_DESC['c']='ALSA pcm capture device. Default: hw:0,2'
38+
OPT_HAS_ARG['c']=1 OPT_VAL['c']='hw:0,2'
3939

4040
OPT_NAME['s']='sof-logger' OPT_DESC['s']="Open sof-logger trace the data will store at $LOG_ROOT"
4141
OPT_HAS_ARG['s']=0 OPT_VAL['s']=1
@@ -55,29 +55,51 @@ init_globals()
5555
all_result_files=()
5656
}
5757

58+
prepare_test_soundfile()
59+
{
60+
dlogi "Generating test audio: sample rate: $sample_rate Hz, chirp rate: $chirp_rate Hz..."
61+
mkdir -p "$HOME/Music"
62+
test_sound_filename="$HOME/Music/${sample_rate}_Hz.wav"
63+
ffmpeg -loglevel error -y -f lavfi -i "aevalsrc='sin($chirp_rate*t*2*PI*t)':s=$sample_rate:d=5" -ac 2 "$test_sound_filename"
64+
}
65+
5866
run_tests()
5967
{
68+
failures=0
6069
set +e
6170
for i in "${!sample_rates[@]}"
6271
do
72+
test_pass=true
6373
sample_rate=${sample_rates[$i]}
6474
chirp_rate=${chirp_rates[$i]}
75+
dlogi "--------------- TEST $((i+1)): PLAY SAMPLE RATE $sample_rate Hz, RECORD IN 48000 Hz ---------------"
6576

66-
test_sound_filename=$LOG_ROOT/play.wav
6777
result_filename=$LOG_ROOT/rec_play_$sample_rate.wav
68-
all_result_files+=("$result_filename")
69-
70-
dlogi "Play $sample_rate Hz chirp 0 - $chirp_rate Hz, capture as 48 kHz"
71-
ffmpeg -y -f lavfi -i "aevalsrc='sin($chirp_rate*t*2*PI*t)':s=$sample_rate:d=5" -ac 2 "$test_sound_filename" #TODO: maybe separate dir for artifacts ??
72-
78+
prepare_test_soundfile
7379
play_and_record "-D$capture_dev $rec_opt $result_filename" "-D$playback_dev $test_sound_filename"
80+
if [ $? -eq 1 ]; then
81+
test_pass=false
82+
dlogi "TEST $((i+1)) FAIL: aplay/arecord failed, look for previous errors"
83+
fi
84+
85+
check_soundfile_for_glitches "$result_filename"
86+
if [ $? -eq 1 ]; then
87+
test_pass=false
88+
dlogi "TEST $((i+1)) FAIL: Found glitch in the recording"
89+
fi
90+
91+
if [ "$test_pass" = true ]; then
92+
dlogi "TEST $((i+1)) PASS: No issues found."
93+
else
94+
failures=$((failures+1))
95+
fi
7496
done
7597
set -e
76-
77-
if check_soundfile_for_glitches "${all_result_files[@]}"; then
78-
dlogi "All files correct"
98+
99+
if [ "$failures" = 0 ]; then
100+
dlogi "PASS: All testcases passed"
79101
else
80-
die "Detected corrupted files!"
102+
die "FAIL: Found $failures testcase failed"
81103
fi
82104
}
83105

‎test-case/check-src-rec.sh‎

Lines changed: 28 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -30,11 +30,11 @@ source "$(dirname "${BASH_SOURCE[0]}")"/../case-lib/lib.sh
3030
OPT_NAME['t']='tplg' OPT_DESC['t']='tplg file, default value is env TPLG: $''TPLG'
3131
OPT_HAS_ARG['t']=1 OPT_VAL['t']="$TPLG"
3232

33-
OPT_NAME['p']='playback_device' OPT_DESC['p']='ALSA pcm playback device. Example: hw:0,1'
34-
OPT_HAS_ARG['p']=1 OPT_VAL['p']=''
33+
OPT_NAME['p']='playback_device' OPT_DESC['p']='ALSA pcm playback device. Default: hw:0,2'
34+
OPT_HAS_ARG['p']=1 OPT_VAL['p']='hw:0,2'
3535

36-
OPT_NAME['c']='capture_device' OPT_DESC['c']='ALSA pcm capture device. Example: hw:0,1'
37-
OPT_HAS_ARG['c']=1 OPT_VAL['c']=''
36+
OPT_NAME['c']='capture_device' OPT_DESC['c']='ALSA pcm capture device. Default: hw:0,2'
37+
OPT_HAS_ARG['c']=1 OPT_VAL['c']='hw:0,2'
3838

3939
OPT_NAME['s']='sof-logger' OPT_DESC['s']="Open sof-logger trace the data will store at $LOG_ROOT"
4040
OPT_HAS_ARG['s']=0 OPT_VAL['s']=1
@@ -51,30 +51,48 @@ init_globals()
5151

5252
rec_opt="-f S16_LE -c 2 -d 7"
5353

54-
test_sound_filename=$LOG_ROOT/play.wav
54+
test_sound_filename=$HOME/Music/play.wav
5555
all_result_files=()
5656
}
5757

5858
run_tests()
5959
{
6060
dlogi "Generate 48 kHz chirp 0 - 20 kHz"
61-
ffmpeg -y -f lavfi -i "aevalsrc='sin(2000*t*2*PI*t)':s=48000:d=5" -ac 2 "$test_sound_filename"
61+
ffmpeg -loglevel error -y -f lavfi -i "aevalsrc='sin(2000*t*2*PI*t)':s=48000:d=5" -ac 2 "$test_sound_filename"
6262

63+
failures=0
6364
set +e
6465
for i in "${!sample_rates[@]}"
6566
do
67+
test_pass=true
6668
sample_rate=${sample_rates[$i]}
69+
dlogi "--------------- TEST $((i+1)): PLAY SAMPLE RATE 48000 Hz, RECORD IN $sample_rate Hz ---------------"
6770

6871
result_filename=$LOG_ROOT/rec_$sample_rate.wav
69-
all_result_files+=("$result_filename")
7072
play_and_record "-D$capture_dev $rec_opt -r $sample_rate $result_filename" "-D$playback_dev $test_sound_filename"
73+
if [ $? -eq 1 ]; then
74+
test_pass=false
75+
dlogi "TEST $((i+1)) FAIL: aplay/arecord failed, look for previous errors"
76+
fi
77+
78+
check_soundfile_for_glitches "$result_filename"
79+
if [ $? -eq 1 ]; then
80+
test_pass=false
81+
dlogi "TEST $((i+1)) FAIL: Found glitch in the recording"
82+
fi
83+
84+
if [ "$test_pass" = true ]; then
85+
dlogi "TEST $((i+1)) PASS: No issues found."
86+
else
87+
failures=$((failures+1))
88+
fi
7189
done
7290
set -e
7391

74-
if check_soundfile_for_glitches "${all_result_files[@]}"; then
75-
dlogi "All files correct"
92+
if [ "$failures" = 0 ]; then
93+
dlogi "PASS: All testcases passed"
7694
else
77-
die "Detected corrupted files!"
95+
die "FAIL: Found $failures testcase failed"
7896
fi
7997
}
8098

0 commit comments

Comments
 (0)