fix(entreprise): don't leak entreprise name if it's not diffusable

app/views/shared/dossiers/_identite_entreprise.html.haml

@@ -20,7 +20,7 @@
     %tbody
       - if etablissement.diffusable_commercialement == false && profile != 'instructeur'
         %tr
-          %td= t('warning_for_private_info', scope: 'views.shared.dossiers.identite_entreprise', etablissement: raison_sociale_or_name(etablissement))
+          %td= t('warning_for_private_info', scope: 'views.shared.dossiers.identite_entreprise', siret: etablissement.siret)
       - else
         %tr
           %td.libelle Dénomination :

app/views/users/dossiers/etablissement.html.haml

@@ -25,7 +25,7 @@
               Vérifier dans l'annuaire des entreprises
 
     - elsif etablissement.diffusable_commercialement == false
-      %p= t('warning_for_private_info', scope: 'views.shared.dossiers.identite_entreprise', etablissement: raison_sociale_or_name(etablissement))
+      %p= t('warning_for_private_info', scope: 'views.shared.dossiers.identite_entreprise', siret: etablissement.siret)
 
     - else
       %p

config/locales/views/shared/en.yml

@@ -3,7 +3,7 @@ en:
     shared:
       dossiers:
         identite_entreprise:
-          warning_for_private_info: "The establishment %{etablissement} applied his right to not publish information regarding his identity. These informaiton won't be visible from instructor services"
+          warning_for_private_info: "The establishment SIRET %{siret} applied his right to not publish information regarding his identity. These informaiton won't be visible from instructor services."
       avis:
         demande_envoyee_le: "Feedback send at %{date}"
         demande_revoquee_le: "Feedback revoked at %{date}"

config/locales/views/shared/fr.yml

@@ -3,7 +3,7 @@ fr:
     shared:
       dossiers:
         identite_entreprise:
-          warning_for_private_info: "L’établissement %{etablissement} a exercé son droit à la non publication des informations relatives à son identité. Les informations ne seront donc visibles que de la part des services instructeurs"
+          warning_for_private_info: "L’établissement SIRET %{siret} a exercé son droit à la non publication des informations relatives à son identité. Les informations ne seront donc visibles que de la part des services instructeurs."
       avis:
         demande_envoyee_le: "Demande d’avis envoyée le %{date}"
         demande_revoquee_le: "Demande d’avis révoquée le %{date}"

spec/views/shared/dossiers/_identite_entreprise.html.haml_spec.rb

@@ -14,10 +14,11 @@ describe 'shared/dossiers/identite_entreprise.html.haml', type: :view do
   end
 
   context "for an entreprise with private infos" do
-    let(:etablissement) { create(:etablissement, :non_diffusable) }
+    let(:etablissement) { create(:etablissement, :non_diffusable, siret: "12345678900001") }
 
-    it "displays only public infos" do
-      expect(rendered).to have_text(etablissement.entreprise_raison_sociale)
+    it "hide any info except siret" do
+      expect(rendered).to have_text("123 456 789 00001")
+      expect(rendered).not_to have_text(etablissement.entreprise_raison_sociale)
       expect(rendered).not_to have_text(etablissement.entreprise.forme_juridique)
     end
   end

spec/views/users/dossiers/etablissement.html.haml_spec.rb

@@ -1,5 +1,5 @@
 describe 'users/dossiers/etablissement.html.haml', type: :view do
-  let(:etablissement) { create(:etablissement, :with_exercices) }
+  let(:etablissement) { create(:etablissement, :with_exercices, siret: "12345678900001") }
   let(:dossier) { create(:dossier, etablissement: etablissement) }
   let(:footer) { view.content_for(:footer) }
 
@@ -13,13 +13,15 @@ describe 'users/dossiers/etablissement.html.haml', type: :view do
   subject! { render }
 
   it 'affiche les informations de l’établissement' do
-    expect(rendered).to have_text(etablissement.siret)
+    expect(rendered).to have_text("12345678900001")
+    expect(rendered).to have_text(etablissement.entreprise_raison_sociale)
   end
 
   context 'etablissement avec infos non diffusables' do
-    let(:etablissement) { create(:etablissement, :with_exercices, :non_diffusable) }
-    it "affiche uniquement le nom de l'établissement si infos non diffusables" do
-      expect(rendered).to have_text(etablissement.entreprise_raison_sociale)
+    let(:etablissement) { create(:etablissement, :with_exercices, :non_diffusable, siret: "12345678900001") }
+    it "affiche uniquement le SIRET si infos non diffusables" do
+      expect(rendered).to have_text("12345678900001")
+      expect(rendered).not_to have_text(etablissement.entreprise_raison_sociale)
       expect(rendered).not_to have_text(etablissement.entreprise.forme_juridique)
     end
   end