Merge branch 'winh-issue-boards-clearDetailIssue' into 'master'

Extract clearDetailIssue() to issue boards store

See merge request gitlab-org/gitlab-ce!28825

app/assets/javascripts/boards/components/sidebar/remove_issue.vue

@@ -48,7 +48,7 @@ export default Vue.extend({
         list.removeIssue(issue);
       });
 
-      boardsStore.detail.issue = {};
+      boardsStore.clearDetailIssue();
     },
     /**
      * Build the default patch request.

app/assets/javascripts/boards/index.js

@@ -167,7 +167,7 @@ export default () => {
         boardsStore.detail.issue = newIssue;
       },
       clearDetailIssue() {
-        boardsStore.detail.issue = {};
+        boardsStore.clearDetailIssue();
       },
       toggleSubscription(id) {
         const { issue } = boardsStore.detail;

app/assets/javascripts/boards/stores/boards_store.js

@@ -188,6 +188,10 @@ const boardsStore = {
   updateFiltersUrl() {
     window.history.pushState(null, null, `?${this.filter.path}`);
   },
+
+  clearDetailIssue() {
+    this.detail.issue = {};
+  },
 };
 
 BoardsStoreEE.initEESpecific(boardsStore);

spec/javascripts/boards/boards_store_spec.js

@@ -268,4 +268,14 @@ describe('Store', () => {
       });
     });
   });
+
+  describe('clearDetailIssue', () => {
+    it('resets issue details', () => {
+      boardsStore.detail.issue = 'something';
+
+      boardsStore.clearDetailIssue();
+
+      expect(boardsStore.detail.issue).toEqual({});
+    });
+  });
 });