Improve error handling and fix test

Author: kjetilk

src/user-store.js

@@ -204,10 +204,12 @@ class UserStore {
 
   deleteUser (user) {
     let userKey = UserStore.normalizeIdKey(user.id)
-    var deletedEmail = Promise.reject(new Error('No email given'))
+    var deletedEmail
     if (user.email) {
       let emailKey = UserStore.normalizeEmailKey(user.email)
       deletedEmail = this.backend.del('users-by-email', emailKey)
+    } else {
+      deletedEmail = Promise.reject(new Error('No email given'))
     }
     let deletedUser = this.backend.del('users', userKey)
     return Promise.all([deletedEmail, deletedUser])

test/unit/user-store-test.js

@@ -220,13 +220,15 @@ describe('UserStore', () => {
       store = UserStore.from({ path: './db' })
     })
 
-    it('should call backend.del with normalized user id', () => {
+    it('should call backend.del with normalized user id and email', () => {
       let userId = 'alice.solidtest.space/profile/card#me'
+      let email = '[email protected]'
 
       store.backend.del = sinon.stub()
 
-      store.deleteUser({ id: userId })
+      store.deleteUser({ id: userId, email: email })
       expect(store.backend.del).to.have.been.calledWith('users', UserStore.normalizeIdKey(userId))
+      expect(store.backend.del).to.have.been.calledWith('users-by-email', UserStore.normalizeEmailKey(email))
     })
   })
 })