diff --git a/app/main/views/sign_in.py b/app/main/views/sign_in.py index 94bcc9a9a..5e6899eae 100644 --- a/app/main/views/sign_in.py +++ b/app/main/views/sign_in.py @@ -26,7 +26,7 @@ from app.utils import hide_from_search_engines from app.utils.login import is_safe_redirect_url -def _reformulate_keystring(orig): +def _reformat_keystring(orig): new_keystring = orig.replace("-----BEGIN PRIVATE KEY-----", "") new_keystring = new_keystring.replace("-----END PRIVATE KEY-----", "") new_keystring = new_keystring.strip() @@ -42,7 +42,7 @@ def _get_access_token(code, state): access_token_url = os.getenv("LOGIN_DOT_GOV_ACCESS_TOKEN_URL") keystring = os.getenv("LOGIN_PEM") if " " in keystring: - keystring = _reformulate_keystring(keystring) + keystring = _reformat_keystring(keystring) payload = { "iss": client_id, diff --git a/tests/app/main/views/test_sign_in.py b/tests/app/main/views/test_sign_in.py index 50ac0b226..345a1bd94 100644 --- a/tests/app/main/views/test_sign_in.py +++ b/tests/app/main/views/test_sign_in.py @@ -3,7 +3,7 @@ import uuid import pytest from flask import url_for -from app.main.views.sign_in import _reformulate_keystring +from app.main.views.sign_in import _reformat_keystring from app.models.user import User from tests.conftest import SERVICE_ONE_ID, normalize_spaces @@ -40,14 +40,14 @@ def test_render_sign_in_template_with_next_link_for_password_reset(client_reques ) -def test_reformulate_keystring(): +def test_reformat_keystring(): orig = "-----BEGIN PRIVATE KEY----- blahblahblah -----END PRIVATE KEY-----" expected = """-----BEGIN PRIVATE KEY----- blahblahblah -----END PRIVATE KEY----- """ - reformulated = _reformulate_keystring(orig) - assert reformulated == expected + reformatted = _reformat_keystring(orig) + assert reformatted == expected def test_sign_in_explains_session_timeout(client_request):