diff --git a/app/models/commentaire.rb b/app/models/commentaire.rb index ccd4de283..6570c9094 100644 --- a/app/models/commentaire.rb +++ b/app/models/commentaire.rb @@ -39,7 +39,7 @@ class Commentaire < ApplicationRecord def redacted_email if sent_by_instructeur? - if dossier.procedure.feature_enabled?(:hide_instructeur_email) + if dossier.procedure.hide_instructeurs_email? "Instructeur n° #{instructeur.id}" else instructeur.email.split('@').first diff --git a/config/initializers/flipper.rb b/config/initializers/flipper.rb index bf87cfae4..8595349bf 100644 --- a/config/initializers/flipper.rb +++ b/config/initializers/flipper.rb @@ -31,7 +31,6 @@ features = [ :expression_reguliere_type_de_champ, :gallery_demande, :groupe_instructeur_api_hack, - :hide_instructeur_email, :sva, :switch_domain ] diff --git a/spec/components/dossiers/message_component_spec.rb b/spec/components/dossiers/message_component_spec.rb index d62dd2489..8614988d6 100644 --- a/spec/components/dossiers/message_component_spec.rb +++ b/spec/components/dossiers/message_component_spec.rb @@ -42,13 +42,11 @@ RSpec.describe Dossiers::MessageComponent, type: :component do context 'with an instructeur message' do let(:instructeur) { create(:instructeur) } - let(:procedure) { create(:procedure) } + let(:procedure) { create(:procedure, hide_instructeurs_email: true) } let(:commentaire) { create(:commentaire, instructeur: instructeur, body: 'Second message') } let(:dossier) { create(:dossier, :en_construction, commentaires: [commentaire], procedure: procedure) } context 'on a procedure with anonymous instructeurs' do - before { Flipper.enable(:hide_instructeur_email, procedure) } - context 'redacts the instructeur email' do it { is_expected.to have_text(commentaire.body) } it { is_expected.to have_text("Instructeur n° #{instructeur.id}") } @@ -57,7 +55,7 @@ RSpec.describe Dossiers::MessageComponent, type: :component do end context 'on a procedure where instructeurs names are not redacted' do - before { Flipper.disable(:hide_instructeur_email, procedure) } + let(:procedure) { create(:procedure, hide_instructeurs_email: false) } context 'redacts the instructeur email but keeps the name' do it { is_expected.to have_text(commentaire.body) } diff --git a/spec/models/commentaire_spec.rb b/spec/models/commentaire_spec.rb index 0a32c59e6..0eda7103e 100644 --- a/spec/models/commentaire_spec.rb +++ b/spec/models/commentaire_spec.rb @@ -51,7 +51,7 @@ describe Commentaire do describe "#redacted_email" do subject { commentaire.redacted_email } - let(:procedure) { create(:procedure) } + let(:procedure) { create(:procedure, hide_instructeurs_email: false) } let(:dossier) { create(:dossier, procedure: procedure) } context 'with a commentaire created by a instructeur' do @@ -59,12 +59,11 @@ describe Commentaire do let(:commentaire) { build :commentaire, instructeur: instructeur, dossier: dossier } context 'when the procedure shows instructeurs email' do - before { Flipper.disable(:hide_instructeur_email, procedure) } it { is_expected.to eq 'some_user' } end context 'when the procedure hides instructeurs email' do - before { Flipper.enable(:hide_instructeur_email, procedure) } + let(:procedure) { create(:procedure, hide_instructeurs_email: true) } it { is_expected.to eq "Instructeur n° #{instructeur.id}" } end end