Ignore completed sessions when deriving consent request status

Fixes a bug where an eligible patient in a new cohort with contactable
parents could be marked as `:no_response` if they were not in an active
session but only had completed sessions.

These patients should instead be marked as `:request_not_scheduled`.

Jira-Issue: MAV-6067

Author: jhenderson

app/lib/status_generator/programme.rb

@@ -339,6 +339,7 @@ def sessions
             .select { it.academic_year == academic_year }
             .flat_map(&:sessions)
             .select { it.programme_types.include?(programme_type) }
+            .reject(&:completed?)
         end
   end
 end

spec/lib/status_generator/programme_spec.rb

@@ -405,6 +405,14 @@
     its(:vaccine_methods) { should be_nil }
     its(:without_gelatine) { should be_nil }
 
+    context "when the only session is completed" do
+      let(:session) { create(:session, :completed, programmes: [programme]) }
+
+      before { ConsentNotification.delete_all }
+
+      its(:status) { should be(:needs_consent_request_not_scheduled) }
+    end
+
     context "when the patient only has a generic clinic location" do
       let(:team) { create(:team, programmes: [programme]) }
       let(:location) { create(:generic_clinic, team:) }