Rename vaccination record outcomes

This renames the outcomes to match the naming convention we're now using
across the service in the various statuses.

Jira-Issue: MAV-2581

Author: thomasleese

app/lib/fhir_mapper/vaccination_record.rb

@@ -136,8 +136,7 @@ def fhir_status
       case outcome
       when "administered"
         "completed"
-      when "refused", "not_well", "contraindications", "already_had",
-           "absent_from_school", "absent_from_session"
+      when "refused", "unwell", "contraindicated", "already_had", "absent"
         "not-done"
       else
         raise ArgumentError, "Unknown outcome: #{outcome}"

app/lib/mavis_cli/stats/vaccinations.rb

@@ -18,10 +18,10 @@ class Vaccinations < Dry::CLI::Command
       option :outcome,
              values: %w[
                administered
-               not_well
+               unwell
                already_had
-               contraindications
-               absent_from_session
+               contraindicated
+               absent
                refused
              ],
              desc: "Filter by specific outcome"

app/lib/status_generator/vaccination.rb

@@ -117,20 +117,19 @@ def status_should_be_due?
   def status_should_be_eligible? = is_eligible?
 
   def latest_session_status_should_be_contraindicated?
-    vaccination_records.first&.contraindications?
+    vaccination_records.first&.contraindicated?
   end
 
   def latest_session_status_should_be_refused?
     vaccination_records.first&.refused?
   end
 
   def latest_session_status_should_be_absent?
-    vaccination_records.first&.absent_from_session? ||
-      attendance_record&.attending == false
+    vaccination_records.first&.absent? || attendance_record&.attending == false
   end
 
   def latest_session_status_should_be_unwell?
-    vaccination_records.first&.not_well?
+    vaccination_records.first&.unwell?
   end
 
   def latest_session_status_should_be_already_had?

app/models/immunisation_import_row.rb

@@ -75,10 +75,10 @@ class ImmunisationImportRow
 
   REASONS_NOT_ADMINISTERED = {
     "refused" => :refused,
-    "unwell" => :not_well,
-    "vaccination contraindicated" => :contraindications,
+    "unwell" => :unwell,
+    "vaccination contraindicated" => :contraindicated,
     "already had elsewhere" => :already_had,
-    "did not attend" => :absent_from_session
+    "did not attend" => :absent
   }.freeze
 
   SCHOOL_URN_HOME_EDUCATED = "999999"

app/models/vaccination_record.rb

@@ -163,10 +163,10 @@ class ActiveRecord_Relation < ActiveRecord::Relation
        {
          administered: 0,
          refused: 1,
-         not_well: 2,
-         contraindications: 3,
+         unwell: 2,
+         contraindicated: 3,
          already_had: 4,
-         absent_from_session: 6
+         absent: 6
        },
        validate: true
 

app/views/draft_vaccination_records/outcome.html.erb

@@ -21,11 +21,11 @@
 
     <%= f.govuk_radio_button :outcome, "already_had",
                              label: { text: "They have already had the vaccine" } %>
-    <%= f.govuk_radio_button :outcome, "contraindications",
+    <%= f.govuk_radio_button :outcome, "contraindicated",
                              label: { text: "They had contraindications" } %>
     <%= f.govuk_radio_button :outcome, "refused",
                              label: { text: "They refused it" } %>
-    <%= f.govuk_radio_button :outcome, "not_well",
+    <%= f.govuk_radio_button :outcome, "unwell",
                              label: { text: "They were not well enough" } %>
   <% end %>
 

config/locales/en.yml

@@ -296,11 +296,11 @@ en:
           right_arm_lower_position: Right arm (lower position)
           right_arm_upper_position: Right arm (upper position)
         outcomes:
-          absent_from_session: Absent from session
+          absent: Absent from session
           administered: Vaccinated
           already_had: Already had the vaccine
-          contraindications: Had contraindications
-          not_well: Unwell
+          contraindicated: Had contraindications
+          unwell: Unwell
           refused: Refused vaccine
         protocols:
           national: National protocol
@@ -628,10 +628,10 @@ en:
         will_be_vaccinated_elsewhere: they will be given the vaccine elsewhere
     vaccination_mailer:
       reasons_did_not_vaccinate:
-        absent_from_session: they were not in the vaccination session
+        absent: they were not in the vaccination session
         already_had: they've already had the vaccine
-        contraindications: they had contraindications
-        not_well: the nurse decided %{short_patient_name} was not well
+        contraindicated: they had contraindications
+        unwell: the nurse decided %{short_patient_name} was not well
         refused: they refused when the nurse offered the vaccination
   notification_banner:
     info: Information

spec/components/app_patient_programmes_table_component_spec.rb

@@ -76,7 +76,7 @@
           :vaccination_record,
           patient:,
           programme: programmes.first,
-          outcome: :contraindications,
+          outcome: :contraindicated,
           performed_at: today,
           session:
         )
@@ -95,7 +95,7 @@
           :vaccination_record,
           patient:,
           programme: programmes.first,
-          outcome: :not_well,
+          outcome: :unwell,
           performed_at: today,
           session:
         )

spec/components/app_vaccination_record_summary_component_spec.rb

@@ -57,7 +57,7 @@
     end
 
     context "when not administered" do
-      let(:outcome) { :not_well }
+      let(:outcome) { :unwell }
 
       it do
         expect(rendered).to have_css(
@@ -86,7 +86,7 @@
     end
 
     context "without a vaccine" do
-      let(:outcome) { :not_well }
+      let(:outcome) { :unwell }
       let(:vaccine) { nil }
       let(:batch) { nil }
       let(:other_batch) { nil }
@@ -122,7 +122,7 @@
     end
 
     context "without a vaccine" do
-      let(:outcome) { :not_well }
+      let(:outcome) { :unwell }
       let(:vaccine) { nil }
       let(:batch) { nil }
       let(:other_batch) { nil }
@@ -184,7 +184,7 @@
     it { should have_css(".nhsuk-summary-list__row", text: "Batch IDABC") }
 
     context "without a vaccine" do
-      let(:outcome) { :not_well }
+      let(:outcome) { :unwell }
       let(:vaccine) { nil }
       let(:batch) { nil }
       let(:other_batch) { nil }
@@ -202,7 +202,7 @@
     end
 
     context "without a vaccine" do
-      let(:outcome) { :not_well }
+      let(:outcome) { :unwell }
       let(:vaccine) { nil }
       let(:batch) { nil }
       let(:other_batch) { nil }

spec/factories/vaccination_records.rb

@@ -134,7 +134,7 @@
     trait :not_administered do
       delivery_site { nil }
       delivery_method { nil }
-      outcome { "not_well" }
+      outcome { "unwell" }
       vaccine { nil }
       dose_sequence { nil }
       full_dose { nil }
@@ -145,19 +145,19 @@
       outcome { "already_had" }
     end
 
-    trait :contraindications do
+    trait :contraindicated do
       not_administered
-      outcome { "contraindications" }
+      outcome { "contraindicated" }
     end
 
     trait :refused do
       not_administered
       outcome { "refused" }
     end
 
-    trait :absent_from_session do
+    trait :absent do
       not_administered
-      outcome { "absent_from_session" }
+      outcome { "absent" }
     end
 
     trait :performed_by_not_user do