Skip to content

Commit 6d1f50a

Browse files
committed
Enable all scans in test_full_summary_by_facet and Fix test data for the same
Signed-off-by: inishchith <[email protected]>
1 parent 3d1dbd7 commit 6d1f50a

File tree

4 files changed

+4197
-1490
lines changed

4 files changed

+4197
-1490
lines changed

src/summarycode/summarizer.py

Lines changed: 10 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -503,29 +503,23 @@ def package_summarizer(resource, children, keep_details=False):
503503
logger_debug('package_summarizer: for:', resource,
504504
'packages are:', packs)
505505

506-
package_purls = []
507-
for package in packages:
508-
if package.get("purl", False):
509-
package_purls.append(package["purl"])
510-
511-
# Collect direct children packages summary
512-
for child in children:
513-
child_summaries = get_resource_summary(child, key='packages', as_attribute=False) or []
514-
for child_summary in child_summaries:
515-
values = [child_summary['value']] * child_summary['count']
516-
package_purls.extend(values)
506+
package_urls = []
507+
for package in packages:
508+
if package.get('purl', False):
509+
package_urls.append(package['purl'])
517510

518511
# summarize proper
519-
packages_counter = summarize_licenses(package_purls)
512+
packages_counter = summarize_packages(package_urls)
520513
summarized = sorted_counter(packages_counter)
521-
set_resource_summary(resource, key="packages", value=summarized, as_attribute=keep_details)
514+
set_resource_summary(resource, key='packages', value=summarized, as_attribute=keep_details)
522515

523516
return summarized
524517

525518

526-
def summarize_packages(package_purls):
519+
def summarize_packages(package_urls):
527520
"""
528-
Given a list of package purls, return a mapping of {expression: count
521+
Given a list of package urls, return a mapping of {expression: count
529522
of occurences}
530523
"""
531-
return Counter(package_purls)
524+
525+
return Counter(package_urls)

0 commit comments

Comments
 (0)