Supprime la vérification par feature flag et utilise la vérif par booléen au booléen
This commit is contained in:
parent
c86ad15b8c
commit
9cebac1d99
4 changed files with 5 additions and 9 deletions
|
@ -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
|
||||
|
|
|
@ -31,7 +31,6 @@ features = [
|
|||
:expression_reguliere_type_de_champ,
|
||||
:gallery_demande,
|
||||
:groupe_instructeur_api_hack,
|
||||
:hide_instructeur_email,
|
||||
:sva,
|
||||
:switch_domain
|
||||
]
|
||||
|
|
|
@ -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) }
|
||||
|
|
|
@ -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
|
||||
|
|
Loading…
Reference in a new issue