BAU: Remove unnecessary log

communitiesuk · Oct 18, 2023 · 037e866 · 037e866
1 parent 29d5761
commit 037e866
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 5 deletions.
diff --git a/models/account.py b/models/account.py
@@ -114,10 +114,6 @@ def update_account(
         if config.FLASK_ENV == "development" and not roles:
             account = get_account_data(email)
             roles = account.get("roles")
-        if not roles:
-            current_app.logger.error(
-                f"account id: {id} has not been assigned any roles"
-            )
         params = {
             "email_address": email,
             "azure_ad_subject_id": azure_ad_subject_id,

diff --git a/tests/test_sso.py b/tests/test_sso.py
@@ -160,7 +160,6 @@ def test_sso_get_token_logs_error_for_roleless_users(
     error_response = flask_test_client.get(endpoint)
 
     assert error_response.status_code == 302
-    assert "account id: usersso has not been assigned any roles" in caplog.text
 
 
 def test_sso_get_token_sets_expected_fsd_user_token_cookie_claims(