Merge pull request #11090 from demarches-simplifiees/fix_avis_task

Do not raise exception when Avis is not valid, just fix the data

app/tasks/maintenance/fix_avis_experts_procedure_mismatch_task.rb

@@ -20,7 +20,7 @@ def process(avis)
         expert: expert
       )
 
-      avis.update!(experts_procedure: correct_experts_procedure)
+      avis.update_column(:experts_procedure_id, correct_experts_procedure.id)
     end
   end
 end

spec/tasks/maintenance/fix_avis_experts_procedure_mismatch_task_spec.rb

@@ -78,6 +78,25 @@ module Maintenance
           expect(avis.experts_procedure).to eq(correct_experts_procedure)
         end
       end
+
+      context "when the avis has an invalid question_answer" do
+        let(:avis) do
+          create(:avis,
+            dossier: dossier,
+            expert: expert,
+            experts_procedure: wrong_experts_procedure,
+            question_label: "Some question",
+            question_answer: nil)
+        end
+
+        it "fixes the experts_procedure association without validation errors" do
+          expect(avis).not_to be_valid
+          expect { process }.not_to raise_error
+
+          avis.reload
+          expect(avis.experts_procedure.procedure_id).to eq(procedure2.id)
+        end
+      end
     end
   end
 end