diff --git a/swh/lister/npm/lister.py b/swh/lister/npm/lister.py index 5931c8d..dae76bf 100644 --- a/swh/lister/npm/lister.py +++ b/swh/lister/npm/lister.py @@ -60,7 +60,8 @@ class NpmListerBase(IndexingHttpLister): package_name = kwargs.get('name') package_metadata_url = kwargs.get('html_url') return create_task_dict(task_type, task_policy, - package_name, origin_url, + package_name=package_name, + package_url=origin_url, package_metadata_url=package_metadata_url) def request_headers(self): diff --git a/swh/lister/npm/tests/test_lister.py b/swh/lister/npm/tests/test_lister.py index 33e0ea7..9888795 100644 --- a/swh/lister/npm/tests/test_lister.py +++ b/swh/lister/npm/tests/test_lister.py @@ -62,16 +62,16 @@ def check_tasks(tasks: List[Any]): assert row['type'] == 'load-npm' # arguments check args = row['arguments']['args'] - assert len(args) == 2 - - package = args[0] - url = args[1] - assert url == 'https://www.npmjs.com/package/%s' % package + assert len(args) == 0 # kwargs kwargs = row['arguments']['kwargs'] + assert len(kwargs) == 3 + package_name = kwargs['package_name'] + package_url = kwargs['package_url'] + assert package_url == 'https://www.npmjs.com/package/%s' % package_name meta_url = kwargs['package_metadata_url'] - assert meta_url == 'https://replicate.npmjs.com/%s' % package + assert meta_url == 'https://replicate.npmjs.com/%s' % package_name assert row['policy'] == 'recurring' assert row['priority'] is None