Changed logic in seleniumbender script to reduce repeated code.

Moved ROOTDIR variable to env_vars.sh .

Author: freskoulix

bin/env_vars.sh.template

@@ -13,3 +13,7 @@ export DISQUS_API_PUBLIC=""
 export DISQUS_SSO_ID=""
 export DISQUS_SSO_USERNAME=""
 export DISQUS_SSO_EMAIL=""
+
+export EMAIL=""
+
+export ROOTDIR="/path/to/seleniumTests"

bin/seleniumbender

@@ -1,25 +1,22 @@
 #!/bin/bash
 
-export CAPABILITIES='capabilities_firefox.yaml'
-
 source ./env_vars.sh
 
-ROOTDIR="/path/to/seleniumTests"
-
 if [ $# -lt 1 ]; then
     echo "Please provide an argument or run -h(--help) option"
     exit 1
 fi
 
 app_name="$0"
-common=0
-libraries=0
+
+target=0
 examples=0
+libraries=0
 sketches=0
-staging=0
+common=0
 noplugin=0
 walkthrough=0
-target=0
+staging=0
 
 while true; do
     case "$1" in
@@ -29,29 +26,32 @@ while true; do
                 " libraries - Visits all the libraries and examples at libraries view\n"  \
                 " examples - Compiles all the examples at libraries view\n"   \
                 " sketches - Compiles the examples of user cb_compile_tester\n"    \
+                " target - Compiles the examples of given libraries as comma seperated list\n"    \
+                " noplugin - Runs tests that do not need plugin installed\n"  \
+                " walkthrough - Runs tests for walkthrough\n"  \
+                " staging - Runs tests in staging\n"  \
                 " help - Display this help and exit"
             exit 0
             ;;
-        common) common=1
-            ;;
-        libraries) libraries=1
-            ;;
-        examples) examples=1
-            ;;
-        target) compile=1
+        target) target=1
             shift
-            target=1
             TARGETS=$@
             break
             ;;
+        examples) examples=1
+            ;;
+        libraries) libraries=1
+            ;;
         sketches) sketches=1
             ;;
-        staging) staging=1
+        common) common=1
             ;;
         noplugin) noplugin=1
             ;;
         walkthrough) walkthrough=1
             ;;
+        staging) staging=1
+            ;;
         -*) echo -e "error: unknown argument: ${1}.\nRun ${app_name} -h."
             exit 1
             ;;
@@ -64,19 +64,16 @@ done
 cd ..
 
 URL="https://codebender.cc"
-URL_STAGING="https://staging.codebender.cc"
 SOURCE="codebender_cc"
 
 email_date=$(date +"%Y-%m-%d %H:%M:%S")
 
-if [ "${common}" -eq 1 ]; then
-    IDENTIFIER="common"
-    tox tests/common -- --url=${URL} --source=${SOURCE} --plugin
-    RETVAL=$?
-    if [ "${RETVAL}" -eq 1 ]; then
-        mail -s "Selenium Tests: ${IDENTIFIER} Failed To Run" ${EMAIL} <<< 'Something went wrong with common tests. Please check the logs.'
-    fi
-    exit ${RETVAL}
+RETVAL=0
+NO_LOGS=0
+
+if [ "${target}" -eq 1 ]; then
+    IDENTIFIER="test_target_libraries"
+    tox tests/target_libraries -- --url=${URL} --source=${SOURCE} -F --plugin --libraries=${TARGETS}
 elif [ "${examples}" -eq 1 ]; then
     IDENTIFIER="libraries_test"
     tox tests/libraries -- --url=${URL} --source=${SOURCE} -F --plugin
@@ -86,18 +83,16 @@ elif [ "${libraries}" -eq 1 ]; then
 elif [ "${sketches}" -eq 1 ]; then
     IDENTIFIER="cb_compile_tester"
     tox tests/compile_tester -- --url=${URL} --source=${SOURCE} -F --plugin
-elif [ "${staging}" -eq 1 ]; then
-    IDENTIFIER="cb_compile_tester_staging"
-    tox tests/compile_tester -- --url=${URL_STAGING} --source=${SOURCE} -F --plugin
-    exit $?
+elif [ "${common}" -eq 1 ]; then
+    IDENTIFIER="common"
+    tox tests/common -- --url=${URL} --source=${SOURCE} --plugin
+    RETVAL=$?
+    NO_LOGS=1
 elif [ "${noplugin}" -eq 1 ]; then
     IDENTIFIER="noplugin"
     tox tests/noplugin -- --url=${URL} --source=${SOURCE}
     RETVAL=$?
-    if [ "${RETVAL}" -eq 1 ]; then
-        mail -s "Selenium Tests: ${IDENTIFIER} Failed To Run" ${EMAIL} <<< 'Something went wrong with noplugin tests. Please check the logs.'
-    fi
-    exit ${RETVAL}
+    NO_LOGS=1
 elif [ "${walkthrough}" -eq 1 ]; then
     IDENTIFIER="walkthrough"
     RETVALS=()
@@ -114,22 +109,31 @@ elif [ "${walkthrough}" -eq 1 ]; then
     tox tests/walkthrough -- --url=${URL} --source=${SOURCE} --plugin
     RETVALS+=($?)
 
-    RETVAL=0
     for i in "${RETVALS[@]}"
     do
         if [ ${i} -ne 0 ]; then
             RETVAL=${i}
         fi
     done
-    if [ "${RETVAL}" -ne 0 ]; then
-        mail -s "Selenium Tests: ${IDENTIFIER} Failed To Run" ${EMAIL} <<< 'Something went wrong with noplugin tests. Please check the logs.'
-    fi
+    NO_LOGS=1
+elif [ "${staging}" -eq 1 ]; then
+    IDENTIFIER="cb_compile_tester_staging"
+    URL="https://staging.codebender.cc"
+    tox tests/compile_tester -- --url=${URL} --source=${SOURCE} -F --plugin
+    # No need to send email for tests in staging
+    exit $?
+fi
+
+# email notification without attaching any logs
+if [ ${RETVAL} -ne 0 ]; then
+    mail -s "Selenium Tests: ${IDENTIFIER} Failed To Run" ${EMAIL} <<< "Something went wrong with ${IDENTIFIER} tests. Please check the logs."
+fi
+
+if [ ${NO_LOGS} -eq 1 ]; then
     exit ${RETVAL}
-elif [ "${target}" -eq 1 ]; then
-    IDENTIFIER="test_target_libraries"
-    tox tests/target_libraries -- --url=${URL} --source=${SOURCE} -F --plugin --libraries=${TARGETS}
 fi
 
+# email notification with attaching the produced reports
 DATE=$(date +"%Y-%m-%d") # Get the current date
 LOGS="${ROOTDIR}/logs"
 REPORTS="${ROOTDIR}/reports"