diff --git a/lib/workers/repository/extract/manager-files.js b/lib/workers/repository/extract/manager-files.js index f54d3ed3dffd4ee6dbeb0454e2f0b9bd0df53799..e91302b6207f1193d5711e13e32cb6d82a4e0cd9 100644 --- a/lib/workers/repository/extract/manager-files.js +++ b/lib/workers/repository/extract/manager-files.js @@ -41,18 +41,22 @@ async function getManagerPackageFiles(config, managerConfig) { const packageFiles = []; for (const packageFile of matchedFiles) { const content = await platform.getFile(packageFile); - const res = await extractDependencies( - manager, - content, - packageFile, - config - ); - if (res) { - packageFiles.push({ - packageFile, + if (content) { + const res = await extractDependencies( manager, - ...res, - }); + content, + packageFile, + config + ); + if (res) { + packageFiles.push({ + packageFile, + manager, + ...res, + }); + } + } else { + logger.warn({ packageFile }, 'Found not content for packageFile'); } } await postExtract(manager, packageFiles); diff --git a/test/workers/repository/extract/manager-files.spec.js b/test/workers/repository/extract/manager-files.spec.js index c67ef1d3d996e68c13981b313939281cef16eb0a..eb5b0428aaffbfaa3e5bccaf77d9dd8e5425171b 100644 --- a/test/workers/repository/extract/manager-files.spec.js +++ b/test/workers/repository/extract/manager-files.spec.js @@ -24,9 +24,16 @@ describe('workers/repository/extract/manager-files', () => { const res = await getManagerPackageFiles(config, managerConfig); expect(res).toHaveLength(0); }); + it('skips files if null content returned', async () => { + const managerConfig = { manager: 'npm', enabled: true }; + fileMatch.getMatchingFiles.mockReturnValue(['package.json']); + const res = await getManagerPackageFiles(config, managerConfig); + expect(res).toHaveLength(0); + }); it('returns files', async () => { const managerConfig = { manager: 'npm', enabled: true }; fileMatch.getMatchingFiles.mockReturnValue(['package.json']); + platform.getFile.mockReturnValue('{}'); npm.extractDependencies = jest.fn(() => ({ some: 'result' })); const res = await getManagerPackageFiles(config, managerConfig); expect(res).toMatchSnapshot();