Skip to content

Commit

Permalink
switch dummy emails to .test domain
Browse files Browse the repository at this point in the history
  • Loading branch information
jonathan-owens-gds committed Sep 19, 2024
1 parent b36d4b8 commit b41b063
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions tests/app/user/test_rest.py
Original file line number Diff line number Diff line change
Expand Up @@ -344,10 +344,10 @@ def test_get_user_by_email_bad_url_returns_404(admin_request, sample_user):


def test_fetch_user_by_email(admin_request, mocker, notify_db_session):
user = create_user(email="foo@bar.com")
user = create_user(email="foo@bar.test")

create_user(email="foo@bar.com.other_email")
create_user(email="other_email.foo@bar.com")
create_user(email="foo@bar.other_email.test")
create_user(email="other_email.foo@bar.test")

mocker.patch("app.user.rest._pending_registration", return_value=False)
add_failed_login_for_requester = mocker.patch("app.user.rest.add_failed_login_for_requester")
Expand All @@ -360,34 +360,34 @@ def test_fetch_user_by_email(admin_request, mocker, notify_db_session):


def test_fetch_user_by_email_not_found_returns_404(admin_request, mocker, notify_db_session):
create_user(email="foo@bar.com.other_email")
create_user(email="foo@bar.other_email.test")

mocker.patch("app.failed_logins.rest.add_failed_login_for_requester")

resp = admin_request.post("user.fetch_user_by_email", _data={"email": "doesnt@exist.com"}, _expected_status=404)
resp = admin_request.post("user.fetch_user_by_email", _data={"email": "doesnt@exist.test"}, _expected_status=404)
assert resp["result"] == "error"
assert resp["message"] == "No result found"


def test_email_not_found_for_active_user_logs_failed_login(admin_request, mocker):
create_user(email="foo@bar.com.other_email")
create_user(email="foo@bar.other_email.test")

mocker.patch("app.user.rest._pending_registration", return_value=False)
add_failed_login_for_requester = mocker.patch("app.user.rest.add_failed_login_for_requester")

resp = admin_request.post("user.fetch_user_by_email", _data={"email": "doesnt@exist.com"}, _expected_status=404)
resp = admin_request.post("user.fetch_user_by_email", _data={"email": "doesnt@exist.test"}, _expected_status=404)

assert resp["result"] == "error"
add_failed_login_for_requester.assert_called_once()


def test_email_not_found_for_registering_user_does_not_log_failed_login(admin_request, mocker):
create_user(email="foo@bar.com.other_email")
create_user(email="foo@bar.other_email.test")

mocker.patch("app.user.rest._pending_registration", return_value=True)
add_failed_login_for_requester = mocker.patch("app.user.rest.add_failed_login_for_requester")

resp = admin_request.post("user.fetch_user_by_email", _data={"email": "doesnt@exist.com"}, _expected_status=404)
resp = admin_request.post("user.fetch_user_by_email", _data={"email": "doesnt@exist.test"}, _expected_status=404)

assert resp["result"] == "error"
add_failed_login_for_requester.assert_not_called()
Expand Down

0 comments on commit b41b063

Please sign in to comment.