(PC-30848)[PRO] fix: Check all offers when clicking on Tout Sélection…

…ner in the indifv offers tables even if they are isEditable: true
pass-culture · Jul 15, 2024 · 248bf82 · 248bf82
1 parent 9495baa
commit 248bf82
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 9 deletions.
diff --git a/pro/src/screens/Offers/Offers.tsx b/pro/src/screens/Offers/Offers.tsx
@@ -136,8 +136,7 @@ export const Offers = ({
 
   const areAllIndividualOffersSelected =
     selectedIndividualOffers.length > 0 &&
-    selectedIndividualOffers.length ===
-      offers.filter((offer) => offer.isEditable).length
+    selectedIndividualOffers.length === offers.length
 
   function clearSelectedOfferIds() {
     setSelectedCollectiveOffers([])
@@ -152,11 +151,7 @@ export const Offers = ({
           : offers.filter((offer) => offer.isEditable)
       )
     } else {
-      setSelectedIndividualOffers(
-        areAllIndividualOffersSelected
-          ? []
-          : offers.filter((offer) => offer.isEditable)
-      )
+      setSelectedIndividualOffers(areAllIndividualOffersSelected ? [] : offers)
     }
   }
 

diff --git a/pro/src/screens/Offers/__specs__/Offers.spec.tsx b/pro/src/screens/Offers/__specs__/Offers.spec.tsx
@@ -553,8 +553,8 @@ describe('screen Offers', () => {
 
       expect(firstOfferCheckbox).toBeChecked()
       expect(secondOfferCheckbox).toBeChecked()
-      expect(thirdOfferCheckbox).not.toBeChecked()
-      expect(fourthOfferCheckbox).not.toBeChecked()
+      expect(thirdOfferCheckbox).toBeChecked()
+      expect(fourthOfferCheckbox).toBeChecked()
 
       await userEvent.click(screen.getByLabelText('Tout désélectionner'))