feat(us-3078): ajouter des testes pour le bouton

SocialGouv · Dec 11, 2024 · addd844 · addd844
1 parent ef9e898
commit addd844
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 18 deletions.
diff --git a/components/jeune/TableauBeneficiaires.tsx b/components/jeune/TableauBeneficiaires.tsx
@@ -8,10 +8,7 @@ import EmptyState from 'components/EmptyState'
 import { IllustrationName } from 'components/ui/IllustrationComponent'
 import Pagination from 'components/ui/Table/Pagination'
 import Table from 'components/ui/Table/Table'
-import {
-  BaseBeneficiaire,
-  BeneficiaireAvecInfosComplementaires,
-} from 'interfaces/beneficiaire'
+import { BeneficiaireAvecInfosComplementaires } from 'interfaces/beneficiaire'
 import { estMilo } from 'interfaces/conseiller'
 import { useConseiller } from 'utils/conseiller/conseillerContext'
 import { toShortDate } from 'utils/date'
@@ -42,7 +39,8 @@ function TableauBeneficiaires(
   const DEBUT_PERIODE = DateTime.now().startOf('week')
   const FIN_PERIODE = DateTime.now().endOf('week')
   const [triDerniereActiviter, setTriDerniereActiviter] = useState(true)
-  const [triBeneficiaires, setTriBeneficiaires] = useState(beneficiairesFiltres)
+  const [filtreBeneficiaires, setFiltreBeneficiaires] =
+    useState(beneficiairesFiltres)
 
   const trierParDerniereActivite = () => {
     const nouvelleTri = !triDerniereActiviter
@@ -58,19 +56,19 @@ function TableauBeneficiaires(
         : dateA.diff(dateB).milliseconds
     })
 
-    setTriBeneficiaires(triBeneficiaires)
+    setFiltreBeneficiaires(triBeneficiaires)
     setPage(1)
     setTriDerniereActiviter(nouvelleTri)
   }
 
   useEffect(() => {
-    setTriBeneficiaires(beneficiairesFiltres)
+    setFiltreBeneficiaires(beneficiairesFiltres)
     setPage(1)
   }, [beneficiairesFiltres])
 
   return (
     <>
-      {triBeneficiaires.length === 0 && (
+      {filtreBeneficiaires.length === 0 && (
         <EmptyState
           shouldFocus={true}
           illustrationName={IllustrationName.People}
@@ -79,7 +77,7 @@ function TableauBeneficiaires(
         />
       )}
 
-      {triBeneficiaires.length > 0 && (
+      {filtreBeneficiaires.length > 0 && (
         <>
           <h2 className='text-m-bold mb-2 text-center text-grey_800'>
             Semaine du {toShortDate(DEBUT_PERIODE)} au{' '}
@@ -94,11 +92,7 @@ function TableauBeneficiaires(
                 ? 'Trier par dernière activité ordre anticronologique'
                 : 'Trier par dernière activité ordre cronologique'
             }
-            aria-label={
-              triDerniereActiviter
-                ? 'Trier par dernière activité ordre anticronologique'
-                : 'Trier par dernière activité ordre cronologique'
-            }
+            aria-label='Trier par dernière activité'
             type='button'
           >
             Trier par dernière activité
@@ -109,21 +103,21 @@ function TableauBeneficiaires(
             ref={ref}
             caption={{
               text: 'Liste des bénéficiaires',
-              count: total === triBeneficiaires.length ? total : undefined,
+              count: total === filtreBeneficiaires.length ? total : undefined,
               visible: true,
             }}
           >
             {estMilo(conseiller) && (
               <TableauBeneficiairesMilo
-                beneficiairesFiltres={triBeneficiaires}
+                beneficiairesFiltres={filtreBeneficiaires}
                 page={page}
                 total={total}
               />
             )}
 
             {!estMilo(conseiller) && (
               <TableauBeneficiairesPasMilo
-                beneficiairesFiltres={triBeneficiaires}
+                beneficiairesFiltres={filtreBeneficiaires}
                 page={page}
                 total={total}
               />

diff --git a/tests/pages/PortefeuillePage.test.tsx b/tests/pages/PortefeuillePage.test.tsx
@@ -1,4 +1,4 @@
-import { act, screen, within } from '@testing-library/react'
+import { act, fireEvent, screen, within } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
 import { AxeResults } from 'axe-core'
 import { axe } from 'jest-axe'
@@ -255,6 +255,25 @@ describe('PortefeuillePage client side', () => {
         screen.getByText(`Semaine du ${DEBUT_PERIODE} au ${FIN_PERIODE}`)
       ).toBeInTheDocument()
     })
+    it('parmet de trier bénéficiaire par dernière activité ', () => {
+      //given
+      //when
+      const button = screen.getByRole('button', {
+        name: /Trier par dernière activité/i,
+      })
+
+      //then
+      expect(button).toHaveAttribute(
+        'title',
+        'Trier par dernière activité ordre anticronologique'
+      )
+      fireEvent.click(button)
+
+      expect(button).toHaveAttribute(
+        'title',
+        'Trier par dernière activité ordre cronologique'
+      )
+    })
 
     it("affiche la colonne nombre d'actions des bénéficiaires", () => {
       // Then