Skip to content

Commit

Permalink
chore(oauth): normalize URL naming convention
Browse files Browse the repository at this point in the history
kebab-case is used elsewhere
  • Loading branch information
thekaveman committed Jul 17, 2024
1 parent 6550a10 commit ac5b80f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion benefits/oauth/urls.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,6 @@
path("authorize", views.authorize, name="authorize"),
path("cancel", views.cancel, name="cancel"),
path("logout", views.logout, name="logout"),
path("post_logout", views.post_logout, name="post_logout"),
path("post-logout", views.post_logout, name="post-logout"),
path("error", views.system_error, name="system-error"),
]
2 changes: 1 addition & 1 deletion benefits/oauth/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@
ROUTE_AUTH = "oauth:authorize"
ROUTE_CONFIRM = "eligibility:confirm"
ROUTE_UNVERIFIED = "eligibility:unverified"
ROUTE_POST_LOGOUT = "oauth:post_logout"
ROUTE_POST_LOGOUT = "oauth:post-logout"

TEMPLATE_SYSTEM_ERROR = "oauth/system_error.html"

Expand Down
2 changes: 1 addition & 1 deletion tests/pytest/oauth/test_views.py
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,7 @@ def test_logout(mocker, mocked_oauth_client_or_error_redirect__client, mocked_an

result = logout(app_request)

mocked_redirect.assert_called_with(mocked_oauth_client, token, "https://testserver/oauth/post_logout")
mocked_redirect.assert_called_with(mocked_oauth_client, token, "https://testserver/oauth/post-logout")
mocked_analytics_module.started_sign_out.assert_called_once()
assert result.status_code == 200
assert message in str(result.content)
Expand Down

0 comments on commit ac5b80f

Please sign in to comment.