darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-02 17:31:14 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #840 from GSA/notify-admin-1277

Browse Source 
fix login.gov to use user uuid instead of email (notify-admin-1277)
This commit is contained in:
Carlo Costino
2024-03-11 11:53:42 -04:00
committed by GitHub
parent c84d01b454 7c95211649
commit 8a6cc76d15
5 changed files with 65 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/app/dao/test_users_dao.py
View File

@@ -15,6 +15,7 @@ from app.dao.users_dao import (
dao_archive_user,
delete_codes_older_created_more_than_a_day_ago,
delete_model_user,
get_login_gov_user,
get_user_by_email,
get_user_by_id,
increment_failed_login_count,
@@ -110,6 +111,12 @@ def test_get_user_by_email(sample_user):
assert sample_user == user_from_db
def test_get_login_gov_user(sample_user):
user_from_db = get_login_gov_user("fake_login_gov_uuid", sample_user.email_address)
assert sample_user.email_address == user_from_db.email_address
assert user_from_db.login_uuid is not None
def test_get_user_by_email_is_case_insensitive(sample_user):
email = sample_user.email_address
user_from_db = get_user_by_email(email.upper())