Get all places from a list

2026-06-29 19:24:25 +02:00
parent d36bef185c
commit e461932aa9
3 changed files with 90 additions and 0 deletions
@@ -203,6 +203,64 @@ describe('Places Module', () => {
      });
    });

+    describe('getPlaces', () => {
+      it('returns all places from a list', async () => {
+        const mockList = {
+          id: 'hiking',
+          title: 'Hiking',
+          placeRefs: [
+            { id: 'place-1', geohash: 'u33dc0' },
+            { id: 'place-2', geohash: 'w1q789' },
+          ],
+        };
+
+        const mockPlace1 = { id: 'place-1', geohash: 'u33dc0', title: 'Hiking Trail' };
+        const mockPlace2 = { id: 'place-2', geohash: 'w1q789', title: 'Mountain Peak' };
+
+        mockClient.getObject.mockImplementation(async (path: string) => {
+          if (path === '_lists/hiking') return mockList;
+          if (path === 'u3/3d/place-1') return mockPlace1;
+          if (path === 'w1/q7/place-2') return mockPlace2;
+          return null;
+        });
+
+        const result = await lists.getPlaces('hiking');
+        expect(mockClient.getObject).toHaveBeenCalledWith('_lists/hiking');
+        expect(mockClient.getObject).toHaveBeenCalledWith('u3/3d/place-1');
+        expect(mockClient.getObject).toHaveBeenCalledWith('w1/q7/place-2');
+        expect(result).toEqual([mockPlace1, mockPlace2]);
+      });
+
+      it('throws an error if the list does not exist', async () => {
+        mockClient.getObject.mockResolvedValue(null);
+
+        await expect(lists.getPlaces('non-existent')).rejects.toThrow('List not found: non-existent');
+        expect(mockClient.getObject).toHaveBeenCalledWith('_lists/non-existent');
+      });
+
+      it('filters out any places that failed to load or are missing', async () => {
+        const mockList = {
+          id: 'hiking',
+          title: 'Hiking',
+          placeRefs: [
+            { id: 'place-1', geohash: 'u33dc0' },
+            { id: 'place-2', geohash: 'w1q789' },
+          ],
+        };
+
+        const mockPlace1 = { id: 'place-1', geohash: 'u33dc0', title: 'Place 1' };
+
+        mockClient.getObject.mockImplementation(async (path: string) => {
+          if (path === '_lists/hiking') return mockList;
+          if (path === 'u3/3d/place-1') return mockPlace1;
+          return null; // place-2 is missing
+        });
+
+        const result = await lists.getPlaces('hiking');
+        expect(result).toEqual([mockPlace1]);
+      });
+    });
+
    describe('create', () => {
      it('stores a new list when none exists', async () => {
        const now = '2023-01-01T00:00:00.000Z';