Added exception for helm-path

Azure · Oct 15, 2019 · 785a9d3 · 785a9d3
1 parent 0c30698
commit 785a9d3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/run.js b/lib/run.js
@@ -60,7 +60,7 @@ function getStableHelmVersion() {
     });
 }
 var walkSync = function (dir, filelist, fileToFind) {
-    var fs = fs || require('fs'), files = fs.readdirSync(dir);
+    var files = fs.readdirSync(dir);
     filelist = filelist || [];
     files.forEach(function (file) {
         if (fs.statSync(path.join(dir, file)).isDirectory()) {
@@ -95,6 +95,7 @@ function downloadHelm(version) {
         }
         const helmpath = findHelm(cachedToolpath);
         if (!helmpath) {
+            throw new Error(util.format("Helm executable not found in path ", cachedToolpath));
         }
         fs.chmodSync(helmpath, '777');
         return helmpath;

diff --git a/src/run.ts b/src/run.ts
@@ -53,8 +53,7 @@ async function getStableHelmVersion(): Promise<string> {
 
 
 var walkSync = function(dir, filelist, fileToFind) {
-    var fs = fs || require('fs'),
-        files = fs.readdirSync(dir);
+    var files = fs.readdirSync(dir);
     filelist = filelist || [];
     files.forEach(function(file) {
       if (fs.statSync(path.join(dir, file)).isDirectory()) {
@@ -89,7 +88,7 @@ async function downloadHelm(version: string): Promise<string> {
 
     const helmpath = findHelm(cachedToolpath);
     if (!helmpath) {
-
+        throw new Error(util.format("Helm executable not found in path ", cachedToolpath));
     }
 
     fs.chmodSync(helmpath, '777');