Merge branch 'kl/scrum-175-tests' of https://github.com/UTSC-CSCC01-Software-Engineering-I/term-group-project-c01w25-project-course-matrix into kl/scrum-175-tests

Author: kevin-lann

course-matrix/frontend/__tests__/unit-tests/useClickOutside.test.tsx

@@ -85,7 +85,16 @@ describe("useClickOutside", () => {
 
   test("should remove event listener when isActive changes from true to false", () => {
     const { rerender } = render(
-      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />
+      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />,
+    );
+
+    rerender(
+      <TestComponent isActive={false} onClickOutside={mockOnClickOutside} />,
+    );
+
+    expect(document.removeEventListener).toHaveBeenCalledWith(
+      "mousedown",
+      expect.any(Function),
     );
 
     rerender(
@@ -100,7 +109,7 @@ describe("useClickOutside", () => {
 
   test("should call onClickOutside when clicking outside the ref element", () => {
     const { getByTestId } = render(
-      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />
+      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />,
     );
 
     // Simulate clicking outside
@@ -111,7 +120,7 @@ describe("useClickOutside", () => {
 
   test("should not call onClickOutside when clicking inside the ref element", () => {
     const { getByTestId } = render(
-      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />
+      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />,
     );
 
     // Simulate clicking inside
@@ -137,7 +146,7 @@ describe("useClickOutside", () => {
 
   test("should clean up event listener when unmounting", () => {
     const { unmount } = render(
-      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />
+      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />,
     );
 
     unmount();
@@ -148,7 +157,7 @@ describe("useClickOutside", () => {
   test("should re-attach event listener when dependencies change", () => {
     const newMockCallback = jest.fn();
     const { rerender } = render(
-      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />
+      <TestComponent isActive={true} onClickOutside={mockOnClickOutside} />,
     );
 
     // First, verify initial setup

course-matrix/frontend/__tests__/unit-tests/useDebounce.test.ts

@@ -29,7 +29,7 @@ describe("useDebounceValue", () => {
     const initialValue = "initial";
     const { result, rerender } = renderHook(
       ({ value, interval }) => useDebounceValue(value, interval),
-      { initialProps: { value: initialValue, interval: 500 } }
+      { initialProps: { value: initialValue, interval: 500 } },
     );
 
     expect(result.current).toBe(initialValue);
@@ -62,7 +62,7 @@ describe("useDebounceValue", () => {
     const initialValue = "initial";
     const { result, rerender } = renderHook(
       ({ value, interval }) => useDebounceValue(value, interval),
-      { initialProps: { value: initialValue, interval: 500 } }
+      { initialProps: { value: initialValue, interval: 500 } },
     );
 
     // Change the value once
@@ -100,7 +100,7 @@ describe("useDebounceValue", () => {
     const initialValue = "initial";
     const { result, rerender } = renderHook(
       ({ value, interval }) => useDebounceValue(value, interval),
-      { initialProps: { value: initialValue, interval: 500 } }
+      { initialProps: { value: initialValue, interval: 500 } },
     );
 
     // Change value and interval
@@ -128,7 +128,7 @@ describe("useDebounceValue", () => {
     const initialObject = { name: "John" };
     const { result: objectResult, rerender: objectRerender } = renderHook(
       ({ value, interval }) => useDebounceValue(value, interval),
-      { initialProps: { value: initialObject, interval: 200 } }
+      { initialProps: { value: initialObject, interval: 200 } },
     );
 
     const newObject = { name: "Jane" };
@@ -143,7 +143,7 @@ describe("useDebounceValue", () => {
     // Test with number
     const { result: numberResult, rerender: numberRerender } = renderHook(
       ({ value, interval }) => useDebounceValue(value, interval),
-      { initialProps: { value: 1, interval: 200 } }
+      { initialProps: { value: 1, interval: 200 } },
     );
 
     numberRerender({ value: 2, interval: 200 });