Extra spec and comments

app/services/participants/determine_eligibility_status.rb

@@ -1,5 +1,20 @@
 # frozen_string_literal: true
 
+##
+# Determine the +status+ and +reason+ for a participant's +ECFParticipantEligibility+ record
+# Uses the flags stored on the +ECFParticipantEligibility+ and related info from the
+# associated +ParticipantProfile+
+# If the +manually_validated+ attribute is set then the eligibility will not be re-evaluated. Use the <tt>force_validation: true</tt>
+# param to override this behaviour.
+#
+# == Parameters
+# @param :ecf_participant_eligibility [ECFParticipantEligibility] the record to determine eligibilty for
+# @param :save_record [Boolean] whether to save the record after setting the +status+ and +reason+ attributes. Default is +true+
+# @param :force_validation [Boolean] override the +manually_validated+ guard flag if set. Default is +false+
+#
+# == Usage:
+# <tt>Participants::DetermineEligibilityStatus.call(ecf_participant_eligibility: eligibility_record)</tt>
+#
 class Participants::DetermineEligibilityStatus < BaseService
   delegate :active_flags?,
            :previous_participation?,

spec/services/participants/determine_eligibility_status_spec.rb

@@ -4,7 +4,7 @@
 
 RSpec.describe Participants::DetermineEligibilityStatus do
   let(:participant_profile) { create(:ect_participant_profile) }
-  let(:eligibility) { create(:ecf_participant_eligibility, participant_profile:, active_flags: false, previous_participation: false, previous_induction: false, qts: true) }
+  let(:eligibility) { create(:ecf_participant_eligibility, participant_profile:) }
   let(:save_record) { true }
   let(:force_validation) { false }
   subject(:service_call) { described_class.call(ecf_participant_eligibility: eligibility, save_record:, force_validation:) }
@@ -38,6 +38,20 @@
       end
     end
 
+    context "when save_record is false" do
+      let(:save_record) { false }
+
+      before do
+        eligibility.active_flags = true
+        service_call
+      end
+
+      it "does not persist the changes" do
+        expect(eligibility).to be_manual_check_status
+        expect(eligibility.reload).to be_eligible_status
+      end
+    end
+
     context "when active_flags are true" do
       before do
         eligibility.active_flags = true