Merge pull request #324 from cosmos/jordan/notify

don't need notify when change is shown in UI
luniehq · Jan 5, 2018 · 1b1b85c · 1b1b85c
2 parents 1736afd + d3c5310
commit 1b1b85c
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 3 deletions.
diff --git a/app/src/renderer/components/common/NiSessionSignIn.vue b/app/src/renderer/components/common/NiSessionSignIn.vue
@@ -58,7 +58,6 @@ export default {
         await this.$store.dispatch('testLogin', { password: this.fields.signInPassword, account: this.fields.signInName })
         this.$store.dispatch('signIn', { password: this.fields.signInPassword, account: this.fields.signInName })
         this.$store.commit('setModalSession', false)
-        this.$store.commit('notify', { title: 'Signed In', body: `You are now signed in to your Cosmos account.` })
       } catch (err) {
         this.$store.commit('notifyError', { title: 'Signing In Failed', body: err.message })
       }

diff --git a/test/unit/specs/NISessionSignIn.spec.js b/test/unit/specs/NISessionSignIn.spec.js
@@ -55,8 +55,6 @@ describe('NiSessionSignIn', () => {
       signInName: 'name'
     }})
     await wrapper.vm.onSubmit()
-    let commitCalls = store.commit.mock.calls.map(args => args[0])
-    expect(commitCalls).toContain('notify')
     expect(store.dispatch).toHaveBeenCalledWith('signIn', {
       password: '1234567890',
       account: 'name'