Skip to content
Snippets Groups Projects
Commit fd85d068 authored by Rhys Arkins's avatar Rhys Arkins
Browse files

fix: trim urls from getPkgReleases

Closes #3169
parent 67cbbba8
No related branches found
No related tags found
No related merge requests found
......@@ -91,7 +91,9 @@ function addMetaData(dep, datasource, lookupName) {
// Clean up any empty urls
const urls = ['homepage', 'sourceUrl', 'changelogUrl'];
for (const url of urls) {
if (!is.nonEmptyString(dep[url])) {
if (is.nonEmptyString(dep[url])) {
dep[url] = dep[url].trim();
} else {
delete dep[url];
}
}
......
......@@ -52,4 +52,14 @@ describe('datasource/index', () => {
expect(res).toMatchSnapshot();
expect(res.sourceUrl).toBeDefined();
});
it('trims sourceUrl', async () => {
npmDatasource.getPkgReleases.mockReturnValue({
sourceUrl: ' https://abc.com',
});
const res = await datasource.getPkgReleases({
datasource: 'npm',
depName: 'abc',
});
expect(res.sourceUrl).toEqual('https://abc.com');
});
});
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment