From 3b3c845b2e04865cd700a5dd42a483177a92a6be Mon Sep 17 00:00:00 2001 From: mfo Date: Fri, 20 Sep 2024 11:54:42 +0200 Subject: [PATCH] fix(fix_champs_commune_having_value_but_not_external_id): only asks for update for dossier en construction --- ...ommune_having_value_but_not_external_id_task.rb | 14 ++++++++------ ...e_having_value_but_not_external_id_task_spec.rb | 2 +- ...r_with_invalid_commune_external_id_task_spec.rb | 4 ++-- 3 files changed, 11 insertions(+), 9 deletions(-) diff --git a/app/tasks/maintenance/fix_champs_commune_having_value_but_not_external_id_task.rb b/app/tasks/maintenance/fix_champs_commune_having_value_but_not_external_id_task.rb index 7ccf5ce32..3520075ac 100644 --- a/app/tasks/maintenance/fix_champs_commune_having_value_but_not_external_id_task.rb +++ b/app/tasks/maintenance/fix_champs_commune_having_value_but_not_external_id_task.rb @@ -26,13 +26,15 @@ module Maintenance champ.code = formated_results.first[:value] champ.save! else # otherwise, we can't find the expected departement - champ.code_departement = nil - champ.code_postal = nil - champ.external_id = nil - champ.value = nil - champ.save(validate: false) + if champ.dossier.en_construction? + champ.code_departement = nil + champ.code_postal = nil + champ.external_id = nil + champ.value = nil + champ.save(validate: false) - ask_user_correction(champ) + ask_user_correction(champ) + end end end end diff --git a/spec/tasks/maintenance/fix_champs_commune_having_value_but_not_external_id_task_spec.rb b/spec/tasks/maintenance/fix_champs_commune_having_value_but_not_external_id_task_spec.rb index c93e1aa3e..3576df68a 100644 --- a/spec/tasks/maintenance/fix_champs_commune_having_value_but_not_external_id_task_spec.rb +++ b/spec/tasks/maintenance/fix_champs_commune_having_value_but_not_external_id_task_spec.rb @@ -30,7 +30,7 @@ module Maintenance end context 'en_instruction (go back to en_construction!), send comment' do - let(:state) { [:en_instruction, :en_construction].sample } + let(:state) { :en_construction } it 'flags as pending correction' do expect { subject }.to change { champ.reload.value }.from('Marseille').to(nil) diff --git a/spec/tasks/maintenance/resolve_pending_correction_for_dossier_with_invalid_commune_external_id_task_spec.rb b/spec/tasks/maintenance/resolve_pending_correction_for_dossier_with_invalid_commune_external_id_task_spec.rb index 4ae1575b8..81f04c127 100644 --- a/spec/tasks/maintenance/resolve_pending_correction_for_dossier_with_invalid_commune_external_id_task_spec.rb +++ b/spec/tasks/maintenance/resolve_pending_correction_for_dossier_with_invalid_commune_external_id_task_spec.rb @@ -6,7 +6,7 @@ module Maintenance RSpec.describe ResolvePendingCorrectionForDossierWithInvalidCommuneExternalIdTask do describe "#process" do subject(:process) { described_class.process } - let(:instructeur) { create(:instructeur, email: ENV.fetch('DEFAULT_INSTRUCTEUR_EMAIL') { CONTACT_EMAIL }) } + let!(:instructeur) { create(:instructeur, email: ENV.fetch('DEFAULT_INSTRUCTEUR_EMAIL') { CONTACT_EMAIL }) } let(:commentaire) { create(:commentaire, instructeur:) } let(:dossier_correction) { create(:dossier_correction, commentaire:, dossier:, resolved_at: nil) } @@ -22,7 +22,7 @@ module Maintenance context 'when dossier didnt transitioned' do let(:dossier) { create(:dossier, :en_construction) } - + before { create(:traitement, dossier:) } it 'noop' do expect { subject }.not_to change { dossier.reload.state } end