test: getData & onChange changes

Author: andipaetzold

src/internal/useGet.spec.ts

@@ -3,8 +3,8 @@ import { newPromise, newSymbol } from "../__testfixtures__";
 import { useGet } from "./useGet";
 import { it, expect, beforeEach, describe, vi } from "vitest";
 
-const result1 = newSymbol("Result 1");
-const result2 = newSymbol("Result 2");
+const result1 = newSymbol<string>("Result 1");
+const result2 = newSymbol<string>("Result 2");
 const error = newSymbol("Error");
 
 const refA1 = newSymbol("Ref A1");
@@ -126,3 +126,24 @@ describe("when ref changes", () => {
         });
     });
 });
+
+it("refetches if `getData` changes", async () => {
+    const getData1 = vi.fn().mockResolvedValue(result1);
+    const getData2 = vi.fn().mockResolvedValue(result2);
+
+    const { result, rerender } = renderHook(({ getData }) => useGet(refA1, getData, isEqual), {
+        initialProps: { getData: getData1 },
+    });
+
+    await waitFor(() => {
+        expect(result.current).toStrictEqual([result1, false, undefined]);
+    });
+    expect(getData1).toHaveBeenCalledTimes(1);
+
+    rerender({ getData: getData2 });
+
+    await waitFor(() => {
+        expect(result.current).toStrictEqual([result2, false, undefined]);
+    });
+    expect(getData2).toHaveBeenCalledTimes(1);
+});

src/internal/useListen.spec.ts

@@ -140,3 +140,20 @@ it("should return emitted error", () => {
     act(() => setValue(result2));
     expect(result.current).toStrictEqual([result2, false, undefined]);
 });
+
+it("resubscribes if `onChange` changes", async () => {
+    const onChange1 = vi.fn().mockReturnValue(onChangeUnsubscribe);
+
+    const { result, rerender } = renderHook(({ onChange }) => useListen(refA1, onChange, isEqual, LoadingState), {
+        initialProps: { onChange: onChange1 },
+    });
+    const setValue1 = onChange1.mock.calls[0][1];
+    act(() => setValue1(result1));
+    expect(result.current).toStrictEqual([result1, false, undefined]);
+
+    const onChange2 = vi.fn().mockReturnValue(onChangeUnsubscribe);
+    rerender({ onChange: onChange2 });
+    const setValue2 = onChange1.mock.calls[0][1];
+    act(() => setValue2(result2));
+    expect(result.current).toStrictEqual([result2, false, undefined]);
+});