fix: 🐛 Fixed api.db.spec test

Author: kostysh

src/node/api/db.ts

@@ -13,7 +13,7 @@ export interface User {
   /** The hashed password of the user */
   hashedPassword: string;
   /** Flag which indicate that the use is admin */
-  isAdmin: boolean;
+  isAdmin?: boolean;
   /** The optional JSON Web Token of the user */
   jwt?: string;
 }

test/node.api.db.spec.ts

@@ -1,36 +1,41 @@
 import { describe, beforeAll, it, expect } from './setup.js';
 import { createInitializer } from '../src/storage/memory.js';
-import { UsersDb, UsersDbOptions, User } from '../src/node/api/db.js';
+import {
+  UsersDb,
+  UsersDbOptions,
+  User,
+  comparePassword,
+} from '../src/node/api/db.js';
 
 describe('Node.API.Db', () => {
+  const password = 'password';
   let userDb: UsersDb;
   let testUser: User;
 
   beforeAll(async () => {
     const options: UsersDbOptions = {
       storage: await createInitializer()(),
       prefix: 'test',
-      salt: 'salt',
     };
     userDb = new UsersDb(options);
     testUser = {
       login: 'testUser',
-      hashedPassword: UsersDb.hashPassword('password', options.salt),
+      hashedPassword: await UsersDb.hashPassword('password'),
     };
   });
 
   describe('#add', () => {
     it('should add a new user', async () => {
-      await userDb.add(testUser.login, 'password');
+      await userDb.add(testUser.login, password);
       const user = await userDb.get(testUser.login);
       expect(user.login).to.be.eq(testUser.login);
-      expect(user.hashedPassword).to.be.eq(testUser.hashedPassword);
+      expect(await comparePassword(password, user.hashedPassword)).to.be.true;
     });
 
     it('should throw an error when trying to add an existing user', async () => {
       let error = null;
       try {
-        await userDb.add(testUser.login, 'password');
+        await userDb.add(testUser.login, password);
       } catch (err) {
         error = err;
       }
@@ -45,7 +50,7 @@ describe('Node.API.Db', () => {
     it('should get a user', async () => {
       const user = await userDb.get(testUser.login);
       expect(user.login).to.be.eq(testUser.login);
-      expect(user.hashedPassword).to.be.eq(testUser.hashedPassword);
+      expect(await comparePassword(password, user.hashedPassword)).to.be.true;
     });
 
     it('should throw an error when trying to get a non-existent user', async () => {