Merge pull request #209 from browserstack/package_err

Handle errors raised without a stacktrace while zipping

Author: nagpalkaran95

bin/helpers/checkUploaded.js

@@ -145,7 +145,8 @@ const checkUploadedMd5 = (bsConfig, args, instrumentBlocks) => {
         }
       });
     }).catch((err) => {
-      resolve({zipUrlPresent: false, packageUrlPresent: false, error: err.stack.substring(0,100)});
+      let errString = err.stack ? err.stack.toString().substring(0,100) : err.toString().substring(0,100);
+      resolve({zipUrlPresent: false, packageUrlPresent: false, error: errString});
     });
   });
 };

bin/helpers/packageInstaller.js

@@ -125,7 +125,7 @@ const packageWrapper = (bsConfig, packageDir, packageFile, md5data, instrumentBl
       Object.assign(obj, { packageArchieveCreated: true });
       return resolve(obj);
     }).catch((err) => {
-      obj.error = err.stack.substring(0,100);
+      obj.error = err.stack ? err.stack.toString().substring(0,100) : err.toString().substring(0,100);
       return resolve(obj);
     })
   })