fix: Send selection collections as undefined is array is empty [fix #16]

Author: oliverroick

src/hooks/useStacSearch.test.ts

@@ -62,6 +62,22 @@ describe('useStacSearch', () => {
     expect(result.current.results).toEqual({ data: '12345' });
   });
 
+  it('clears collections when array is empty', async () => {
+    fetch.mockResponseOnce(JSON.stringify({ data: '12345' }));
+
+    const { result, waitForNextUpdate } = renderHook(
+      () => useStacSearch(stacApi)
+    );
+
+    act(() => result.current.setCollections([]));
+    act(() => result.current.submit());
+    await waitForNextUpdate();
+
+    const postPayload = parseRequestPayload(fetch.mock.calls[0][1]);
+    expect(postPayload).toEqual({ limit: 25 });
+    expect(result.current.results).toEqual({ data: '12345' });
+  });
+
   it('includes date range in search', async () => {
     fetch.mockResponseOnce(JSON.stringify({ data: '12345' }));
 

src/stac-api/index.ts

@@ -1,5 +1,5 @@
 import type { ApiError, GenericObject } from '../types';
-import type { Bbox, SearchPayload, DateRange } from '../types/stac';
+import type { Bbox, SearchPayload, DateRange, CollectionIdList } from '../types/stac';
 
 type RequestPayload = SearchPayload;
 type FetchOptions = {
@@ -36,6 +36,10 @@ class StacApi {
     return sortedBbox;
   }
 
+  makeCollectionsPayload(collections?: CollectionIdList) {
+    return collections?.length ? collections : undefined;
+  }
+
   makeDatetimePayload(dateRange?: DateRange): string | undefined {
     if (!dateRange) {
       return undefined;
@@ -88,9 +92,10 @@ class StacApi {
   }
 
   search(payload: SearchPayload, headers = {}): Promise<Response> {
-    const { bbox, dateRange, ...restPayload } = payload;
+    const { bbox, dateRange, collections, ...restPayload } = payload;
     const requestPayload = {
       ...restPayload,
+      collections: this.makeCollectionsPayload(collections),
       bbox: this.fixBboxCoordinateOrder(bbox),
       datetime: this.makeDatetimePayload(dateRange),
       limit: 25