Skip to content

Commit 9396387

Browse files
committed
fix(tests): Add missing requirement and improve logging
1 parent f75b7db commit 9396387

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

Diff for: .github/scripts/tests_run.sh

+4-4
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,10 @@ function run_test {
4848
return 0
4949
fi
5050

51-
local right_target
52-
right_target=$(grep -E "^CONFIG_IDF_TARGET=\"$target\"$" "$sdkconfig_path")
53-
if [ -z "$right_target" ]; then
54-
printf "\033[91mError: Sketch %s compiled for different target\n\033[0m\n" "$sketchname"
51+
local compiled_target
52+
compiled_target=$(grep -E "CONFIG_IDF_TARGET=" "$sdkconfig_path" | cut -d'"' -f2)
53+
if [ "$compiled_target" != "$target" ]; then
54+
printf "\033[91mError: Sketch %s compiled for %s, expected %s\033[0m\n" "$sketchname" "$compiled_target" "$target"
5555
printf "\n\n\n"
5656
return 1
5757
fi

Diff for: tests/requirements.txt

+1
Original file line numberDiff line numberDiff line change
@@ -5,3 +5,4 @@ pytest-embedded-serial-esp==1.12.0
55
pytest-embedded-arduino==1.12.0
66
pytest-embedded-wokwi==1.12.0
77
pytest-embedded-qemu==1.12.0
8+
esptool==4.8.1

0 commit comments

Comments
 (0)