diff options
Diffstat (limited to 'spec/frontend/notes/components/discussion_actions_spec.js')
-rw-r--r-- | spec/frontend/notes/components/discussion_actions_spec.js | 33 |
1 files changed, 3 insertions, 30 deletions
diff --git a/spec/frontend/notes/components/discussion_actions_spec.js b/spec/frontend/notes/components/discussion_actions_spec.js index b26eb00bfdc..48e569720e9 100644 --- a/spec/frontend/notes/components/discussion_actions_spec.js +++ b/spec/frontend/notes/components/discussion_actions_spec.js @@ -4,7 +4,6 @@ import DiscussionActions from '~/notes/components/discussion_actions.vue'; import ReplyPlaceholder from '~/notes/components/discussion_reply_placeholder.vue'; import ResolveDiscussionButton from '~/notes/components/discussion_resolve_button.vue'; import ResolveWithIssueButton from '~/notes/components/discussion_resolve_with_issue_button.vue'; -import JumpToNextDiscussionButton from '~/notes/components/discussion_jump_to_next_button.vue'; import createStore from '~/notes/stores'; // NOTE: clone mock_data so that it is not accidentally mutated @@ -21,7 +20,7 @@ const createUnallowedNote = () => describe('DiscussionActions', () => { let wrapper; - const createComponentFactory = (shallow = true) => (props, options) => { + const createComponentFactory = (shallow = true) => (props) => { const store = createStore(); const mountFn = shallow ? shallowMount : mount; @@ -35,11 +34,6 @@ describe('DiscussionActions', () => { shouldShowJumpToNextDiscussion: true, ...props, }, - provide: { - glFeatures: { - hideJumpToNextUnresolvedInThreads: options?.hideJumpToNextUnresolvedInThreads, - }, - }, }); }; @@ -55,7 +49,6 @@ describe('DiscussionActions', () => { expect(wrapper.find(ReplyPlaceholder).exists()).toBe(true); expect(wrapper.find(ResolveDiscussionButton).exists()).toBe(true); expect(wrapper.find(ResolveWithIssueButton).exists()).toBe(true); - expect(wrapper.find(JumpToNextDiscussionButton).exists()).toBe(true); }); it('only renders reply placholder if disccusion is not resolvable', () => { @@ -66,7 +59,6 @@ describe('DiscussionActions', () => { expect(wrapper.find(ReplyPlaceholder).exists()).toBe(true); expect(wrapper.find(ResolveDiscussionButton).exists()).toBe(false); expect(wrapper.find(ResolveWithIssueButton).exists()).toBe(false); - expect(wrapper.find(JumpToNextDiscussionButton).exists()).toBe(false); }); it('does not render resolve with issue button if resolveWithIssuePath is falsy', () => { @@ -75,12 +67,6 @@ describe('DiscussionActions', () => { expect(wrapper.find(ResolveWithIssueButton).exists()).toBe(false); }); - it('does not render jump to next discussion button if shouldShowJumpToNextDiscussion is false', () => { - createComponent({ shouldShowJumpToNextDiscussion: false }); - - expect(wrapper.find(JumpToNextDiscussionButton).exists()).toBe(false); - }); - describe.each` desc | notes | shouldRender ${'with no notes'} | ${[]} | ${true} @@ -101,13 +87,6 @@ describe('DiscussionActions', () => { }); }); - it('does not render jump to next discussion button if feature flag is enabled', () => { - const createComponent = createComponentFactory(); - createComponent({}, { hideJumpToNextUnresolvedInThreads: true }); - - expect(wrapper.find(JumpToNextDiscussionButton).exists()).toBe(false); - }); - describe('events handling', () => { const createComponent = createComponentFactory(false); @@ -117,19 +96,13 @@ describe('DiscussionActions', () => { it('emits showReplyForm event when clicking on reply placeholder', () => { jest.spyOn(wrapper.vm, '$emit'); - wrapper - .find(ReplyPlaceholder) - .find('button') - .trigger('click'); + wrapper.find(ReplyPlaceholder).find('button').trigger('click'); expect(wrapper.vm.$emit).toHaveBeenCalledWith('showReplyForm'); }); it('emits resolve event when clicking on resolve button', () => { jest.spyOn(wrapper.vm, '$emit'); - wrapper - .find(ResolveDiscussionButton) - .find('button') - .trigger('click'); + wrapper.find(ResolveDiscussionButton).find('button').trigger('click'); expect(wrapper.vm.$emit).toHaveBeenCalledWith('resolve'); }); }); |