do not display the action for instructeur

app/views/users/profil/show.html.haml

@@ -23,21 +23,21 @@
         = f.email_field :email, value: nil, placeholder: 'Nouvelle adresse email', required: true
         = f.submit "Changer mon adresse", class: 'button primary'
 
-  .card
+  - if !instructeur_signed_in?
-    .card-title= t('.transfer_title')
+    .card
-    = t('.transfer_explication_html')
+      .card-title= t('.transfer_title')
+      = t('.transfer_explication_html')
 
-    = form_tag transfer_all_dossiers_path, class: 'form'  do
+      = form_tag transfer_all_dossiers_path, class: 'form'  do
-      = email_field_tag :next_owner, nil, required: true
+        = email_field_tag :next_owner, nil, required: true
-      = submit_tag "Transférer tous mes dossiers", class: 'button primary', data: { confirm: t('.transfer_confirmation') }
+        = submit_tag "Transférer tous mes dossiers", class: 'button primary', data: { confirm: t('.transfer_confirmation') }
-
-    - if @waiting_transfers.any?
-      .card.warning
-        .card-title= t('.waiting_transfers')
-        %ul
-          - @waiting_transfers.each do |email, nb_dossier|
-            %li= t('.one_waiting_transfer', email: email, count: nb_dossier)
 
+      - if @waiting_transfers.any?
+        .card.warning
+          .card-title= t('.waiting_transfers')
+          %ul
+            - @waiting_transfers.each do |email, nb_dossier|
+              %li= t('.one_waiting_transfer', email: email, count: nb_dossier)
 
   - if current_administrateur.present?
     .card

spec/controllers/users/profil_controller_spec.rb

@@ -6,15 +6,29 @@ describe Users::ProfilController, type: :controller do
   before { sign_in(user) }
 
   describe 'GET #show' do
-    let(:dossiers) { Array.new(3) { create(:dossier, user: user) } }
-    let(:next_owner) { 'loulou@lou.com' }
-    let!(:transfer) { DossierTransfer.initiate(next_owner, dossiers) }
-
     render_views
 
     before { post :show }
 
-    it { expect(response.body).to include(I18n.t('users.profil.show.one_waiting_transfer', count: dossiers.count, email: next_owner)) }
+    context 'when the current user is not an instructeur' do
+      it { expect(response.body).to include(I18n.t('users.profil.show.transfer_title')) }
+
+      context 'when an existing transfer exists' do
+        let(:dossiers) { Array.new(3) { create(:dossier, user: user) } }
+        let(:next_owner) { 'loulou@lou.com' }
+        let!(:transfer) { DossierTransfer.initiate(next_owner, dossiers) }
+
+        before { post :show }
+
+        it { expect(response.body).to include(I18n.t('users.profil.show.one_waiting_transfer', count: dossiers.count, email: next_owner)) }
+      end
+    end
+
+    context 'when the current user is an instructeur' do
+      let(:user) { create(:instructeur).user }
+
+      it { expect(response.body).not_to include(I18n.t('users.profil.show.transfer_title')) }
+    end
   end
 
   describe 'POST #renew_api_token' do