diff options
Diffstat (limited to 'spec/frontend/diffs/components/app_spec.js')
-rw-r--r-- | spec/frontend/diffs/components/app_spec.js | 34 |
1 files changed, 31 insertions, 3 deletions
diff --git a/spec/frontend/diffs/components/app_spec.js b/spec/frontend/diffs/components/app_spec.js index 34547238c23..8a1c5547581 100644 --- a/spec/frontend/diffs/components/app_spec.js +++ b/spec/frontend/diffs/components/app_spec.js @@ -56,6 +56,7 @@ describe('diffs/components/app', () => { endpointMetadata: `${TEST_HOST}/diff/endpointMetadata`, endpointBatch: `${TEST_HOST}/diff/endpointBatch`, endpointCoverage: `${TEST_HOST}/diff/endpointCoverage`, + endpointCodequality: '', projectPath: 'namespace/project', currentUser: {}, changesEmptyStateIllustration: '', @@ -105,7 +106,6 @@ describe('diffs/components/app', () => { jest.spyOn(wrapper.vm, 'fetchDiffFilesBatch').mockImplementation(fetchResolver); jest.spyOn(wrapper.vm, 'fetchCoverageFiles').mockImplementation(fetchResolver); jest.spyOn(wrapper.vm, 'setDiscussions').mockImplementation(() => {}); - jest.spyOn(wrapper.vm, 'startRenderDiffsQueue').mockImplementation(() => {}); jest.spyOn(wrapper.vm, 'unwatchDiscussions').mockImplementation(() => {}); jest.spyOn(wrapper.vm, 'unwatchRetrievingBatches').mockImplementation(() => {}); store.state.diffs.retrievingBatches = true; @@ -119,7 +119,6 @@ describe('diffs/components/app', () => { await nextTick(); - expect(wrapper.vm.startRenderDiffsQueue).toHaveBeenCalled(); expect(wrapper.vm.fetchDiffFilesMeta).toHaveBeenCalled(); expect(wrapper.vm.fetchDiffFilesBatch).toHaveBeenCalled(); expect(wrapper.vm.fetchCoverageFiles).toHaveBeenCalled(); @@ -134,7 +133,6 @@ describe('diffs/components/app', () => { await nextTick(); - expect(wrapper.vm.startRenderDiffsQueue).toHaveBeenCalled(); expect(wrapper.vm.fetchDiffFilesMeta).toHaveBeenCalled(); expect(wrapper.vm.fetchDiffFilesBatch).toHaveBeenCalled(); expect(wrapper.vm.fetchCoverageFiles).toHaveBeenCalled(); @@ -144,6 +142,16 @@ describe('diffs/components/app', () => { }); }); + describe('codequality diff', () => { + it('does not fetch code quality data on FOSS', async () => { + createComponent(); + jest.spyOn(wrapper.vm, 'fetchCodequality'); + wrapper.vm.fetchData(false); + + expect(wrapper.vm.fetchCodequality).not.toHaveBeenCalled(); + }); + }); + it.each` props | state | expected ${{ isFluidLayout: true }} | ${{ isParallelView: false }} | ${false} @@ -697,4 +705,24 @@ describe('diffs/components/app', () => { ); }); }); + + describe('diff file tree is aware of review bar', () => { + it('it does not have review-bar-visible class when review bar is not visible', () => { + createComponent({}, ({ state }) => { + state.diffs.diffFiles = [{ file_hash: '111', file_path: '111.js' }]; + }); + + expect(wrapper.find('.js-diff-tree-list').exists()).toBe(true); + expect(wrapper.find('.js-diff-tree-list.review-bar-visible').exists()).toBe(false); + }); + + it('it does have review-bar-visible class when review bar is visible', () => { + createComponent({}, ({ state }) => { + state.diffs.diffFiles = [{ file_hash: '111', file_path: '111.js' }]; + state.batchComments.drafts = ['draft message']; + }); + + expect(wrapper.find('.js-diff-tree-list.review-bar-visible').exists()).toBe(true); + }); + }); }); |