From 0e812f388a9549cbf6cf930dc33fd866566e612e Mon Sep 17 00:00:00 2001 From: Isabelle Lafont Date: Wed, 16 Oct 2024 10:43:20 +0200 Subject: [PATCH] Fix standardrb --- .../app/models/concerns/dgfip_validation_methods.rb | 11 ++++------- .../enrollment/api_impot_particulier_sandbox_spec.rb | 4 ++-- 2 files changed, 6 insertions(+), 9 deletions(-) diff --git a/backend/app/models/concerns/dgfip_validation_methods.rb b/backend/app/models/concerns/dgfip_validation_methods.rb index cbe0657fa..6ceb29b77 100644 --- a/backend/app/models/concerns/dgfip_validation_methods.rb +++ b/backend/app/models/concerns/dgfip_validation_methods.rb @@ -11,7 +11,6 @@ def rgpd_general_agreement_validation return if additional_content&.fetch("rgpd_general_agreement", false) errors.add(:additional_content, :invalid, message: "Vous devez attester que votre organisation déclarera à la DGFiP l'accomplissement des formalités en terme de protection des données") - end def api_impot_particulier_scope_validation @@ -33,7 +32,6 @@ def api_impot_particulier_acces_validation return if additional_content.any? { |k, v| v && %w[acces_spi acces_etat_civil].include?(k) } errors.add(:scopes, :invalid, message: "Vous devez cocher au moins une modalité d’accès avant de continuer") - end def production_form_validation @@ -60,8 +58,8 @@ def production_form_validation def validate_revenue_years_selection selected_years = %w[ - dgfip_annee_n_moins_1 dgfip_annee_n_moins_2 dgfip_annee_n_moins_3 - dgfip_annee_n_moins_2_si_indispo_n_moins_1 + dgfip_annee_n_moins_1 dgfip_annee_n_moins_2 dgfip_annee_n_moins_3 + dgfip_annee_n_moins_2_si_indispo_n_moins_1 ] return unless (scopes & selected_years).empty? @@ -71,18 +69,17 @@ def validate_revenue_years_selection def validate_incompatible_scopes(incompatible_scopes) special_scopes = %w[ - dgfip_annee_n_moins_2_si_indispo_n_moins_1 dgfip_annee_df_au_3112_si_deces_ctb_mp + dgfip_annee_n_moins_2_si_indispo_n_moins_1 dgfip_annee_df_au_3112_si_deces_ctb_mp ] return unless (scopes & special_scopes).present? && (scopes & incompatible_scopes).present? errors.add(:scopes, :invalid, message: "Des données incompatibles entre elles ont été cochées. Pour connaître les modalités d’appel et de réponse de l’API Impôt particulier ainsi que les données proposées, vous pouvez consulter le guide de présentation de cette API dans la rubrique « Les données nécessaires > Comment choisir les données »") - end def validate_exclusive_years_scope_combination if (scopes & %w[dgfip_annee_n_moins_2_si_indispo_n_moins_1]).present? && - (scopes & %w[dgfip_annee_n_moins_1 dgfip_annee_n_moins_2 dgfip_annee_n_moins_3]).present? + (scopes & %w[dgfip_annee_n_moins_1 dgfip_annee_n_moins_2 dgfip_annee_n_moins_3]).present? errors.add(:scopes, :invalid, message: "Vous ne pouvez pas sélectionner la donnée 'avant dernière année de revenu, si la dernière année de revenu est indisponible' avec d'autres années de revenus") end end diff --git a/backend/spec/models/enrollment/api_impot_particulier_sandbox_spec.rb b/backend/spec/models/enrollment/api_impot_particulier_sandbox_spec.rb index f8b1770bc..9d477390f 100644 --- a/backend/spec/models/enrollment/api_impot_particulier_sandbox_spec.rb +++ b/backend/spec/models/enrollment/api_impot_particulier_sandbox_spec.rb @@ -15,7 +15,7 @@ context "when incompatible years combination years are selected" do before { enrollment.update(scopes: ["dgfip_annee_n_moins_2_si_indispo_n_moins_1", "dgfip_annee_n_moins_1"]) } - it 'is not valid' do + it "is not valid" do enrollment.send(:submit_validation) expect(enrollment.errors[:scopes]).to include("Vous ne pouvez pas sélectionner la donnée 'avant dernière année de revenu, si la dernière année de revenu est indisponible' avec d'autres années de revenus") end @@ -24,7 +24,7 @@ context "when exclusive years combination years are selected" do before { enrollment.update(scopes: ["dgfip_annee_n_moins_1", "dgfip_annee_n_moins_2"]) } - it 'is valid' do + it "is valid" do enrollment.send(:submit_validation) expect(enrollment).to be_valid end