diff --git a/lib/manager/pip_requirements/extract.js b/lib/manager/pip_requirements/extract.js index 107c283493215f074f07637272eba4e6c8b3715f..caae795cd9c520c2cf20ccb41a916ba2643ad27e 100644 --- a/lib/manager/pip_requirements/extract.js +++ b/lib/manager/pip_requirements/extract.js @@ -56,14 +56,15 @@ function extractPackageFile(content) { ) { dep.fromVersion = currentValue.replace(/^==/, ''); } - if (registryUrls) { - dep.registryUrls = registryUrls; - } return dep; }) .filter(Boolean); if (!deps.length) { return null; } - return { deps }; + const res = { deps }; + if (registryUrls) { + res.registryUrls = registryUrls; + } + return res; } diff --git a/test/manager/pip_requirements/__snapshots__/extract.spec.js.snap b/test/manager/pip_requirements/__snapshots__/extract.spec.js.snap index 52c89fceaffc89be50649ae8f77fd3a625040287..5b093bc9daae19d47f06afd6c57f851869428669 100644 --- a/test/manager/pip_requirements/__snapshots__/extract.spec.js.snap +++ b/test/manager/pip_requirements/__snapshots__/extract.spec.js.snap @@ -1,38 +1,34 @@ // Jest Snapshot v1, https://goo.gl/fbAQLP exports[`lib/manager/pip_requirements/extract extractPackageFile() extracts dependencies 1`] = ` -Array [ - Object { - "currentValue": "==0.3.1", - "datasource": "pypi", - "depName": "some-package", - "fromVersion": "0.3.1", - "lineNumber": 2, - "registryUrls": Array [ - "http://example.com/private-pypi/", - ], - }, - Object { - "currentValue": "==1.0.0", - "datasource": "pypi", - "depName": "some-other-package", - "fromVersion": "1.0.0", - "lineNumber": 3, - "registryUrls": Array [ - "http://example.com/private-pypi/", - ], - }, - Object { - "currentValue": "==1.9", - "datasource": "pypi", - "depName": "not_semver", - "fromVersion": "1.9", - "lineNumber": 4, - "registryUrls": Array [ - "http://example.com/private-pypi/", - ], - }, -] +Object { + "deps": Array [ + Object { + "currentValue": "==0.3.1", + "datasource": "pypi", + "depName": "some-package", + "fromVersion": "0.3.1", + "lineNumber": 2, + }, + Object { + "currentValue": "==1.0.0", + "datasource": "pypi", + "depName": "some-other-package", + "fromVersion": "1.0.0", + "lineNumber": 3, + }, + Object { + "currentValue": "==1.9", + "datasource": "pypi", + "depName": "not_semver", + "fromVersion": "1.9", + "lineNumber": 4, + }, + ], + "registryUrls": Array [ + "http://example.com/private-pypi/", + ], +} `; exports[`lib/manager/pip_requirements/extract extractPackageFile() extracts multiple dependencies 1`] = ` diff --git a/test/manager/pip_requirements/extract.spec.js b/test/manager/pip_requirements/extract.spec.js index 749ac0ae2c458e945521b0423585d6cf647be848..60c7adc012aecf472edb38ac5a587ebc3da311c4 100644 --- a/test/manager/pip_requirements/extract.spec.js +++ b/test/manager/pip_requirements/extract.spec.js @@ -31,9 +31,10 @@ describe('lib/manager/pip_requirements/extract', () => { expect(extractPackageFile('nothing here', config)).toBeNull(); }); it('extracts dependencies', () => { - const res = extractPackageFile(requirements1, config).deps; + const res = extractPackageFile(requirements1, config); expect(res).toMatchSnapshot(); - expect(res).toHaveLength(3); + expect(res.registryUrls).toEqual(['http://example.com/private-pypi/']); + expect(res.deps).toHaveLength(3); }); it('extracts multiple dependencies', () => { const res = extractPackageFile(requirements2, config).deps;