Skip to content

Commit 35d16ca

Browse files
committed
chore(in-person): simplify redirect route logic
the get_target_route_name method doesn't do much, so we remove it and instead we just include the logic in get_redirect_url. this simplifies the code a bit more.
1 parent bd02ceb commit 35d16ca

4 files changed

Lines changed: 6 additions & 11 deletions

File tree

benefits/enrollment/views.py

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,8 @@ class IndexView(AgencySessionRequiredMixin, EligibleSessionRequiredMixin, Redire
2828

2929
route_origin = routes.ENROLLMENT_INDEX
3030

31-
def get_target_route_name(self):
32-
return self.agency.enrollment_index_route
33-
3431
def get_redirect_url(self, *args, **kwargs):
35-
route_name = self.get_target_route_name()
32+
route_name = self.agency.enrollment_index_route
3633
return reverse(route_name)
3734

3835
def get(self, request, *args, **kwargs):

benefits/in_person/views.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,9 @@ class EnrollmentView(IndexView):
105105

106106
route_origin = routes.IN_PERSON_ENROLLMENT
107107

108-
def get_target_route_name(self):
109-
return self.agency.in_person_enrollment_index_route
108+
def get_redirect_url(self, *args, **kwargs):
109+
route_name = self.agency.in_person_enrollment_index_route
110+
return reverse(route_name)
110111

111112

112113
def enrollment(request):

tests/pytest/enrollment/test_views.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,6 @@ def view(self, app_request, model_LittlepayConfig):
3838
v.agency = model_LittlepayConfig.transit_agency
3939
return v
4040

41-
def test_get_target_route_name(self, view):
42-
assert view.get_target_route_name() == view.agency.enrollment_index_route
43-
4441
def test_get_redirect_url(self, view):
4542

4643
assert view.get_redirect_url() == reverse(view.agency.enrollment_index_route)

tests/pytest/in_person/test_views.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -285,8 +285,8 @@ def view(self, app_request, mocked_session_agency_littlepay):
285285
v.agency = mocked_session_agency_littlepay(app_request)
286286
return v
287287

288-
def test_get_target_route_name_for_littlepay(self, view):
289-
assert view.get_target_route_name() == routes.IN_PERSON_ENROLLMENT_LITTLEPAY_INDEX
288+
def test_get_redirect_url_for_littlepay(self, view):
289+
assert view.get_redirect_url() == reverse(view.agency.in_person_enrollment_index_route)
290290

291291

292292
@pytest.mark.django_db

0 commit comments

Comments
 (0)