fix: Reset state with each new StacAPI instance [fix: #17]

Author: oliverroick

src/hooks/useStacSearch.test.ts

@@ -2,6 +2,7 @@ import fetch from 'jest-fetch-mock';
 import { renderHook, act } from '@testing-library/react-hooks';
 import useStacSearch from './useStacSearch';
 import StacApi from '../stac-api';
+import { Bbox } from '../types/stac';
 
 function parseRequestPayload(mockApiCall?: RequestInit) {
   if (!mockApiCall) {
@@ -391,4 +392,27 @@ describe('useStacSearch', () => {
     expect(fetch.mock.calls[1][1]?.method).toEqual('GET');
     expect(result.current.results).toEqual({ data: '12345' });
   });
+
+  it('should reset state with each new StacApi instance', async () => {
+    const bbox: Bbox = [-0.59, 51.24, 0.30, 51.74];
+    fetch.mockResponseOnce(JSON.stringify({ data: '12345' }));
+
+    const { result, rerender, waitForNextUpdate } = renderHook(
+      ({ stacApi }) => useStacSearch(stacApi), {
+        initialProps: { stacApi },
+      }
+    );
+
+    act(() => result.current.setBbox(bbox));
+    act(() => result.current.submit());
+    await waitForNextUpdate();
+
+    expect(result.current.results).toEqual({ data: '12345' });
+    expect(result.current.bbox).toEqual(bbox);
+
+    const newStac = new StacApi('https://otherstack.com');
+    rerender({ stacApi: newStac });
+    expect(result.current.results).toBeUndefined();
+    expect(result.current.bbox).toBeUndefined();
+  });
 });

src/hooks/useStacSearch.ts

@@ -1,4 +1,4 @@
-import { useCallback, useState, useMemo } from 'react';
+import { useCallback, useState, useMemo, useEffect } from 'react';
 import StacApi from '../stac-api';
 import debounce from '../utils/debounce';
 import type { ApiError, LoadingState } from '../types';
@@ -42,6 +42,19 @@ function useStacSearch(stacApi: StacApi): StacSearchHook {
   const [ nextPageConfig, setNextPageConfig ] = useState<Link>();
   const [ previousPageConfig, setPreviousPageConfig ] = useState<Link>();
 
+  const reset = () => {
+    setResults(undefined);
+    setBbox(undefined);
+    setCollections(undefined);
+    setDateRangeFrom('');
+    setDateRangeTo('');
+  };
+
+  /**
+   * Reset state when stacApu changes
+   */
+  useEffect(reset, [stacApi]);
+
   /**
    * Extracts the pagination config from the the links array of the items response
    */