PR required fixes

linuxserver · Apr 21, 2021 · f1eacb4 · f1eacb4
1 parent 44acdb7
commit f1eacb4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/routes/users.js b/routes/users.js
@@ -92,8 +92,6 @@ router.post(
     let pass = null
     if (req.body.password) {
       pass = bcrypt.hashSync(req.body.password, 10)
-    } else {
-      pass = null
     }
 
     const user = await User.create({

diff --git a/tests/tests.test.js b/tests/tests.test.js
@@ -1,4 +1,5 @@
 const { User, Item } = require('../models/index')
+const bcrypt = require('bcrypt')
 
 beforeAll(async () => {
   await User.destroy({ where: {} })
@@ -8,7 +9,7 @@ test('Creates a user', async () => {
   const user = await User.create({
     username: 'admin',
     email: '[email protected]',
-    password: 'admin'
+    password: bcrypt.hashSync('admin', 10)
   })
 
   expect(user.username).toBe('admin')