diff options
author | Phil Hughes <me@iamphill.com> | 2017-05-24 18:14:23 +0000 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2017-05-24 18:14:23 +0000 |
commit | 30462b8814232724ab4875835225f992790eba95 (patch) | |
tree | 37b2f3ee2a83cf357c7b1945165d8f49a025113f /spec | |
parent | 71b80f96cbf2c11b7f6274133c6aa70244315fa2 (diff) | |
parent | 6b8fac9de471646b36a822e2b18e1b3a97965288 (diff) | |
download | gitlab-ce-30462b8814232724ab4875835225f992790eba95.tar.gz |
Merge branch '32170-assignees-spinner' into 'master'
show loading indicator while waiting for assignees first fetch
Closes #32170
See merge request !11434
Diffstat (limited to 'spec')
-rw-r--r-- | spec/javascripts/sidebar/sidebar_assignees_spec.js | 12 | ||||
-rw-r--r-- | spec/javascripts/sidebar/sidebar_store_spec.js | 5 |
2 files changed, 17 insertions, 0 deletions
diff --git a/spec/javascripts/sidebar/sidebar_assignees_spec.js b/spec/javascripts/sidebar/sidebar_assignees_spec.js index 865951b2ad7..929ba75e67d 100644 --- a/spec/javascripts/sidebar/sidebar_assignees_spec.js +++ b/spec/javascripts/sidebar/sidebar_assignees_spec.js @@ -43,4 +43,16 @@ describe('sidebar assignees', () => { expect(SidebarMediator.prototype.assignYourself).toHaveBeenCalled(); expect(this.mediator.store.assignees.length).toEqual(1); }); + + it('hides assignees until fetched', (done) => { + component = new SidebarAssigneeComponent().$mount(this.sidebarAssigneesEl); + const currentAssignee = this.sidebarAssigneesEl.querySelector('.value'); + expect(currentAssignee).toBe(null); + + component.store.isFetching.assignees = false; + Vue.nextTick(() => { + expect(component.$el.querySelector('.value')).toBeVisible(); + done(); + }); + }); }); diff --git a/spec/javascripts/sidebar/sidebar_store_spec.js b/spec/javascripts/sidebar/sidebar_store_spec.js index 29facf483b5..b3fa156eb64 100644 --- a/spec/javascripts/sidebar/sidebar_store_spec.js +++ b/spec/javascripts/sidebar/sidebar_store_spec.js @@ -35,6 +35,10 @@ describe('Sidebar store', () => { SidebarStore.singleton = null; }); + it('has default isFetching values', () => { + expect(this.store.isFetching.assignees).toBe(true); + }); + it('adds a new assignee', () => { this.store.addAssignee(assignee); expect(this.store.assignees.length).toEqual(1); @@ -67,6 +71,7 @@ describe('Sidebar store', () => { }; this.store.setAssigneeData(users); + expect(this.store.isFetching.assignees).toBe(false); expect(this.store.assignees.length).toEqual(3); }); |