Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 2ad2af6

Browse filesBrowse files
committed
fix(tests): Additional checks before running tests
1 parent 339d659 commit 2ad2af6
Copy full SHA for 2ad2af6

File tree

1 file changed

+12
-6
lines changed
Filter options

1 file changed

+12
-6
lines changed

‎.github/scripts/tests_run.sh

Copy file name to clipboardExpand all lines: .github/scripts/tests_run.sh
+12-6Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,13 +36,19 @@ function run_test() {
3636
printf "\n\n\n"
3737
return 0
3838
fi
39+
fi
3940

40-
local has_requirements=$(${CHECK_REQUIREMENTS} $sketchdir "$sdkconfig_path")
41-
if [ "$has_requirements" == "0" ]; then
42-
printf "\033[93mTarget $target does not meet the requirements for $sketchname. Skipping.\033[0m\n"
43-
printf "\n\n\n"
44-
return 0
45-
fi
41+
if [ ! -f $sdkconfig_path ]; then
42+
printf "\033[93mSketch $sketchname not built\nMight be due to missing target requirements or build failure\033[0m\n"
43+
printf "\n\n\n"
44+
return 0
45+
fi
46+
47+
local right_target=$(grep -E "^CONFIG_IDF_TARGET=\"$target\"$" "$sdkconfig_path")
48+
if [ -z "$right_target" ]; then
49+
printf "\033[91mError: Sketch $sketchname compiled for different target\n\033[0m\n"
50+
printf "\n\n\n"
51+
return 1
4652
fi
4753

4854
if [ $len -eq 1 ]; then

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.