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

Extract setIssueDetail() into issue boards store

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

app/assets/javascripts/boards/components/board_new_issue.vue

@@ -72,7 +72,7 @@ export default {
           // Need this because our jQuery very kindly disables buttons on ALL form submissions
           $(this.$refs.submitButton).enable();
 
-          boardsStore.detail.issue = issue;
+          boardsStore.setIssueDetail(issue);
           boardsStore.detail.list = this.list;
         })
         .catch(() => {

app/assets/javascripts/boards/index.js

@@ -168,7 +168,7 @@ export default () => {
             });
         }
 
-        boardsStore.detail.issue = newIssue;
+        boardsStore.setIssueDetail(newIssue);
       },
       clearDetailIssue() {
         boardsStore.clearDetailIssue();

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

@@ -212,7 +212,11 @@ const boardsStore = {
   },
 
   clearDetailIssue() {
-    this.detail.issue = {};
+    this.setIssueDetail({});
+  },
+
+  setIssueDetail(issueDetail) {
+    this.detail.issue = issueDetail;
   },
 };
 

spec/javascripts/boards/boards_store_spec.js

@@ -322,6 +322,17 @@ describe('Store', () => {
     });
   });
 
+  describe('setIssueDetail', () => {
+    it('sets issue details', () => {
+      boardsStore.detail.issue = 'some details';
+
+      const dummyValue = 'new details';
+      boardsStore.setIssueDetail(dummyValue);
+
+      expect(boardsStore.detail.issue).toEqual(dummyValue);
+    });
+  });
+
   describe('startMoving', () => {
     it('stores list and issue', () => {
       const dummyIssue = 'some issue';