Rename none programme outcome

To none yet. This avoids potential confusion with the built in Rails
`none?` method and helps to clarify exactly what this status is for.

I've also removed the label and colour for the partially vaccinated
status as this status doesn't exist in the service.

Author: thomasleese

app/models/patient/programme_outcome.rb

@@ -8,14 +8,14 @@ def initialize(patient)
   STATUSES = [
     VACCINATED = :vaccinated,
     COULD_NOT_VACCINATE = :could_not_vaccinate,
-    NONE = :none
+    NONE_YET = :none_yet
   ].freeze
 
   def vaccinated?(programme) = status[programme] == VACCINATED
 
   def could_not_vaccinate?(programme) = status[programme] == COULD_NOT_VACCINATE
 
-  def none?(programme) = status[programme] == NONE
+  def none_yet?(programme) = status[programme] == NONE_YET
 
   def status
     @status ||=
@@ -43,7 +43,7 @@ def programme_status(programme)
     elsif programme_could_not_vaccinate?(programme)
       COULD_NOT_VACCINATE
     else
-      NONE
+      NONE_YET
     end
   end
 

app/models/patient_session.rb

@@ -109,7 +109,7 @@ def destroy_if_safe!
   end
 
   def can_record_as_already_vaccinated?(programme:)
-    !session.today? && patient.programme_outcome.none?(programme)
+    !session.today? && patient.programme_outcome.none_yet?(programme)
   end
 
   def programmes

config/locales/status.en.yml

@@ -57,11 +57,9 @@ en:
     programme:
       label:
         could_not_vaccinate: Could not vaccinate
-        none: No outcome yet
-        partially_vaccinated: Partially vaccinated
+        none_yet: No outcome yet
         vaccinated: Vaccinated
       colour:
         could_not_vaccinate: red
-        none: grey
-        partially_vaccinated: green
+        none_yet: grey
         vaccinated: green

spec/models/patient/programme_outcome_spec.rb

@@ -12,7 +12,7 @@
     subject(:status) { instance.status[programme] }
 
     context "with no vaccination record" do
-      it { should be(described_class::NONE) }
+      it { should be(described_class::NONE_YET) }
     end
 
     context "with a vaccination administered" do
@@ -40,7 +40,7 @@
         create(:vaccination_record, :not_administered, patient:, programme:)
       end
 
-      it { should be(described_class::NONE) }
+      it { should be(described_class::NONE_YET) }
     end
 
     context "with a consent refused" do
@@ -58,7 +58,7 @@
     context "with a discarded vaccination administered" do
       before { create(:vaccination_record, :discarded, patient:, programme:) }
 
-      it { should be(described_class::NONE) }
+      it { should be(described_class::NONE_YET) }
     end
   end