refactor!: rename reload to refetch in hooks API

BREAKING CHANGE: reload() method renamed to refetch() in useCollection,
useCollections, and useItem hooks.

- Rename reload() to refetch() in useCollection, useCollections, and useItem
- Update all tests to use refetch() instead of reload()
- Maintain proper async typing with Promise<QueryObserverResult>

Using refetch aligns with React Query's naming convention and makes the API
more consistent with the underlying library. The function returns the Promise
from React Query's refetch directly.

Migration: Replace .reload() with .refetch() in your code.

Author: AliceR

src/hooks/useCollection.test.ts

@@ -84,7 +84,7 @@ describe('useCollection', () => {
     );
 
     await act(async () => {
-      await result.current.reload();
+      await result.current.refetch();
     });
 
     await waitFor(() =>

src/hooks/useCollection.ts

@@ -9,8 +9,8 @@ type StacCollectionHook = {
   collection?: Collection;
   isLoading: boolean;
   isFetching: boolean;
+  refetch: () => Promise<QueryObserverResult<Collection, ApiErrorType>>;
   error?: ApiErrorType;
-  reload: () => Promise<QueryObserverResult<Collection, ApiErrorType>>;
 };
 
 function useCollection(collectionId: string): StacCollectionHook {
@@ -49,8 +49,8 @@ function useCollection(collectionId: string): StacCollectionHook {
     collection,
     isLoading,
     isFetching,
+    refetch,
     error: error as ApiErrorType,
-    reload: refetch,
   };
 }
 

src/hooks/useCollections.test.ts

@@ -33,7 +33,7 @@ describe('useCollections', () => {
     await waitFor(() => expect(result.current.isLoading).toEqual(false));
 
     await act(async () => {
-      await result.current.reload();
+      await result.current.refetch();
     });
 
     await waitFor(() => expect(result.current.collections).toEqual({ data: 'reloaded' }));

src/hooks/useCollections.ts

@@ -7,7 +7,7 @@ import { useStacApiContext } from '../context/useStacApiContext';
 
 type StacCollectionsHook = {
   collections?: CollectionsResponse;
-  reload: () => Promise<QueryObserverResult<CollectionsResponse, ApiErrorType>>;
+  refetch: () => Promise<QueryObserverResult<CollectionsResponse, ApiErrorType>>;
   isLoading: boolean;
   isFetching: boolean;
   error?: ApiErrorType;
@@ -47,7 +47,7 @@ function useCollections(): StacCollectionsHook {
 
   return {
     collections,
-    reload: refetch,
+    refetch,
     isLoading,
     isFetching,
     error: error as ApiErrorType,

src/hooks/useItem.test.ts

@@ -69,7 +69,7 @@ describe('useItem', () => {
     await waitFor(() => expect(result.current.item).toEqual({ id: 'abc' }));
 
     await act(async () => {
-      await result.current.reload();
+      await result.current.refetch();
     });
 
     await waitFor(() => expect(result.current.item).toEqual({ id: 'abc', description: 'Updated' }));

src/hooks/useItem.ts

@@ -10,7 +10,7 @@ type ItemHook = {
   isLoading: boolean;
   isFetching: boolean;
   error?: ApiErrorType;
-  reload: () => Promise<QueryObserverResult<Item, ApiErrorType>>;
+  refetch: () => Promise<QueryObserverResult<Item, ApiErrorType>>;
 };
 
 function useItem(url: string): ItemHook {
@@ -50,7 +50,7 @@ function useItem(url: string): ItemHook {
     isLoading,
     isFetching,
     error: error as ApiErrorType,
-    reload: refetch,
+    refetch,
   };
 }