Switch from BMI to Book an appointment

Author: jamiefalcus

features/check_need_appointment.feature

@@ -18,7 +18,7 @@ Feature: Check if you need an appointment page
     When I go to "/check-if-you-need-an-appointment"
     Then I see a back link to "/date-of-birth"
     When I check "Yes, one or more of these things applies to me and I need a face-to-face appointment" and submit
-    Then I am on "/bmi-exit"
+    Then I am on "/call-us-to-book-an-appointment"
 
   Scenario: Navigating backwards and forwards
     Given I am logged in

…creening/questions/jinja2/bmi_exit.jinja …estions/jinja2/book_an_appointment.jinjalung_cancer_screening/questions/jinja2/bmi_exit.jinja renamed to lung_cancer_screening/questions/jinja2/book_an_appointment.jinja lung_cancer_screening/questions/jinja2/bmi_exit.jinja renamed to lung_cancer_screening/questions/jinja2/book_an_appointment.jinja

@@ -133,14 +133,14 @@ def test_post_redirects_to_the_next_page(self):
         # Assuming it redirects to the next question page - adjust as needed
         self.assertRedirects(response, reverse("questions:height"))
 
-    def test_post_redirects_to_bmi_exit_page(self):
+    def test_post_redirects_to_book_an_appointment_page(self):
         response = self.client.post(
             reverse("questions:check_need_appointment"),
             {"value": True}
         )
 
         # Assuming it redirects to the next question page - adjust as needed
-        self.assertRedirects(response, reverse("questions:bmi_exit"))
+        self.assertRedirects(response, reverse("questions:book_an_appointment"))
 
     def test_post_responds_with_422_if_the_response_fails_to_create(self):
         response = self.client.post(

lung_cancer_screening/questions/tests/unit/views/test_check_need_appointment.py

@@ -19,7 +19,7 @@
 
 from .views.age_range_exit import AgeRangeExitView
 from .views.asbestos_exposure import AsbestosExposureView
-from .views.bmi_exit import BmiExitView
+from .views.book_an_appointment import BookAnAppointmentExitView
 from .views.cancer_diagnosis import CancerDiagnosisView
 from .views.check_need_appointment import CheckNeedAppointmentView
 from .views.date_of_birth import DateOfBirthView
@@ -40,24 +40,24 @@
 
 urlpatterns = [
     path('', RedirectView.as_view(url='/start'), name='root'),
-    path('start', StartView.as_view(), name='start'),
-    path('have-you-ever-smoked', HaveYouEverSmokedView.as_view(), name='have_you_ever_smoked'),
-    path('date-of-birth', DateOfBirthView.as_view(), name='date_of_birth'),
-    path('check-if-you-need-an-appointment', CheckNeedAppointmentView.as_view(), name='check_need_appointment'),
-    path('height', HeightView.as_view(), name='height'),
-    path('weight', WeightView.as_view(), name='weight'),
-    path('sex-at-birth', SexAtBirthView.as_view(), name='sex_at_birth'),
-    path('gender', GenderView.as_view(), name='gender'),
-    path('ethnicity', EthnicityView.as_view(), name='ethnicity'),
-    path('education', EducationView.as_view(), name='education'),
-    path('respiratory-conditions', RespiratoryConditionsView.as_view(), name='respiratory_conditions'),
+    path('age-range-exit', AgeRangeExitView.as_view(), name='age_range_exit'),
     path('asbestos-exposure', AsbestosExposureView.as_view(), name='asbestos_exposure'),
+    path('call-us-to-book-an-appointment', BookAnAppointmentExitView.as_view(), name='book_an_appointment'),
     path('cancer-diagnosis', CancerDiagnosisView.as_view(), name='cancer_diagnosis'),
+    path('check-if-you-need-an-appointment', CheckNeedAppointmentView.as_view(), name='check_need_appointment'),
+    path('date-of-birth', DateOfBirthView.as_view(), name='date_of_birth'),
+    path('education', EducationView.as_view(), name='education'),
+    path('ethnicity', EthnicityView.as_view(), name='ethnicity'),
     path('family-history-lung-cancer', FamilyHistoryLungCancerView.as_view(), name='family_history_lung_cancer'),
+    path('gender', GenderView.as_view(), name='gender'),
+    path('have-you-ever-smoked', HaveYouEverSmokedView.as_view(), name='have_you_ever_smoked'),
+    path('height', HeightView.as_view(), name='height'),
+    path('non-smoker-exit', NonSmokerExitView.as_view(), name='non_smoker_exit'),
     path('relatives-age-when-diagnosed', RelativesAgeWhenDiagnosedView.as_view(), name='relatives_age_when_diagnosed'),
+    path('respiratory-conditions', RespiratoryConditionsView.as_view(), name='respiratory_conditions'),
     path('responses', ResponsesView.as_view(), name='responses'),
-    path('age-range-exit', AgeRangeExitView.as_view(), name='age_range_exit'),
-    path('bmi-exit', BmiExitView.as_view(), name='bmi_exit'),
-    path('non-smoker-exit', NonSmokerExitView.as_view(), name='non_smoker_exit'),
+    path('sex-at-birth', SexAtBirthView.as_view(), name='sex_at_birth'),
+    path('start', StartView.as_view(), name='start'),
+    path('weight', WeightView.as_view(), name='weight'),
     path('your-results', YourResultsView.as_view(), name='your_results'),
 ]

lung_cancer_screening/questions/urls.py

@@ -5,9 +5,9 @@
 from .mixins.ensure_response_set import EnsureResponseSet
 
 
-class BmiExitView(LoginRequiredMixin, EnsureResponseSet, View):
+class BookAnAppointmentExitView(LoginRequiredMixin, EnsureResponseSet, View):
     def get(self, request):
         return render(
             request,
-            "bmi_exit.jinja"
+            "book_an_appointment.jinja"
         )

…er_screening/questions/views/bmi_exit.py …g/questions/views/book_an_appointment.pylung_cancer_screening/questions/views/bmi_exit.py renamed to lung_cancer_screening/questions/views/book_an_appointment.py lung_cancer_screening/questions/views/bmi_exit.py renamed to lung_cancer_screening/questions/views/book_an_appointment.py

@@ -29,7 +29,7 @@ def post(self, request):
             response.save()
 
             if form.cleaned_data["value"] :
-                return redirect(reverse("questions:bmi_exit"))
+                return redirect(reverse("questions:book_an_appointment"))
             else :
                 return redirect(reverse("questions:height"))
         else: