Relance en paiement les PFMPs bloquées pour des motifs liés à l'adresse

app/controllers/ribs_controller.rb

@@ -152,7 +152,7 @@ def rib_is_readonly
 
   def retry_rejected_or_unpaid_payment_request!
     @student.pfmps.in_state(:validated).each do |pfmp|
-      if pfmp.latest_payment_request&.eligible_for_rejected_or_unpaid_auto_retry?
+      if pfmp.latest_payment_request&.eligible_for_rejected_or_unpaid_auto_retry?(%w[RIB BIC PAIEMENT])
         p_r = PfmpManager.new(pfmp).create_new_payment_request!
         p_r.mark_ready!
       end

app/jobs/sync/student_job.rb

@@ -23,6 +23,19 @@ def fetch_student_data(schooling)
       api.fetch_resource(:student, ine: schooling.student.ine)
          .then { |data| map_student_attributes(data, api) }
          .then { |attributes| schooling.student.update!(attributes) }
+
+      student = schooling.student
+      if !student.previous_changes.key?("address_line1") ||
+          !student.previous_changes.key?("address_line2") ||
+          !student.previous_changes.key?("address_city_insee_code") ||
+          !student.previous_changes.key?("address_country_code")
+        student.pfmps.in_state(:validated).each do |pfmp|
+          if pfmp.latest_payment_request&.eligible_for_rejected_or_unpaid_auto_retry?(%w[ADRESSE PAYS])
+            p_r = PfmpManager.new(pfmp).create_new_payment_request!
+            p_r.mark_ready!
+          end
+        end
+      end
     end
 
     def map_student_attributes(data, api)

app/models/asp/payment_request.rb

@@ -126,12 +126,12 @@ def eligible_for_incomplete_retry?
       last_transition.metadata["incomplete_reasons"]["ready_state_validation"].intersect?(retryable_messages)
     end
 
-    def eligible_for_rejected_or_unpaid_auto_retry?
+    def eligible_for_rejected_or_unpaid_auto_retry?(reasons)
       return false unless in_state?(:rejected) || in_state?(:unpaid)
 
       decorator = ActiveDecorator::Decorator.instance.decorate(self)
       message = in_state?(:rejected) ? decorator.rejected_reason : decorator.unpaid_reason
-      %w[RIB BIC PAIEMENT].any? { |word| message.upcase.include?(word) }
+      reasons.any? { |word| message.upcase.include?(word) }
     end
   end
 end

app/models/student.rb

@@ -66,6 +66,7 @@ class Student < ApplicationRecord # rubocop:disable Metrics/ClassLength
 
   sourced_from_external_api :birthdate,
                             :address_line1,
+                            :address_line2,
                             :address_postal_code,
                             :address_city_insee_code,
                             :address_country_code,

spec/models/asp/payment_request_spec.rb

@@ -170,7 +170,7 @@
       let(:p_r) { create(:asp_payment_request, :rejected) }
 
       it "returns false" do
-        expect(p_r.eligible_for_rejected_or_unpaid_auto_retry?).to be false
+        expect(p_r.eligible_for_rejected_or_unpaid_auto_retry?(%w[RIB BIC PAIEMENT])).to be false
       end
     end
 
@@ -180,15 +180,15 @@
       end
 
       it "returns true" do
-        expect(p_r.eligible_for_rejected_or_unpaid_auto_retry?).to be true
+        expect(p_r.eligible_for_rejected_or_unpaid_auto_retry?(%w[RIB BIC PAIEMENT])).to be true
       end
     end
 
     context "when the payment request is in 'unpaid' state without a RIB reason" do
       let(:p_r) { create(:asp_payment_request, :unpaid) }
 
       it "returns false" do
-        expect(p_r.eligible_for_rejected_or_unpaid_auto_retry?).to be false
+        expect(p_r.eligible_for_rejected_or_unpaid_auto_retry?(%w[RIB BIC PAIEMENT])).to be false
       end
     end
 
@@ -198,7 +198,7 @@
       end
 
       it "returns true" do
-        expect(p_r.eligible_for_rejected_or_unpaid_auto_retry?).to be true
+        expect(p_r.eligible_for_rejected_or_unpaid_auto_retry?(%w[RIB BIC PAIEMENT])).to be true
       end
     end
   end