Merge pull request #12548 from himdel/euwe-travis-js

[EUWE] Travis JS failures - fail early when bower install fails
ManageIQ · Nov 10, 2016 · e222d67 · e222d67
2 parents 5f2e38f + c52a427
commit e222d67
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/tools/ci/before_install.sh b/tools/ci/before_install.sh
@@ -17,8 +17,7 @@ export BUNDLE_GEMFILE=${PWD}/Gemfile
 
 # suites that need bower assets to work: javascript, vmdb
 if [[ "$TEST_SUITE" = "javascript" ]] || [[ "$TEST_SUITE" = "vmdb" ]]; then
-  which bower || npm install -g bower
-  bower install --allow-root -F --config.analytics=false
+  source $TRAVIS_BUILD_DIR/tools/ci/setup_js_env.sh
 fi
 
 set +v
diff --git a/tools/ci/setup_js_env.sh b/tools/ci/setup_js_env.sh
@@ -0,0 +1,8 @@
+which bower || npm install -g bower
+bower install --allow-root -F --config.analytics=false
+STATUS=$?
+echo bower exit code: $STATUS
+
+# fail the whole test suite if bower install failed
+[ $STATUS = 0 ] || exit 1
+[ -d vendor/assets/bower_components ] || exit 1