darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2025-12-11 23:53:52 -05:00
Code Issues Packages Projects Releases Wiki Activity

write test

Browse Source
This commit is contained in:
Kenneth Kehl
2024-07-26 09:49:12 -07:00
parent 34d727748b
commit e884f31fa0
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/notify_client/__init__.py
View File

@@ -57,7 +57,7 @@ class NotifyAdminAPIClient(BaseAPIClient):
abort(403) abort(403)
def is_calling_signin_url(self, arg): def is_calling_signin_url(self, arg):
return arg[0].startswith("/user") return arg.startswith("('/user")
def check_inactive_user(self, *args): def check_inactive_user(self, *args):
still_signing_in = False still_signing_in = False

5
tests/app/notify_client/test_notify_admin_api_client.py
View File

@@ -45,11 +45,12 @@ def test_active_service_can_be_modified(notify_admin, method, user, service):
("arg", "expected_result"), ("arg", "expected_result"),
[ [
( (
("/user/c5f8a5c9-56d5-4fa9-8c30-3449ae10c072/verify/code",), "('/user/c5f8a5c9-56d5-4fa9-8c30-3449ae10c072/verify/code',)",
True, True,
), ),
("('/user/get-login-gov-user',)", True),
( (
("/service/blahblahblah",), "('/service/blahblahblah',)",
False, False,
), ),
], ],