Merge pull request #44 from Aclify/implement-remove-user

removeUser() implementation

__tests__/tests.ts

@@ -818,5 +818,24 @@ import { Acl, HttpError, MemoryStore, MongoDBStore, RedisStore } from '../src';
         expect(res2.res1.sort()).toEqual(['perm1', 'perm2', 'perm3'].sort()); // tslint:disable-line no-unsafe-any
       });
     });
+
+    describe('Removing a user', () => {
+      it('Remove a user', async () => {
+        const guest = await acl.allow('member', 'panel', 'edit');
+        expect(guest).toBeUndefined();
+
+        const addUserRoles = await acl.addUserRoles('dimitri', 'member');
+        expect(addUserRoles).toBeUndefined();
+
+        const isAllowed = await acl.isAllowed('dimitri', 'panel', 'edit');
+        expect(isAllowed).toBeTruthy();
+
+        const removeUser = await acl.removeUser('dimitri');
+        expect(removeUser).toBeUndefined();
+
+        const isAllowedAfterRemoveUser = await acl.isAllowed('dimitri', 'panel', 'edit');
+        expect(isAllowedAfterRemoveUser).toBeFalsy();
+      });
+    });
   });
 });

src/classes/acl.ts

@@ -395,6 +395,19 @@ export class Acl extends Common {
     return this.store.end();
   }
 
+  /**
+   * @description Removes user with his associated roles.
+   * @param userId
+   * @return Promise<void>
+   */
+  public async removeUser(userId: string | number): Promise<void> {
+    const userRoles = await this.userRoles(userId);
+    await this.removeUserRoles(userId, userRoles);
+    this.store.begin();
+    await this.store.del(this.options.buckets.users, userId);
+    await this.store.end();
+  }
+
   /**
    * @description Remove roles from a given user.
    * @param userId

src/interfaces/IStore.ts

@@ -56,7 +56,7 @@ export interface IStore {
    * @param keys
    * @return Promise<void>
    */
-  del(bucket: string, keys: string | string[]): Promise<void>;
+  del(bucket: string, keys: number | string | string[]): Promise<void>;
 
   /**
    * @description Removes values from a given key inside a bucket.