Merge pull request #30 from yipeeio/build-fixes

fix api test and fail top-level build on errors
CATechnologiesTest · Dec 14, 2018 · 32c04da · 32c04da
2 parents a96281e + ccd9eaa
commit 32c04da
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/api/test/download_api.js b/api/test/download_api.js
@@ -13,7 +13,7 @@ function logAndThrow(err) {
 }
 
 describe('Yipee Download API Tests:', function() {
-    this.timeout(60000);
+    this.timeout(10000);
 
     after(function(done) {
         if (app.server) {
@@ -118,7 +118,7 @@ describe('Yipee Download API Tests:', function() {
             payload: flatYipee,
             retkey: 'helmFile',
             expectStatus: 200,
-            expectEntries: ["Chart.yaml", "values.yaml"]
+            expectEntries: ["flatYipee/Chart.yaml", "flatYipee/values.yaml"]
         }
     ];
 

diff --git a/build.sh b/build.sh
@@ -3,6 +3,6 @@ CVT_IMG=yipee-converter
 API_IMG=yipee-api
 UI_IMG=yipee-ui
 
-(cd converter; docker build -t $CVT_IMG .)
-(cd api; bash api_build.sh $CVT_IMG $API_IMG)
-(cd ui; docker build -t $UI_IMG .)
+(cd converter; docker build -t $CVT_IMG .) || exit 1
+(cd api; bash api_build.sh $CVT_IMG $API_IMG) || exit 1
+(cd ui; docker build -t $UI_IMG .) || exit 1