Redirect to show clinic page if appointment cannot proceed

Author: swebberuk

manage_breast_screening/mammograms/views/appointment_workflow_views.py

@@ -9,7 +9,7 @@
 from django.http import Http404, HttpResponse, StreamingHttpResponse
 from django.shortcuts import redirect
 from django.template.loader import render_to_string
-from django.urls import reverse, reverse_lazy
+from django.urls import reverse
 from django.views import View
 from django.views.decorators.http import require_http_methods
 from django.views.generic import FormView, TemplateView
@@ -159,7 +159,11 @@ def form_valid(self, form):
 class AppointmentCannotGoAhead(InProgressAppointmentMixin, FormView):
     template_name = "mammograms/appointment_cannot_go_ahead.jinja"
     form_class = AppointmentCannotGoAheadForm
-    success_url = reverse_lazy("clinics:list_clinics")
+
+    def get_success_url(self):
+        return reverse(
+            "clinics:show_clinic", kwargs={"pk": self.appointment.clinic_slot.clinic.pk}
+        )
 
     def get_context_data(self, **kwargs):
         context = super().get_context_data(**kwargs)

manage_breast_screening/tests/system/clinical/test_cannot_go_ahead_form.py

@@ -97,7 +97,11 @@ def when_i_fill_in_other_details(self):
         self.page.locator("#id_other_details").fill("Explain other choice")
 
     def then_i_see_the_clinics_page(self):
-        expect(self.page).to_have_url(re.compile(reverse("clinics:list_clinics")))
+        path = reverse(
+            "clinics:show_clinic",
+            kwargs={"pk": self.appointment.clinic_slot.clinic.pk},
+        )
+        expect(self.page).to_have_url(re.compile(path))
 
     def and_the_appointment_is_updated(self):
         self.appointment.refresh_from_db()