[REMANIEMENT][TABLEAU DE BORD] Modifie la manière de générer le table…

…au de bord en vue de la mise en place du tableau de bord utilisateur inscrit
betagouv · Jan 22, 2025 · e13fc73 · e13fc73
1 parent 3c5907a
commit e13fc73
Show file tree

Hide file tree

Showing 3 changed files with 70 additions and 39 deletions.
diff --git a/mon-aide-cyber-api/src/api/tableau-de-bord/routesAPITableauDeBord.ts b/mon-aide-cyber-api/src/api/tableau-de-bord/routesAPITableauDeBord.ts
@@ -1,6 +1,6 @@
 import express, { Response } from 'express';
 import { NextFunction } from 'express-serve-static-core';
-import { constructeurActionsHATEOAS, ReponseHATEOAS } from '../hateoas/hateoas';
+import { ReponseHATEOAS } from '../hateoas/hateoas';
 import {
   Diagnostic,
   ServiceTableauDeBord,
@@ -29,24 +29,15 @@ export const routesAPITableauDeBord = (configuration: ConfigurationServeur) => {
       reponse: Response,
       __suite: NextFunction
     ) => {
-      const diagnostics = await new ServiceTableauDeBord(
+      const tableauDeBord = await new ServiceTableauDeBord(
         configuration.adaptateurRelations,
-        new ServiceDiagnostic(configuration.entrepots)
-      ).diagnosticsInitiesPar(requete.identifiantUtilisateurCourant!);
+        new ServiceDiagnostic(configuration.entrepots),
+        !!requete.estProConnect
+      ).pour(requete.identifiantUtilisateurCourant!);
 
       return reponse.status(200).json({
-        diagnostics,
-        ...constructeurActionsHATEOAS()
-          .pour({
-            contexte: 'aidant:acceder-au-tableau-de-bord',
-          })
-          .pour({
-            contexte: requete.estProConnect
-              ? 'se-deconnecter-avec-pro-connect'
-              : 'se-deconnecter',
-          })
-          .afficherLesDiagnostics(diagnostics.map((d) => d.identifiant))
-          .construis(),
+        diagnostics: tableauDeBord.diagnostics,
+        ...tableauDeBord.liens,
       });
     }
   );

diff --git a/mon-aide-cyber-api/src/espace-aidant/tableau-de-bord/ServiceTableauDeBord.ts b/mon-aide-cyber-api/src/espace-aidant/tableau-de-bord/ServiceTableauDeBord.ts
@@ -3,6 +3,10 @@ import crypto, { UUID } from 'crypto';
 import { ServiceDiagnostic } from '../../diagnostic/ServiceDiagnostic';
 import { FournisseurHorloge } from '../../infrastructure/horloge/FournisseurHorloge';
 import { isAfter } from 'date-fns';
+import {
+  constructeurActionsHATEOAS,
+  ReponseHATEOAS,
+} from '../../api/hateoas/hateoas';
 
 export type Diagnostic = {
   dateCreation: string;
@@ -11,20 +15,48 @@ export type Diagnostic = {
   secteurGeographique: string | 'non renseigné';
 };
 
+type TableauDeBord = {
+  diagnostics: Diagnostic[];
+  liens: ReponseHATEOAS;
+};
+
 export class ServiceTableauDeBord {
   constructor(
     private readonly adaptateurRelation: AdaptateurRelations,
-    private readonly serviceDiagnostic: ServiceDiagnostic
+    private readonly serviceDiagnostic: ServiceDiagnostic,
+    private readonly estProConnect: boolean
   ) {}
 
-  async diagnosticsInitiesPar(
-    identifiantAidant: crypto.UUID
-  ): Promise<Diagnostic[]> {
+  async pour(identifiantUtilisateur: crypto.UUID): Promise<TableauDeBord> {
     const identifiantDiagnosticsLie =
       await this.adaptateurRelation.identifiantsObjetsLiesAUtilisateur(
-        identifiantAidant
+        identifiantUtilisateur
       );
 
+    const liensPourAidant = (diagnostics: Diagnostic[]): ReponseHATEOAS => {
+      return constructeurActionsHATEOAS()
+        .pour({
+          contexte: 'aidant:acceder-au-tableau-de-bord',
+        })
+        .pour({
+          contexte: this.estProConnect
+            ? 'se-deconnecter-avec-pro-connect'
+            : 'se-deconnecter',
+        })
+        .afficherLesDiagnostics(diagnostics.map((d) => d.identifiant))
+        .construis();
+    };
+
+    return this.recupereLesDiagnostics(identifiantDiagnosticsLie).then(
+      (diagnostics) => {
+        return { diagnostics, liens: liensPourAidant(diagnostics) };
+      }
+    );
+  }
+
+  private recupereLesDiagnostics(
+    identifiantDiagnosticsLie: string[]
+  ): Promise<Diagnostic[]> {
     return this.serviceDiagnostic
       .contextes(identifiantDiagnosticsLie as UUID[])
       .then((diagnostics) => {

diff --git a/mon-aide-cyber-api/test/espace-aidant/tableau-de-bord/ServiceTableauDeBord.spec.ts b/mon-aide-cyber-api/test/espace-aidant/tableau-de-bord/ServiceTableauDeBord.spec.ts
@@ -27,10 +27,11 @@ describe('Service Tableau De Bord', () => {
         ),
         new ServiceDiagnosticTest(
           new Map([[identifiantDiagnostic, unContexte().construis()]])
-        )
-      ).diagnosticsInitiesPar(identifiantAidant);
+        ),
+        true
+      ).pour(identifiantAidant);
 
-      expect(diagnosticTableauDeBord[0]).toStrictEqual<Diagnostic>({
+      expect(diagnosticTableauDeBord.diagnostics[0]).toStrictEqual<Diagnostic>({
         identifiant: identifiantDiagnostic,
         dateCreation: '17.04.2024',
         secteurActivite: 'non renseigné',
@@ -54,10 +55,11 @@ describe('Service Tableau De Bord', () => {
               unContexte().avecDateCreation(date).construis(),
             ],
           ])
-        )
-      ).diagnosticsInitiesPar(identifiantAidant);
+        ),
+        true
+      ).pour(identifiantAidant);
 
-      expect(diagnosticTableauDeBord[0]).toStrictEqual<Diagnostic>({
+      expect(diagnosticTableauDeBord.diagnostics[0]).toStrictEqual<Diagnostic>({
         dateCreation: '28.04.2024',
         identifiant: identifiantDiagnostic,
         secteurActivite: 'non renseigné',
@@ -82,10 +84,13 @@ describe('Service Tableau De Bord', () => {
                 unContexte().avecLeDepartement(departement).construis(),
               ],
             ])
-          )
-        ).diagnosticsInitiesPar(identifiantAidant);
+          ),
+          true
+        ).pour(identifiantAidant);
 
-        expect(diagnosticTableauDeBord[0]).toStrictEqual<Diagnostic>({
+        expect(
+          diagnosticTableauDeBord.diagnostics[0]
+        ).toStrictEqual<Diagnostic>({
           dateCreation: '17.04.2024',
           identifiant: identifiantDiagnostic,
           secteurActivite: 'non renseigné',
@@ -111,10 +116,11 @@ describe('Service Tableau De Bord', () => {
               unContexte().avecSecteurActivite(secteurActivite).construis(),
             ],
           ])
-        )
-      ).diagnosticsInitiesPar(identifiantAidant);
+        ),
+        true
+      ).pour(identifiantAidant);
 
-      expect(diagnosticTableauDeBord[0]).toStrictEqual<Diagnostic>({
+      expect(diagnosticTableauDeBord.diagnostics[0]).toStrictEqual<Diagnostic>({
         dateCreation: '17.04.2024',
         identifiant: identifiantDiagnostic,
         secteurActivite: secteurActivite,
@@ -158,10 +164,11 @@ describe('Service Tableau De Bord', () => {
                 .construis(),
             ],
           ])
-        )
-      ).diagnosticsInitiesPar(identifiantAidant);
+        ),
+        true
+      ).pour(identifiantAidant);
 
-      expect(diagnosticTableauDeBord).toStrictEqual<Diagnostic[]>([
+      expect(diagnosticTableauDeBord.diagnostics).toStrictEqual<Diagnostic[]>([
         {
           dateCreation: '17.04.2024',
           identifiant: identifiantDiagnosticInitie1,
@@ -236,10 +243,11 @@ describe('Service Tableau De Bord', () => {
                 .construis(),
             ],
           ])
-        )
-      ).diagnosticsInitiesPar(identifiantAidant);
+        ),
+        true
+      ).pour(identifiantAidant);
 
-      expect(diagnosticTableauDeBord).toStrictEqual<Diagnostic[]>([
+      expect(diagnosticTableauDeBord.diagnostics).toStrictEqual<Diagnostic[]>([
         {
           dateCreation: '16.03.2024',
           identifiant: identifiantDiagnosticInitie3,