diff options
author | Phil Hughes <me@iamphill.com> | 2018-11-09 09:44:07 +0000 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2018-11-27 11:40:39 +0000 |
commit | adf8ad9eee20a2b4ea08054e36fede62ba110e57 (patch) | |
tree | ffdab5c766778eaf28a9a7cc1ddaea6928287516 /spec/javascripts/notes/components/noteable_discussion_spec.js | |
parent | 921d6b1a13b5ec59217ab714b4daa6800500d95f (diff) | |
download | gitlab-ce-adf8ad9eee20a2b4ea08054e36fede62ba110e57.tar.gz |
Improve discussion rendering performance
Improve the renderign of new and existing discussions
by reducing the number of watchers on each object & array.
Previously every discussion change would trigger an update for every
discussion component.
Also tidied up some components to get them closer to our docs.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/51506
Diffstat (limited to 'spec/javascripts/notes/components/noteable_discussion_spec.js')
-rw-r--r-- | spec/javascripts/notes/components/noteable_discussion_spec.js | 37 |
1 files changed, 0 insertions, 37 deletions
diff --git a/spec/javascripts/notes/components/noteable_discussion_spec.js b/spec/javascripts/notes/components/noteable_discussion_spec.js index 4b4403689d9..76e9cd03d2d 100644 --- a/spec/javascripts/notes/components/noteable_discussion_spec.js +++ b/spec/javascripts/notes/components/noteable_discussion_spec.js @@ -80,43 +80,6 @@ describe('noteable_discussion component', () => { }); describe('computed', () => { - describe('hasMultipleUnresolvedDiscussions', () => { - it('is false if there are no unresolved discussions', done => { - spyOnProperty(vm, 'unresolvedDiscussions').and.returnValue([]); - - Vue.nextTick() - .then(() => { - expect(vm.hasMultipleUnresolvedDiscussions).toBe(false); - }) - .then(done) - .catch(done.fail); - }); - - it('is false if there is one unresolved discussion', done => { - spyOnProperty(vm, 'unresolvedDiscussions').and.returnValue([discussionMock]); - - Vue.nextTick() - .then(() => { - expect(vm.hasMultipleUnresolvedDiscussions).toBe(false); - }) - .then(done) - .catch(done.fail); - }); - - it('is true if there are two unresolved discussions', done => { - const discussion = getJSONFixture(discussionWithTwoUnresolvedNotes)[0]; - discussion.notes[0].resolved = false; - vm.$store.dispatch('setInitialNotes', [discussion, discussion]); - - Vue.nextTick() - .then(() => { - expect(vm.hasMultipleUnresolvedDiscussions).toBe(true); - }) - .then(done) - .catch(done.fail); - }); - }); - describe('isRepliesCollapsed', () => { it('should return false for diff discussions', done => { const diffDiscussion = getJSONFixture(diffDiscussionFixture)[0]; |