From a35d16f9a191afbb31e2c185e87e5eec5e23122f Mon Sep 17 00:00:00 2001 From: Frederick Muriuki Muriithi Date: Wed, 8 Mar 2023 11:42:04 +0300 Subject: auth: users: Use the same basic functions for password hashing To avoid repeating the same thing in multiple places, leading to errors and breakages, reuse the same basic functions for password hashing. --- tests/unit/auth/fixtures/user_fixtures.py | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) (limited to 'tests/unit') diff --git a/tests/unit/auth/fixtures/user_fixtures.py b/tests/unit/auth/fixtures/user_fixtures.py index 4e42abe..d248f54 100644 --- a/tests/unit/auth/fixtures/user_fixtures.py +++ b/tests/unit/auth/fixtures/user_fixtures.py @@ -2,10 +2,9 @@ import uuid import pytest -import bcrypt from gn3.auth import db -from gn3.auth.authentication.users import User +from gn3.auth.authentication.users import User, hash_password TEST_USERS = ( User(uuid.UUID("ecb52977-3004-469e-9428-2a1856725c7f"), "group@lead.er", @@ -50,9 +49,8 @@ def fxtr_users_with_passwords(fxtr_users): # pylint: disable=[redefined-outer-na """Fixture: add passwords to the users""" conn, users = fxtr_users user_passwords_params = tuple( - (str(user.user_id), bcrypt.hashpw( - f"password_for_user_{idx:03}".encode("utf8"), - bcrypt.gensalt())) + (str(user.user_id), hash_password( + f"password_for_user_{idx:03}".encode("utf8"))) for idx, user in enumerate(users, start=1)) with db.cursor(conn) as cursor: -- cgit v1.2.3