move before build logic into separate file

Author: Narretz

.travis.yml

@@ -52,11 +52,7 @@ install:
   - npm install
 
 before_script:
-  - mkdir -p $LOGS_DIR
-  - ./scripts/travis/start_browser_provider.sh
-  - npm install -g grunt-cli
-  - grunt package
-  - ./scripts/travis/wait_for_browser_provider.sh
+  - ./scripts/travis/before_build.sh
 
 script:
   - ./scripts/travis/build.sh

scripts/travis/before_build.sh

@@ -0,0 +1,16 @@
+#!/bin/bash
+
+set -e
+
+mkdir -p $LOGS_DIR
+
+if ["$JOB" != "ci-checks" ]; then
+  start_browser_provider.sh
+fi
+
+npm install -g grunt-cli
+
+if ["$JOB" != "ci-checks" ]; then
+  grunt package
+  wait_for_browser_provider.sh
+fi

scripts/travis/start_browser_provider.sh

@@ -1,15 +1,13 @@
 #!/bin/bash
 # Has to be run from project root directory.
 
-if ["$JOB" != "ci-checks" ]; then
-  if [ "$BROWSER_PROVIDER" == "browserstack" ]; then
-    echo "Using BrowserStack"
-  elif [ "$BROWSER_PROVIDER" == "saucelabs" ]; then
-    echo "Using SauceLabs"
-  else
-    echo "Invalid BROWSER_PROVIDER. Please set env var BROWSER_PROVIDER to 'saucelabs' or 'browserstack'."
-    exit 1
-  fi
-
-  ./lib/${BROWSER_PROVIDER}/start_tunnel.sh
+if [ "$BROWSER_PROVIDER" == "browserstack" ]; then
+  echo "Using BrowserStack"
+elif [ "$BROWSER_PROVIDER" == "saucelabs" ]; then
+  echo "Using SauceLabs"
+else
+  echo "Invalid BROWSER_PROVIDER. Please set env var BROWSER_PROVIDER to 'saucelabs' or 'browserstack'."
+  exit 1
 fi
+
+./lib/${BROWSER_PROVIDER}/start_tunnel.sh