Check if location key is present and improves tests

ee/app/assets/javascripts/vue_merge_request_widget/stores/mr_widget_store.js

@@ -76,15 +76,18 @@ export default class MergeRequestStore extends CEMergeRequestStore {
 
   static addPathToIssues(issues, path) {
     return issues.map((issue) => {
-      let parsedUrl = `${path}/${issue.location.path}`;
+      if (issue.location) {
+        let parsedUrl = `${path}/${issue.location.path}`;
 
-      if (issue.location.lines && issue.location.lines.begin) {
-        parsedUrl += `#L${issue.location.lines.begin}`;
-      }
+        if (issue.location.lines && issue.location.lines.begin) {
+          parsedUrl += `#L${issue.location.lines.begin}`;
+        }
 
-      return Object.assign({}, issue, {
-        location: Object.assign({}, issue.location, { urlPath: parsedUrl }),
-      });
+        return Object.assign({}, issue, {
+          location: Object.assign({}, issue.location, { urlPath: parsedUrl }),
+        });
+      }
+      return issue;
     });
   }
 }

spec/javascripts/vue_mr_widget/stores/mr_widget_store_spec.js

@@ -76,5 +76,11 @@ describe('MergeRequestStore', () => {
         MergeRequestStore.addPathToIssues(headIssues, 'path')[0].location.urlPath,
       ).toEqual(`path/${headIssues[0].location.path}#L${headIssues[0].location.lines.begin}`);
     });
+
+    it('should return the same object whe there is no locaiton', () => {
+      expect(
+        MergeRequestStore.addPathToIssues([{ check_name: 'foo' }], 'path'),
+      ).toEqual([{ check_name: 'foo' }]);
+    });
   });
 });