Merge pull request #32 from solid/delete-pod

Delete User from store

Author: kjetilk

package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "standard": "standard",
     "test": "npm run standard && npm run mocha",
-    "mocha": "nyc mocha --exit --timeout=20000 test/**/*.js",
+    "mocha": "nyc mocha",
     "preversion": "npm test",
     "postversion": "git push --follow-tags"
   },

src/user-store.js

@@ -194,6 +194,19 @@ class UserStore {
       })
       .then(() => this.backend.put('users', userKey, user))
   }
+  /**
+   * Permanently deletes the files of a user
+   *
+   * @param user {UserAccount}
+   *
+   * @return {Promise}
+   */
+
+  deleteUser (user) {
+    let userKey = UserStore.normalizeIdKey(user.id)
+
+    return this.backend.del('users', userKey)
+  }
 
   /**
    * Saves an "alias" user object, used for linking local account IDs to

test/integration/user-store-test.js

@@ -143,4 +143,35 @@ describe('UserStore (integration)', () => {
         })
     })
   })
+  describe('deleteUser()', () => {
+    it('deletes a previously saved user', () => {
+      let options = { path: dbPath, saltRounds: 2 }
+      let store = UserStore.from(options)
+      store.initCollections()
+
+      let user = {
+        id: 'alice.example.com',
+        email: '[email protected]'
+      }
+      let password = '12345'
+
+      return store.createUser(user, password)
+        .then(() => {
+          return store.findUser(user.id)
+        })
+        .then(foundUser => {
+          expect(foundUser.id).to.equal(user.id)
+          expect(foundUser.email).to.equal(user.email)
+        })
+        .then(() => {
+          return store.deleteUser(user)
+        })
+        .then(() => {
+          return store.findUser(user.id)
+        })
+        .then(foundUser => {
+          expect(foundUser).to.not.exist()
+        })
+    })
+  })
 })

test/mocha.opts

@@ -0,0 +1,3 @@
+--exit
+--recursive
+--timeout 20000

test/unit/user-store-test.js

@@ -212,4 +212,21 @@ describe('UserStore', () => {
         })
     })
   })
+
+  describe('deleteUser', () => {
+    let store
+
+    beforeEach(() => {
+      store = UserStore.from({ path: './db' })
+    })
+
+    it('should call backend.del with normalized user id', () => {
+      let userId = 'alice.solidtest.space/profile/card#me'
+
+      store.backend.del = sinon.stub()
+
+      store.deleteUser({ id: userId })
+      expect(store.backend.del).to.have.been.calledWith('users', UserStore.normalizeIdKey(userId))
+    })
+  })
 })