Extract ReplyPlaceholder from NoteableDiscussion component

2 files changed, 48 insertions, 17 deletions

diff --git a/spec/javascripts/notes/components/discussion_reply_placeholder_spec.js b/spec/javascripts/notes/components/discussion_reply_placeholder_spec.js
new file mode 100644
index 00000000000..07a366cf339
--- /dev/null
+++ b/spec/javascripts/notes/components/discussion_reply_placeholder_spec.js
@@ -0,0 +1,34 @@
+import ReplyPlaceholder from '~/notes/components/discussion_reply_placeholder.vue';
+import { shallowMount, createLocalVue } from '@vue/test-utils';
+
+const localVue = createLocalVue();
+
+describe('ReplyPlaceholder', () => {
+  let wrapper;
+
+  beforeEach(() => {
+    wrapper = shallowMount(ReplyPlaceholder, {
+      localVue,
+    });
+  });
+
+  afterEach(() => {
+    wrapper.destroy();
+  });
+
+  it('emits onClick even on button click', () => {
+    const button = wrapper.find({ ref: 'button' });
+
+    button.trigger('click');
+
+    expect(wrapper.emitted()).toEqual({
+      onClick: [[]],
+    });
+  });
+
+  it('should render reply button', () => {
+    const button = wrapper.find({ ref: 'button' });
+
+    expect(button.text()).toEqual('Reply...');
+  });
+});
diff --git a/spec/javascripts/notes/components/noteable_discussion_spec.js b/spec/javascripts/notes/components/noteable_discussion_spec.js
index c4b7eb17393..2eae22e095f 100644
--- a/spec/javascripts/notes/components/noteable_discussion_spec.js
+++ b/spec/javascripts/notes/components/noteable_discussion_spec.js
@@ -1,6 +1,7 @@
 import { shallowMount, createLocalVue } from '@vue/test-utils';
 import createStore from '~/notes/stores';
 import noteableDiscussion from '~/notes/components/noteable_discussion.vue';
+import ReplyPlaceholder from '~/notes/components/discussion_reply_placeholder.vue';
 import '~/behaviors/markdown/render_gfm';
 import { noteableDataMock, discussionMock, notesDataMock } from '../mock_data';
 import mockDiffFile from '../../diffs/mock_data/diff_file';
@@ -57,27 +58,23 @@ describe('noteable_discussion component', () => {
   });
 
   describe('actions', () => {
-    it('should render reply button', () => {
-      expect(
-        wrapper
-          .find('.js-vue-discussion-reply')
-          .text()
-          .trim(),
-      ).toEqual('Reply...');
-    });
-
     it('should toggle reply form', done => {
-      wrapper.find('.js-vue-discussion-reply').trigger('click');
+      const replyPlaceholder = wrapper.find(ReplyPlaceholder);
 
-      wrapper.vm.$nextTick(() => {
-        expect(wrapper.vm.isReplying).toEqual(true);
+      wrapper.vm
+        .$nextTick()
+        .then(() => {
+          expect(wrapper.vm.isReplying).toEqual(false);
 
-        // There is a watcher for `isReplying` which will init autosave in the next tick
-        wrapper.vm.$nextTick(() => {
+          replyPlaceholder.vm.$emit('onClick');
+        })
+        .then(() => wrapper.vm.$nextTick())
+        .then(() => {
+          expect(wrapper.vm.isReplying).toEqual(true);
           expect(wrapper.vm.$refs.noteForm).not.toBeNull();
-          done();
-        });
-      });
+        })
+        .then(done)
+        .catch(done.fail);
     });
 
     it('does not render jump to discussion button', () => {