Merge pull request #50 from getmetal/task/sp/consistent-get-many

Update `getMany` to return array consistently

src/metal.ts

@@ -205,6 +205,10 @@ export class Metal implements Client {
       },
     })
 
+    if (!Array.isArray(data)) {
+      return [data]
+    }
+
     return data
   }
 

tests/metal.test.ts

@@ -537,7 +537,7 @@ describe('Metal', () => {
       await expect(result).rejects.toThrowError('indexId required')
     })
 
-    it('should get one by id', async () => {
+    it('should get multiple by id', async () => {
       const metal = new Metal(API_KEY, CLIENT_ID, 'index-id')
 
       fetchMock.mockImplementationOnce(
@@ -556,6 +556,25 @@ describe('Metal', () => {
         }
       )
     })
+
+    it('should get one by id', async () => {
+      const metal = new Metal(API_KEY, CLIENT_ID, 'index-id')
+
+      fetchMock.mockImplementationOnce(
+        getMockRes({ id: 'megadeth', metadata: { vocalist: 'Dave Mustain' } })
+      )
+
+      const res = await metal.getMany(['megadeth'])
+
+      expect(Array.isArray(res)).toBe(true)
+
+      expect(fetchMock).toHaveBeenCalledWith(
+        `https://api.getmetal.io/v1/indexes/index-id/documents/megadeth`,
+        {
+          headers: HEADERS,
+        }
+      )
+    })
   })
 
   describe('deleteOne()', () => {