[#159] Can untoggle every calendars (#161)

Co-authored-by: Camille Moussu <cmoussu@linagora.com>

__test__/components/Calendar.test.tsx

@@ -250,4 +250,21 @@ describe("calendar Availability search", () => {
 
     expect(spy).not.toHaveBeenCalledWith();
   });
+
+  it("BUGFIX: can untoggle all personnal calendars", () => {
+    renderWithProviders(<CalendarTestWrapper />, {
+      user: preloadedState.user,
+      calendars: {
+        list: { "user1/cal1": preloadedState.calendars.list["user1/cal1"] },
+        pending: false,
+      },
+    });
+
+    const checkbox = screen.getByLabelText("Calendar personnal");
+    expect(checkbox).toBeChecked();
+
+    fireEvent.click(checkbox); // toggle off
+
+    expect(checkbox).not.toBeChecked();
+  });
 });

src/components/Calendar/Calendar.tsx

@@ -100,18 +100,17 @@ export default function CalendarApp({
   const fetchedRangesRef = useRef<Record<string, string>>({});
 
   // Auto-select personal calendars when first loaded
+  const initialLoadRef = useRef(true);
+
   useEffect(() => {
-    if (
+    if (initialLoadRef.current && Object.keys(calendars).length > 0 && userId) {
-      Object.keys(calendars).length > 0 &&
-      userId &&
-      selectedCalendars.length === 0
-    ) {
       const personalCalendarIds = Object.keys(calendars).filter(
         (id) => id.split("/")[0] === userId
       );
       setSelectedCalendars(personalCalendarIds);
+      initialLoadRef.current = false;
     }
-  }, [calendars, userId, selectedCalendars.length]);
+  }, [calendars, userId]);
 
   const calendarRange = getCalendarRange(selectedDate);
 

src/components/Calendar/utils/calendarUtils.ts

@@ -94,7 +94,7 @@ export const updateCalsDetails = (
   });
 
   if (rangeKey !== previousRangeKey) {
-    selectedCalendars.forEach((id) => {
+    selectedCalendars?.forEach((id) => {
       dispatch(
         getCalendarDetailAsync({
           calId: id,