fix(cdnjs): handle empty 200 ok response

renovatebot · Apr 23, 2020 · e79319d · e79319d
1 parent f61c416
commit e79319d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/lib/datasource/cdnjs/index.spec.ts b/lib/datasource/cdnjs/index.spec.ts
@@ -40,6 +40,12 @@ describe('datasource/cdnjs', () => {
       got.mockRejectedValueOnce({ statusCode: 404 });
       expect(await getReleases({ lookupName: 'foo/bar' })).toBeNull();
     });
+    it('returns null for empty 200 OK', async () => {
+      got.mockResolvedValueOnce({ body: {} });
+      expect(
+        await getReleases({ lookupName: 'doesnotexist/doesnotexist' })
+      ).toBeNull();
+    });
     it('throws for 401', async () => {
       got.mockRejectedValueOnce({ statusCode: 401 });
       await expect(getReleases({ lookupName: 'foo/bar' })).rejects.toThrowError(

diff --git a/lib/datasource/cdnjs/index.ts b/lib/datasource/cdnjs/index.ts
@@ -37,6 +37,9 @@ export async function getReleases({
       lookup: library,
       cb: downloadLibrary,
     });
+    if (!assets) {
+      return null;
+    }
     const assetName = lookupName.replace(`${library}/`, '');
     const releases = assets
       .filter(({ files }) => files.includes(assetName))