Skip to content

Commit

Permalink
Fix standardrb
Browse files Browse the repository at this point in the history
  • Loading branch information
Isalafont committed Oct 16, 2024
1 parent 4d272e8 commit 0e812f3
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 9 deletions.
11 changes: 4 additions & 7 deletions backend/app/models/concerns/dgfip_validation_methods.rb
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
Expand All @@ -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?
Expand All @@ -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
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand All @@ -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
Expand Down

0 comments on commit 0e812f3

Please sign in to comment.