Refactor GillickAssessmentsController

Because we now don't really have the concept of editing a Gillick
assessment (they're always created new) then we can reduce duplication
and go down to one set of controller actions.

Author: thomasleese

app/components/app_patient_page_component.html.erb

@@ -53,7 +53,7 @@
     <% elsif gillick_assessment_can_be_recorded? %>
       <p class="nhsuk-body">
         <%= govuk_button_link_to "Assess Gillick competence",
-                                 new_session_patient_gillick_assessment_path(
+                                 edit_session_patient_gillick_assessment_path(
                                    session,
                                    patient,
                                    section: @section,

app/controllers/gillick_assessments_controller.rb

@@ -4,19 +4,9 @@ class GillickAssessmentsController < ApplicationController
   before_action :set_patient
   before_action :set_session
   before_action :set_patient_session
+  before_action :set_is_first_assessment
   before_action :set_gillick_assessment
 
-  def new
-  end
-
-  def create
-    if @gillick_assessment.update(gillick_assessment_params)
-      redirect_to session_patient_path(id: @patient.id)
-    else
-      render :new, status: :unprocessable_entity
-    end
-  end
-
   def edit
   end
 
@@ -45,6 +35,10 @@ def set_patient_session
       policy_scope(PatientSession).find_by(session: @session, patient: @patient)
   end
 
+  def set_is_first_assessment
+    @is_first_assessment = @patient_session.gillick_assessments.empty?
+  end
+
   def set_gillick_assessment
     @gillick_assessment =
       authorize @patient_session.latest_gillick_assessment&.dup ||

app/controllers/patient_sessions_controller.rb

@@ -24,7 +24,7 @@ def set_patient_session
       policy_scope(PatientSession)
         .includes(:patient, :vaccination_records)
         .eager_load(:session)
-        .preload(:consents, :triages)
+        .preload(:consents, :gillick_assessments, :triages)
         .find_by!(
           session: {
             slug: params.fetch(:session_slug)

app/views/gillick_assessments/_form_fields.html.erb

@@ -5,7 +5,7 @@
       ) %>
 <% end %>
 
-<% page_title = "Edit Gillick competence" %>
+<% page_title = @is_first_assessment ? "Assess Gillick competence" : "Edit Gillick competence" %>
 
 <%= h1 page_title: do %>
   <span class="nhsuk-caption-l">
@@ -14,12 +14,54 @@
   <%= page_title %>
 <% end %>
 
-<%= render "guidance" %>
+<%= govuk_inset_text do %>
+  <p class="nhsuk-body">
+    Before you make your assessment, you should give
+    <%= @patient.given_name %> a chance to ask questions.
+  </p>
+<% end %>
 
-<%= form_with model: @gillick_assessment, url: session_patient_gillick_assessment_path do |f| %>
+<%= form_with model: @gillick_assessment, url: session_patient_gillick_assessment_path, method: :put do |f| %>
   <% content_for(:before_content) { f.govuk_error_summary } %>
 
-  <%= render "form_fields", f: %>
+  <% options = [OpenStruct.new(value: true, label: "Yes"), OpenStruct.new(value: false, label: "No")] %>
+
+  <%= f.govuk_collection_radio_buttons :knows_vaccination,
+                                       options,
+                                       :value,
+                                       :label,
+                                       inline: true,
+                                       legend: { text: "The child knows which vaccination they will have", size: "s" } %>
+
+  <%= f.govuk_collection_radio_buttons :knows_disease,
+                                       options,
+                                       :value,
+                                       :label,
+                                       inline: true,
+                                       legend: { text: "The child knows which disease the vaccination protects against", size: "s" } %>
+
+  <%= f.govuk_collection_radio_buttons :knows_consequences,
+                                       options,
+                                       :value,
+                                       :label,
+                                       inline: true,
+                                       legend: { text: "The child knows what could happen if they got the disease", size: "s" } %>
+
+  <%= f.govuk_collection_radio_buttons :knows_delivery,
+                                       options,
+                                       :value,
+                                       :label,
+                                       inline: true,
+                                       legend: { text: "The child knows how the injection will be given", size: "s" } %>
+
+  <%= f.govuk_collection_radio_buttons :knows_side_effects,
+                                       options,
+                                       :value,
+                                       :label,
+                                       inline: true,
+                                       legend: { text: "The child knows which side effects they might experience", size: "s" } %>
+
+  <%= f.govuk_text_area :notes, label: { text: "Assessment notes (optional)" } %>
 
-  <%= f.govuk_submit "Update your assessment" %>
+  <%= f.govuk_submit @is_first_assessment ? "Complete your assessment" : "Update your assessment" %>
 <% end %>

app/views/gillick_assessments/_guidance.html.erb

@@ -312,9 +312,7 @@
           end
         end
 
-        resource :gillick_assessment,
-                 path: "gillick",
-                 only: %i[new create edit update]
+        resource :gillick_assessment, path: "gillick", only: %i[edit update]
         resource :triages, only: %i[new create]
         resource :vaccinations, only: %i[create] do
           member do