Index: build/unittest.sh
===================================================================
diff -u -ree8def4767355f1424798bb13d8414d2af553ead -r49ce245222f48364bd40c33355be660b65ea4b01
--- build/unittest.sh	(.../unittest.sh)	(revision ee8def4767355f1424798bb13d8414d2af553ead)
+++ build/unittest.sh	(.../unittest.sh)	(revision 49ce245222f48364bd40c33355be660b65ea4b01)
@@ -2,7 +2,7 @@
 
 # Local example $./run.unittest.sh :0 5.12.5 /media/sf_VMSHARE
 
-PYCOMMON="" # use anything other that "" to use pycommon as SquishQt API lib source
+PYCOMMON="" # use anything other that "" to use pycommon as SquishQt API lib source # TODO pycommon is not needed right?
 reportDir=""
 
 function readTestsList() {
@@ -27,6 +27,7 @@
 	source ./supportscript/supportscript.sh
 }
 
+# shellcheck disable=SC2120
 function setup_Script_Environment() {
 
     if [ -z "$RUN_FOR_LOCAL" ];then #is NOT local
@@ -47,6 +48,7 @@
 
     DENALI_SQUISH=denaliSquish
 
+    # TODO what are these for local? Do we still need them?
     if [ ! -z "$RUN_FOR_LOCAL" ];then #is local
         DISPLAY_NO=:0
         QT_VER="5.12.5"
@@ -101,6 +103,7 @@
 }
 
 function setup_System_Environment() {
+    # TODO why are these exported?
     export PATH=$PATH:$QT_DIR/bin/:$QT_DIR/lib
     export PATH=$COCO_DIR/wrapper/bin/:$COCO_DIR/bin/:$PATH
     export LD_LIBRARY_PATH=$QT_DIR/lib
@@ -140,7 +143,7 @@
     cd $BUILD_PATH
 }
 
-function clone_Repositories() {
+function clone_Repositories() {  # TODO this function is not needed anymore
 
     cd $WORKING_DIR