refactor: ajouter activité bénéficiaire dans portefeuille context

app/(connected)/(with-sidebar)/(with-chat)/mes-jeunes/creation-jeune/CreationBeneficiaireFranceTravailPage.tsx

@@ -38,7 +38,9 @@ function CreationBeneficiaireFranceTravailPage(): ReactElement {
         email: nouveauBeneficiaire.email,
       })
 
-      setPortefeuille(portefeuille.concat(beneficiaireCree))
+      setPortefeuille(
+        portefeuille.concat({ ...beneficiaireCree, estAArchiver: false })
+      )
       setAlerte(AlerteParam.creationBeneficiaire, beneficiaireCree.id)
       router.push('/mes-jeunes')
     } catch (error) {

app/(connected)/(with-sidebar)/(with-chat)/mes-jeunes/creation-jeune/CreationBeneficiaireMiloPage.tsx

@@ -59,7 +59,9 @@ function CreationBeneficiaireMiloPage() {
         surcharge
       )
 
-      setPortefeuille(portefeuille.concat(beneficiaireCree))
+      setPortefeuille(
+        portefeuille.concat({ ...beneficiaireCree, estAArchiver: false })
+      )
       setAlerte(AlerteParam.creationBeneficiaire, beneficiaireCree.id)
       router.push('/mes-jeunes')
       router.refresh()

fixtures/beneficiaire.ts

@@ -5,6 +5,7 @@ import {
   BeneficiaireAvecCompteursActionsRdvs,
   BeneficiaireEtChat,
   BeneficiaireFromListe,
+  BeneficiaireWithActivity,
   CategorieSituation,
   Chat,
   ConseillerHistorique,
@@ -35,6 +36,16 @@ export const uneBaseBeneficiaire = (
   return { ...defaults, ...overrides }
 }
 
+export const unBeneficiaireWithActivity = (
+  overrides: Partial<BeneficiaireWithActivity> = {}
+): BeneficiaireWithActivity => {
+  const defaults: BeneficiaireWithActivity = {
+    ...uneBaseBeneficiaire(),
+    estAArchiver: false,
+  }
+  return { ...defaults, ...overrides }
+}
+
 export const unDetailBeneficiaire = (
   overrides: Partial<DetailBeneficiaire> = {}
 ): DetailBeneficiaire => {
@@ -103,10 +114,9 @@ export const unItemBeneficiaire = (
   overrides: Partial<BeneficiaireFromListe> = {}
 ): BeneficiaireFromListe => {
   const defaults: BeneficiaireFromListe = {
-    ...uneBaseBeneficiaire(),
+    ...unBeneficiaireWithActivity({ lastActivity: '2021-12-07T17:30:07.756Z' }),
     isActivated: true,
     isReaffectationTemporaire: false,
-    lastActivity: '2021-12-07T17:30:07.756Z',
     situationCourante: CategorieSituation.SANS_SITUATION,
   }
   return { ...defaults, ...overrides }
@@ -167,6 +177,7 @@ export const unItemBeneficiaireJson = (
     id: 'beneficiaire-1',
     firstName: 'Kenji',
     lastName: 'Jirac',
+    estAArchiver: false,
     isActivated: true,
     isReaffectationTemporaire: false,
     lastActivity: '2021-12-07T17:30:07.756Z',
@@ -311,12 +322,6 @@ export const desConseillersBeneficiaireJson =
     }))
   }
 
-export function extractBaseBeneficiaire(
-  base: BaseBeneficiaire
-): BaseBeneficiaire {
-  return { id: base.id, nom: base.nom, prenom: base.prenom }
-}
-
 export const desIndicateursSemaineJson = (
   overrides: Partial<IndicateursSemaineJson> = {}
 ): IndicateursSemaineJson => {

interfaces/beneficiaire.ts

@@ -1,7 +1,6 @@
 import { DateTime } from 'luxon'
 
 import { StatutDemarche } from 'interfaces/json/beneficiaire'
-import { compareDates, compareDatesDesc } from 'utils/date'
 
 export enum EtatSituation {
   EN_COURS = 'en cours',
@@ -21,27 +20,33 @@ export enum CategorieSituation {
   SANS_SITUATION = 'Sans situation',
 }
 
-export interface BaseBeneficiaire {
+export type Portefeuille = BeneficiaireWithActivity[]
+
+export type BaseBeneficiaire = {
   id: string
   prenom: string
   nom: string
 }
 
-export interface BeneficiaireFromListe extends BaseBeneficiaire {
+export type BeneficiaireWithActivity = BaseBeneficiaire & {
+  estAArchiver: boolean
+  lastActivity?: string
+  dateFinCEJ?: string
+}
+
+export type BeneficiaireFromListe = BeneficiaireWithActivity & {
   isActivated: boolean
   isReaffectationTemporaire: boolean
-  lastActivity?: string
   conseillerPrecedent?: {
     nom: string
     prenom: string
     email?: string
   }
   situationCourante: CategorieSituation
   structureMilo?: { id: string }
-  dateFinCEJ?: string
 }
 
-export interface DetailBeneficiaire extends BaseBeneficiaire {
+export type DetailBeneficiaire = BaseBeneficiaire & {
   creationDate: string
   isActivated: boolean
   isReaffectationTemporaire: boolean
@@ -60,7 +65,7 @@ export interface DetailBeneficiaire extends BaseBeneficiaire {
   structureMilo?: { id: string }
 }
 
-export interface MetadonneesFavoris {
+export type MetadonneesFavoris = {
   autoriseLePartage: boolean
   offres: {
     total: number
@@ -95,7 +100,7 @@ export type BeneficiaireEtablissement = {
   dateDerniereActivite?: string
 }
 
-export interface Chat {
+export type Chat = {
   chatId: string
   seenByConseiller: boolean
   flaggedByConseiller: boolean
@@ -110,7 +115,7 @@ export interface Chat {
 
 export type BeneficiaireEtChat = BaseBeneficiaire & Chat
 
-export interface DossierMilo {
+export type DossierMilo = {
   id: string
   prenom: string
   nom: string
@@ -119,7 +124,7 @@ export interface DossierMilo {
   email?: string
 }
 
-export interface ConseillerHistorique {
+export type ConseillerHistorique = {
   id: string
   nom: string
   prenom: string
@@ -163,13 +168,6 @@ export function compareBeneficiairesByNom(
   )
 }
 
-export function compareBeneficiairesByLastNameDesc(
-  beneficiaire1: BaseBeneficiaire,
-  beneficiaire2: BaseBeneficiaire
-): number {
-  return -compareBeneficiairesByNom(beneficiaire1, beneficiaire2)
-}
-
 export function compareBeneficiairesBySituation(
   beneficiaire1: BeneficiaireFromListe,
   beneficiaire2: BeneficiaireFromListe
@@ -179,13 +177,6 @@ export function compareBeneficiairesBySituation(
   )
 }
 
-export function compareBeneficiairesBySituationDesc(
-  beneficiaire1: BeneficiaireFromListe,
-  beneficiaire2: BeneficiaireFromListe
-): number {
-  return -compareBeneficiairesBySituation(beneficiaire1, beneficiaire2)
-}
-
 export function compareBeneficiaireChat(
   a: BeneficiaireEtChat,
   b: BeneficiaireEtChat
@@ -197,34 +188,6 @@ export function compareBeneficiaireChat(
   )
 }
 
-export function compareBeneficiairesByLastActivity(
-  beneficiaire1: BeneficiaireFromListe,
-  beneficiaire2: BeneficiaireFromListe,
-  sortStatutCompteActif: number
-) {
-  const date1 = beneficiaire1.lastActivity
-    ? DateTime.fromISO(beneficiaire1.lastActivity)
-    : undefined
-  const date2 = beneficiaire2.lastActivity
-    ? DateTime.fromISO(beneficiaire2.lastActivity)
-    : undefined
-  return compareDates(date1, date2) || sortStatutCompteActif
-}
-
-export function compareBeneficiairesByLastActivityDesc(
-  beneficiaire1: BeneficiaireFromListe,
-  beneficiaire2: BeneficiaireFromListe,
-  sortStatutCompteActif: number
-) {
-  const date1 = beneficiaire1.lastActivity
-    ? DateTime.fromISO(beneficiaire1.lastActivity)
-    : undefined
-  const date2 = beneficiaire2.lastActivity
-    ? DateTime.fromISO(beneficiaire2.lastActivity)
-    : undefined
-  return compareDatesDesc(date1, date2) || -sortStatutCompteActif
-}
-
 export function getNomBeneficiaireComplet(
   b: Pick<BaseBeneficiaire, 'nom' | 'prenom'>
 ): string {
@@ -265,3 +228,20 @@ function comparerParDate(a: BeneficiaireEtChat, b: BeneficiaireEtChat): number {
   }
   return 0
 }
+
+export function extractBaseBeneficiaire(
+  base: BaseBeneficiaire
+): BaseBeneficiaire {
+  return { id: base.id, nom: base.nom, prenom: base.prenom }
+}
+
+export function extractBeneficiaireWithActivity(
+  beneficiaire: BeneficiaireWithActivity
+): BeneficiaireWithActivity {
+  return {
+    ...extractBaseBeneficiaire(beneficiaire),
+    estAArchiver: beneficiaire.estAArchiver,
+    lastActivity: beneficiaire.lastActivity,
+    dateFinCEJ: beneficiaire.dateFinCEJ,
+  }
+}

interfaces/json/beneficiaire.ts

@@ -32,6 +32,7 @@ export interface BaseBeneficiaireJson {
 export interface ItemBeneficiaireJson extends BaseBeneficiaireJson {
   isActivated: boolean
   isReaffectationTemporaire: boolean
+  estAArchiver: boolean
   lastActivity?: string
   situationCourante?: Situation
   dateFinCEJ?: string

tests/components/ChatContainer.test.tsx

@@ -5,11 +5,10 @@ import React from 'react'
 import ChatContainer from 'components/chat/ChatContainer'
 import {
   desItemsBeneficiaires,
-  extractBaseBeneficiaire,
   unBeneficiaireChat,
 } from 'fixtures/beneficiaire'
 import { desListesDeDiffusion } from 'fixtures/listes-de-diffusion'
-import { BaseBeneficiaire, BeneficiaireEtChat } from 'interfaces/beneficiaire'
+import { BaseBeneficiaire, BeneficiaireEtChat, extractBaseBeneficiaire } from 'interfaces/beneficiaire'
 import { getConseillersDuJeuneClientSide } from 'services/beneficiaires.service'
 import { getListesDeDiffusionClientSide } from 'services/listes-de-diffusion.service'
 import { getMessageImportant } from 'services/messages.service'

tests/components/ChatRoom.test.tsx

@@ -9,11 +9,14 @@ import ChatRoom from 'components/chat/ChatRoom'
 import AlerteDisplayer from 'components/layouts/AlerteDisplayer'
 import {
   desItemsBeneficiaires,
-  extractBaseBeneficiaire,
   unBeneficiaireChat,
 } from 'fixtures/beneficiaire'
 import { unConseiller } from 'fixtures/conseiller'
-import { BaseBeneficiaire, BeneficiaireEtChat } from 'interfaces/beneficiaire'
+import {
+  BaseBeneficiaire,
+  BeneficiaireEtChat,
+  extractBaseBeneficiaire,
+} from 'interfaces/beneficiaire'
 import {
   desactiverMessageImportant,
   getMessageImportant,

tests/pages/CreationBeneficiaireFranceTravailPage.test.tsx

@@ -1,16 +1,18 @@
 import { screen } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 import { axe } from 'jest-axe'
-import { Mock } from 'jest-mock'
 import { useRouter } from 'next/navigation'
 
 import CreationBeneficiaireFranceTravailPage from 'app/(connected)/(with-sidebar)/(with-chat)/mes-jeunes/creation-jeune/CreationBeneficiaireFranceTravailPage'
 import {
   desItemsBeneficiaires,
-  extractBaseBeneficiaire,
+  unBeneficiaireWithActivity,
   uneBaseBeneficiaire,
 } from 'fixtures/beneficiaire'
-import { BaseBeneficiaire } from 'interfaces/beneficiaire'
+import {
+  extractBeneficiaireWithActivity,
+  Portefeuille,
+} from 'interfaces/beneficiaire'
 import { AlerteParam } from 'referentiel/alerteParam'
 import { createCompteJeuneFranceTravail } from 'services/beneficiaires.service'
 import renderWithContexts from 'tests/renderWithContexts'
@@ -21,17 +23,17 @@ describe('CreationBeneficiaireFranceTravailPage client side', () => {
   let container: HTMLElement
   let submitButton: HTMLElement
 
-  let push: Function
+  let push: () => void
   let alerteSetter: (key: AlerteParam | undefined, target?: string) => void
-  let portefeuilleSetter: (updatedBeneficiaires: BaseBeneficiaire[]) => void
-  let portefeuille: BaseBeneficiaire[]
+  let portefeuilleSetter: (updatedBeneficiaires: Portefeuille) => void
+  let portefeuille: Portefeuille
   const emailLabel: string = '* E-mail (ex : monemail@exemple.com)'
   beforeEach(async () => {
-    push = jest.fn(() => Promise.resolve())
+    push = jest.fn()
     ;(useRouter as jest.Mock).mockReturnValue({ push })
     alerteSetter = jest.fn()
     portefeuilleSetter = jest.fn()
-    portefeuille = desItemsBeneficiaires().map(extractBaseBeneficiaire)
+    portefeuille = desItemsBeneficiaires().map(extractBeneficiaireWithActivity)
     ;({ container } = renderWithContexts(
       <CreationBeneficiaireFranceTravailPage />,
       {
@@ -139,7 +141,7 @@ describe('CreationBeneficiaireFranceTravailPage client side', () => {
 
     it('a11y', async () => {
       // Given
-      ;(createCompteJeuneFranceTravail as Mock<any>).mockResolvedValue(
+      ;(createCompteJeuneFranceTravail as jest.Mock).mockResolvedValue(
         uneBaseBeneficiaire()
       )
 
@@ -153,7 +155,7 @@ describe('CreationBeneficiaireFranceTravailPage client side', () => {
 
     it('devrait revenir sur la page des jeunes du conseiller', async () => {
       // Given
-      ;(createCompteJeuneFranceTravail as Mock<any>).mockResolvedValue(
+      ;(createCompteJeuneFranceTravail as jest.Mock).mockResolvedValue(
         uneBaseBeneficiaire()
       )
 
@@ -170,7 +172,7 @@ describe('CreationBeneficiaireFranceTravailPage client side', () => {
 
       expect(portefeuilleSetter).toHaveBeenCalledWith([
         ...portefeuille,
-        uneBaseBeneficiaire(),
+        unBeneficiaireWithActivity(),
       ])
       expect(alerteSetter).toHaveBeenCalledWith(
         'creationBeneficiaire',
@@ -181,7 +183,7 @@ describe('CreationBeneficiaireFranceTravailPage client side', () => {
 
     it("devrait afficher un message d'erreur en cas de création de compte en échec", async () => {
       // Given
-      ;(createCompteJeuneFranceTravail as Mock<any>).mockRejectedValue({
+      ;(createCompteJeuneFranceTravail as jest.Mock).mockRejectedValue({
         message: "un message d'erreur",
       })