add try/catch blocks

Author: msandula12

src/common/rollbar-api.js

@@ -21,44 +21,57 @@ class RollbarAPI {
 
   async deploy(request, deployId) {
     let resp;
-    if(deployId) {
-      output.verbose('', 'Update to an existing deploy with deploy_id: ' + deployId);
-      resp = await this.axios.patch('/deploy/' + deployId, request);
-    } else {
-      output.verbose('','deploy_id not present so likely a new deploy');
-      resp = await this.axios.post('/deploy', request);
-    }
+    try {
+      if(deployId) {
+        output.verbose('', 'Update to an existing deploy with deploy_id: ' + deployId);
+        resp = await this.axios.patch('/deploy/' + deployId, request);
+      } else {
+        output.verbose('','deploy_id not present so likely a new deploy');
+        resp = await this.axios.post('/deploy', request);
+      }
 
-    // Output deploy-id
-    if (resp.status === 200) {
-      output.success('', resp.data.data);
+      // Output deploy-id
+      if (resp.status === 200) {
+        output.success('', resp.data.data);
+      }
+      return this.processResponse(resp);
+    } catch (error) {
+      output.verbose('', 'axios threw error:', error);
+      return this.processResponse(error.response || { data: error.message, status: error.status || 500, statusText: error.statusText || 'Axios Error' });
     }
-    return this.processResponse(resp);
   }
 
   async sigendURLsourcemaps(request) {
-
-    const resp = await this.axios.post(
-      '/signed_url/sourcemap_bundle',  { version: request.version , prefix_url: request.baseUrl}
-    );
-    return this.processSignedURLResponse(resp);
+    try {
+      const resp = await this.axios.post(
+        '/signed_url/sourcemap_bundle',  { version: request.version , prefix_url: request.baseUrl}
+      );
+      return this.processSignedURLResponse(resp);
+    } catch (error) {
+      output.verbose('', 'axios threw error:', error);
+      return this.processSignedURLResponse(error.response || { data: error.message, status: error.status || 500, statusText: error.statusText || 'Axios Error' });
+    }
   }
 
   async sourcemaps(request) {
     output.verbose('', 'minified_url: ' + request.minified_url);
 
     const form = this.convertRequestToForm(request);
-    const resp = await this.axios.post(
-      '/sourcemap',
-      form.getBuffer(), // use buffer to prevent unwanted string escaping.
-      { headers: {
-        // axios needs some help with headers for form data.
-        'Content-Type': `multipart/form-data; boundary=${form.getBoundary()}`,
-        'Content-Length': form.getLengthSync()
-      }}
-    );
-
-    return this.processResponse(resp);
+    try {
+      const resp = await this.axios.post(
+        '/sourcemap',
+        form.getBuffer(), // use buffer to prevent unwanted string escaping.
+        { headers: {
+          // axios needs some help with headers for form data.
+          'Content-Type': `multipart/form-data; boundary=${form.getBoundary()}`,
+          'Content-Length': form.getLengthSync()
+        }}
+      );
+      return this.processResponse(resp);
+    } catch (error) {
+      output.verbose('', 'axios threw error:', error);
+      return this.processResponse(error.response || { data: error.message, status: error.status || 500, statusText: error.statusText || 'Axios Error' });
+    }
   }
 
   convertRequestToForm(request) {

test/sourcemaps/scanner.test.js

@@ -139,7 +139,7 @@ describe('.scan()', function() {
     await scanner.scan();
     const files = scanner.mappedFiles();
 
-    expect(files[0].errors[0].error).to.have.string('Error parsing map file: Unexpected token $ in JSON at position 24');
+    expect(files[0].errors[0].error).to.have.string('Error parsing map file:');
     expect(files[1].errors[0].error).to.have.string('Error parsing map file: "sources" is a required argument');
   });
 });