Merge branch 'ss/add-close-event' into 'master'

Splits changed event with close and open in board_editable_item

See merge request gitlab-org/gitlab!43756

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

@@ -36,7 +36,7 @@ export default {
       }
 
       this.edit = true;
-      this.$emit('changed', this.edit);
+      this.$emit('open');
       window.addEventListener('click', this.collapseWhenOffClick);
     },
     collapse() {
@@ -45,7 +45,7 @@ export default {
       }
 
       this.edit = false;
-      this.$emit('changed', this.edit);
+      this.$emit('close');
       window.removeEventListener('click', this.collapseWhenOffClick);
     },
   },

spec/frontend/boards/components/sidebar/board_editable_item_spec.js

@@ -96,12 +96,22 @@ describe('boards sidebar remove issue', () => {
       expect(findExpanded().isVisible()).toBe(false);
     });
 
-    it('emits changed event', async () => {
+    it('emits close event', async () => {
       document.body.click();
 
       await wrapper.vm.$nextTick();
 
-      expect(wrapper.emitted().changed[1][0]).toBe(false);
+      expect(wrapper.emitted().close.length).toBe(1);
     });
   });
+
+  it('emits open when edit button is clicked and edit is initailized to false', async () => {
+    createComponent({ canUpdate: true });
+
+    findEditButton().vm.$emit('click');
+
+    await wrapper.vm.$nextTick();
+
+    expect(wrapper.emitted().open.length).toBe(1);
+  });
 });