Merge pull request #5736 from betagouv/dev

2020-11-12-01

app/controllers/instructeurs/procedures_controller.rb

@@ -102,7 +102,6 @@ module Instructeurs
       @has_termine_notifications = current_instructeur.notifications_for_procedure(@procedure, :termine).exists?
 
       @not_archived_notifications_dossier_ids = current_instructeur.notifications_for_procedure(@procedure, :not_archived).pluck(:id)
-      @counters = @procedure.dossiers_count_for_instructeur(current_instructeur)
 
       sorted_ids = procedure_presentation.sorted_ids(@dossiers, current_instructeur)
 

app/views/instructeurs/procedures/show.html.haml

@@ -24,29 +24,29 @@
           = tab_item('à suivre',
             instructeur_procedure_path(@procedure, statut: 'a-suivre'),
             active: @statut == 'a-suivre',
-            badge: number_with_html_delimiter(@counters['a_suivre']))
+            badge: number_with_html_delimiter(@a_suivre_dossiers.count))
 
-          = tab_item(t('pluralize.followed', count: @counters['suivis']),
+          = tab_item(t('pluralize.followed', count: @followed_dossiers.count),
             instructeur_procedure_path(@procedure, statut: 'suivis'),
             active: @statut == 'suivis',
-            badge: number_with_html_delimiter(@counters['suivis']),
+            badge: number_with_html_delimiter(@followed_dossiers.count),
             notification: @has_en_cours_notifications)
 
-          = tab_item(t('pluralize.processed', count: @counters['termines']),
+          = tab_item(t('pluralize.processed', count: @termines_dossiers.count),
             instructeur_procedure_path(@procedure, statut: 'traites'),
             active: @statut == 'traites',
-            badge: number_with_html_delimiter(@counters['termines']),
+            badge: number_with_html_delimiter(@termines_dossiers.count),
             notification: @has_termine_notifications)
 
           = tab_item('tous les dossiers',
             instructeur_procedure_path(@procedure, statut: 'tous'),
             active: @statut == 'tous',
-            badge: number_with_html_delimiter(@counters['total']))
+            badge: number_with_html_delimiter(@all_state_dossiers.count))
 
           = tab_item(t('pluralize.archived', count: @archived_dossiers.count),
             instructeur_procedure_path(@procedure, statut: 'archives'),
             active: @statut == 'archives',
-            badge: number_with_html_delimiter(@counters['archived']))
+            badge: number_with_html_delimiter(@archived_dossiers.count))
 
       .procedure-actions
         = render partial: "download_dossiers",

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

@@ -1,4 +1,8 @@
 .card.featured
   .flex.justify-center
     = image_tag "logo-france-connect.png", alt: "France Connect logo", width: 200, class: "mb-2"
-  .card-title Le dossier a été déposé par le compte de #{user_information.given_name} #{user_information.family_name}, authentifié par France Connect le #{user_information.updated_at.strftime('%d/%m/%Y')}.
+  .card-title
+    - if user_information.updated_at.present?
+      Le dossier a été déposé par le compte de #{user_information&.given_name} #{user_information&.family_name}, authentifié par France Connect le #{user_information.updated_at.strftime('%d/%m/%Y')}.
+    - else
+      Le dossier a été déposé par le compte de #{user_information&.given_name} #{user_information&.family_name}.

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

@@ -0,0 +1,32 @@
+describe 'shared/dossiers/france_connect_informations.html.haml', type: :view do
+  subject do
+    render(
+      'shared/dossiers/france_connect_informations.html.haml',
+      user_information: user_information
+    )
+  end
+
+  context "with complete france_connect information" do
+    let(:user_information) { build(:france_connect_information, updated_at: Time.zone.now) }
+    it {
+      expect(subject).to have_text("Le dossier a été déposé par le compte de #{user_information.given_name} #{user_information.family_name}, authentifié par France Connect le #{user_information.updated_at.strftime('%d/%m/%Y')}")
+    }
+  end
+
+  context "with missing updated_at" do
+    let(:user_information) { build(:france_connect_information, updated_at: nil) }
+
+    it {
+      expect(subject).to have_text("Le dossier a été déposé par le compte de #{user_information.given_name} #{user_information.family_name}")
+      expect(subject).not_to have_text("authentifié par France Connect le ")
+    }
+  end
+
+  context "with missing given_name" do
+    let(:user_information) { build(:france_connect_information, given_name: nil) }
+
+    it {
+      expect(subject).to have_text("Le dossier a été déposé par le compte de  #{user_information.family_name}")
+    }
+  end
+end