fix: Remove dependency on collections object in useCollections effect…

… hook

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@developmentseed/stac-react",
-  "version": "0.1.0-alpha.8",
+  "version": "0.1.0-alpha.8.1",
   "description": "React components and hooks for building STAC-API front-ends",
   "repository": "[email protected]:developmentseed/stac-react.git",
   "author": "Oliver Roick <[email protected]>",

src/hooks/useCollections.test.ts

@@ -24,6 +24,27 @@ describe('useCollections', () => {
     expect(result.current.state).toEqual('IDLE');
   });
 
+  it('reloads collections', async () => {
+    fetch
+      .mockResponseOnce(JSON.stringify({ links: [] }), { url: 'https://fake-stac-api.net' })
+      .mockResponseOnce(JSON.stringify({ data: 'original' }))
+      .mockResponseOnce(JSON.stringify({ data: 'reloaded' }));
+
+    const { result, waitForNextUpdate } = renderHook(
+      () => useCollections(),
+      { wrapper }
+    );
+    await waitForNextUpdate();
+    await waitForNextUpdate();
+    expect(result.current.collections).toEqual({ data: 'original' });
+
+    expect(result.current.state).toEqual('IDLE');
+    act(() => result.current.reload());
+
+    await waitForNextUpdate();
+    expect(result.current.collections).toEqual({ data: 'reloaded' });
+  });
+
   it('handles error with JSON response', async () => {
     fetch
       .mockResponseOnce(JSON.stringify({ links: [] }), { url: 'https://fake-stac-api.net' })
@@ -61,25 +82,4 @@ describe('useCollections', () => {
       detail: 'Wrong query'
     });
   });
-
-  it('reloads collections', async () => {
-    fetch
-      .mockResponseOnce(JSON.stringify({ links: [] }), { url: 'https://fake-stac-api.net' })
-      .mockResponseOnce(JSON.stringify({ data: 'original' }))
-      .mockResponseOnce(JSON.stringify({ data: 'reloaded' }));
-
-    const { result, waitForNextUpdate } = renderHook(
-      () => useCollections(),
-      { wrapper }
-    );
-    await waitForNextUpdate();
-    await waitForNextUpdate();
-    expect(result.current.collections).toEqual({ data: 'original' });
-
-    expect(result.current.state).toEqual('IDLE');
-    act(() => result.current.reload());
-
-    await waitForNextUpdate();
-    expect(result.current.collections).toEqual({ data: 'reloaded' });
-  });
 });

src/hooks/useCollections.ts

@@ -36,11 +36,11 @@ function useCollections(): StacCollectionsHook {
 
   useEffect(
     () => {
-      if (stacApi && !collections) {
+      if (stacApi) {
         getCollections();
       }
     },
-    [getCollections, stacApi, collections]
+    [getCollections, stacApi]
   );
 
   return {