write tests

lib/services/credentials/credential.dao.js

@@ -156,7 +156,7 @@ dao.removeAllCredentials = function (id) {
   const awaitAllPromises = credentialTypes.map(type => {
     const authKey = buildIdKey(type, id);
 
-    if (type === 'key-auth' || 'jwt') {
+    if (type === 'key-auth' || type === 'jwt') {
       const promises = [];
 
       // id in this call is actually consumerId, so we need to get all referenced keyIds

test/services/users.test.js

@@ -3,6 +3,7 @@ const uuid = require('uuid');
 const redisConfig = require('../../lib/config').systemConfig.db.redis;
 const services = require('../../lib/services');
 const userService = services.user;
+const credentialService = services.credential;
 const db = require('../../lib/db');
 
 describe('User service tests', function () {
@@ -314,7 +315,7 @@ describe('User service tests', function () {
 
   describe('Delete user tests', function () {
     let user;
-    before(() => db.flushdb().then(() => {
+    beforeEach(() => db.flushdb().then(() => {
       user = createRandomUserObject();
       return userService.insert(user);
     }).then(newUser => {
@@ -337,6 +338,23 @@ describe('User service tests', function () {
           deleted.should.eql(false);
         });
     });
+
+    describe('should delete all the related credentials', () => {
+      const credentials = [];
+      before(() =>
+        Promise.all([
+          credentialService.insertScopes(['someScope']),
+          credentialService.insertCredential(user.id, 'jwt'),
+          credentialService.insertCredential(user.id, 'jwt')]
+        ).then(([scope, jwt1, jwt2]) =>
+          Promise.all([jwt1, jwt2].map(cred => {
+            credentials.push(cred);
+            return credentialService.addScopesToCredential(cred.id, 'jwt', ['someScope']);
+          }))
+        ));
+      it('should remove the user', () => should(userService.remove(user.id)).resolvedWith(true));
+      it('should remove the credentials', () => should(credentialService.getCredential(credentials[0].id, 'jwt')).resolvedWith(null));
+    });
   });
 });