From c6e455ce9b66911eafde7d78d80d6706368b08de Mon Sep 17 00:00:00 2001 From: Nicolas Dandrimont Date: Tue, 10 Oct 2017 16:37:15 +0200 Subject: [PATCH] debian.lister: use get_packages() method on the snapshot to list packages Allows to list the packages outside of lister context (e.g. for loader tests) --- swh/lister/debian/lister.py | 13 ++----------- 1 file changed, 2 insertions(+), 11 deletions(-) diff --git a/swh/lister/debian/lister.py b/swh/lister/debian/lister.py index f668f36..db61aec 100644 --- a/swh/lister/debian/lister.py +++ b/swh/lister/debian/lister.py @@ -177,18 +177,9 @@ class DebianLister(SWHListerHttpTransport, SWHListerBase): return def create_tasks_for_snapshot(self, snapshot): - packages = defaultdict(dict) - for area_snapshot in snapshot.areas: - area_name = area_snapshot.area.name - for package in area_snapshot.packages: - ref_name = '%s/%s' % (area_name, package.version) - packages[package.name][ref_name] = package.loader_dict() - tasks = [ - snapshot.task_for_package( - package_name, package_versions - ) - for package_name, package_versions in packages.items() + snapshot.task_for_package(name, versions) + for name, versions in snapshot.get_packages().items() ] return self.scheduler.create_tasks(tasks)