Added waiting with testcases after attaching CalliopEO, see issue cal…

…liope-edu#76
rzbrk · Aug 10, 2021 · e98c3f4 · e98c3f4
1 parent 11174a6
commit e98c3f4
Show file tree

Hide file tree

Showing 15 changed files with 124 additions and 12 deletions.
diff --git a/testcases/03_no-zip.sh b/testcases/03_no-zip.sh
@@ -13,6 +13,11 @@
 # Necessary clean-up
 #   Nothing to clean up
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -30,6 +35,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/04_transmission-5s.sh b/testcases/04_transmission-5s.sh
@@ -18,6 +18,11 @@
 # Necessary clean-up
 #   Remove created *.done and folder run_*/ and folder tmp/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -40,6 +45,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/05_corrupted_hex.sh b/testcases/05_corrupted_hex.sh
@@ -22,6 +22,11 @@
 # Necessary clean-up
 #   Remove created *.done and folder run_*/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -44,6 +49,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/06a_corrupted-zip.sh b/testcases/06a_corrupted-zip.sh
@@ -17,6 +17,11 @@
 # Necessary clean-up
 #   Remove created *.failed and folder run_*/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -35,6 +40,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/06b_corrupted-zip.sh b/testcases/06b_corrupted-zip.sh
@@ -17,6 +17,11 @@
 # Necessary clean-up
 #   Remove created *.failed and folder run_*/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -42,6 +47,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/07a_transm-timeout.sh b/testcases/07a_transm-timeout.sh
@@ -25,6 +25,12 @@
 # Necessary clean-up
 #   Remove created *.done, folder run_*/ and tmp files
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/comp.sh
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -37,23 +43,21 @@ max_tdiff=5 # seconds
 tmpdir="./tmp"
 ERRORS=0
 
-###############################################################################
-# Import necessary functions
-###############################################################################
-source testcases/shfuncs/comp.sh
-
 ###############################################################################
 # Information and instructions for the test operator
 ###############################################################################
 echo "Test: Handle transmission timeout"
 echo "---------------------------------"
 echo ""
-# Make sure, Calliope is disconnected from Astro Pi
+# Make sure, Calliope is connected to Astro Pi
 if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     ans=""
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 
@@ -149,7 +153,6 @@ else
     ERRORS=$((ERRORS+1))
 fi
 
-# Check md5sums for hex and data files
 run_folder=$(find . -type d -ipath "./run_*")
 echo -n "Check 5/6: Check .data file in ${run_folder} ... "
 # Compare .data file created in test with "template" data file in folder

diff --git a/testcases/07b_transm-timeout.sh b/testcases/07b_transm-timeout.sh
@@ -24,6 +24,12 @@
 # Necessary clean-up
 #   Remove created *.done, folder run_*/ and tmp files
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/comp.sh
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -37,11 +43,6 @@ zipfile="01.zip"
 tmpdir="./tmp"
 ERRORS=0
 
-###############################################################################
-# Import necessary functions
-###############################################################################
-source testcases/shfuncs/comp.sh
-
 ###############################################################################
 # Information and instructions for the test operator
 ###############################################################################
@@ -54,6 +55,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/08a_data-limit.sh b/testcases/08a_data-limit.sh
@@ -22,6 +22,11 @@
 # Necessary clean-up
 #   Remove created *.done and folders run_*/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -45,6 +50,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/08b_data-limit.sh b/testcases/08b_data-limit.sh
@@ -23,6 +23,11 @@
 # Necessary clean-up
 #   Remove created *.done and folders run_*/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -36,6 +41,11 @@ md5file="checksum.md5"
 tmpdir="./tmp"
 ERRORS=0
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Information and instructions for the test operator
 ###############################################################################
@@ -48,6 +58,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/09a_transmission-no-start.sh b/testcases/09a_transmission-no-start.sh
@@ -18,6 +18,11 @@
 # Necessary clean-up
 #   Remove created *.done and folder run_*/ and folder tmp/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -39,6 +44,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/09b_transmission-no-start.sh b/testcases/09b_transmission-no-start.sh
@@ -18,6 +18,11 @@
 # Necessary clean-up
 #   Remove created *.done and folder run_*/ and folder tmp/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -42,6 +47,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/10_multi.sh b/testcases/10_multi.sh
@@ -19,6 +19,11 @@
 # Necessary clean-up
 #   Remove created *.done and folder run_*/ and folder tmp/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -42,6 +47,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/11_no-crlf.sh b/testcases/11_no-crlf.sh
@@ -19,6 +19,11 @@
 # Necessary clean-up
 #   Remove created *.done and folders run_*/
 
+###############################################################################
+# Import necessary functions
+###############################################################################
+source testcases/shfuncs/wait_for_calliope.sh
+
 ###############################################################################
 # Variables and definitions for this testcase
 ###############################################################################
@@ -41,6 +46,9 @@ if [ "${CALLIOPE_ATTACHED}" != "yes" ]; then
     while [[ ! ${ans} =~ [Yy] ]]; do
         read -p "Confirm, Calliope Mini is attached to USB [y] " ans
     done
+    if [ ${WAIT_AFTER_CALL_ATTACHED} -eq 1 ]; then
+        wait_for_calliope
+    fi
     CALLIOPE_ATTACHED="yes"
 fi
 

diff --git a/testcases/shfuncs/wait_for_calliope.sh b/testcases/shfuncs/wait_for_calliope.sh
@@ -0,0 +1,8 @@
+#!/bin/bash
+
+wait_for_calliope() {
+    # sleep 5 seconds
+    sleep 5
+    # set variable to 0 (no need to wait anymore)
+    WAIT_AFTER_CALL_ATTACHED=0
+}
diff --git a/testing.sh b/testing.sh
@@ -19,6 +19,10 @@ TEST_RESULTS=""
 
 CALLIOPE_ATTACHED=""
 
+# This variable indicates if we need to wait a little bit after attaching
+# the CalliopEO (see issue #76)
+WAIT_AFTER_CALL_ATTACHED=1
+
 # Cd into the directory where testing.sh is located in the case
 # this script was called from somewhere else.
 cd $(dirname ${0})