Fail early and travis_retry when npm install fails

scripts/run_travis.sh

@@ -1,17 +1,18 @@
 #!/bin/bash
+
 toplevel=$(git rev-parse --show-toplevel)
 testdir=${toplevel}/tests/selenium
 
 function run_lint {
-  make -C ${toplevel} lint
+  make -C "$toplevel" lint
   if [ $? != 0 ]; then
     echo "Linting errors"
     exit 1
   fi
 }
 
 function run_selenium {
-  py.test --capture=no --verbose --durations=10 ${testdir} # autodiscover and run the tests
+  py.test --capture=no --verbose --durations=10 "$testdir" # autodiscover and run the tests
 }
 
 if [ "$INFO" == "lint" ]; then

scripts/setup_travis.sh

@@ -1,4 +1,5 @@
 #!/bin/bash
+
 toplevel=$(git rev-parse --show-toplevel)
 
 function setup_chrome {
@@ -51,11 +52,8 @@ function browser_setup {
 }
 
 function setup_lint {
-  pushd "$toplevel"
-    # "--production" to skip installing devDependencies modules
-    npm install --production
-  popd
-
+  # "--production" to skip installing devDependencies modules
+  npm install --production || exit 1
 }
 
 # check that the desired browser is present as it might fail to install