Skip to content

Commit 8f94757

Browse files
committed
Rename resources_count to files_count
* Update expected test results Signed-off-by: Jono Yang <jyang@nexb.com>
1 parent 810b4d2 commit 8f94757

9 files changed

Lines changed: 16 additions & 16 deletions

plugins/scancode-consolidate-scan/src/plugin_consolidate/__init__.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ class Consolidation(object):
8080
core_holders = attr.ib(default=attr.Factory(list))
8181
other_license_expression = attr.ib(default=None)
8282
other_holders = attr.ib(default=attr.Factory(list))
83-
resources_count = attr.ib(default=None)
83+
files_count = attr.ib(default=None)
8484
resources = attr.ib(default=attr.Factory(list))
8585

8686
def to_dict(self, **kwargs):
@@ -265,7 +265,7 @@ def get_consolidated_packages(codebase):
265265
core_holders=sorted(set(package_holders)),
266266
other_license_expression=simplified_discovered_license_expression,
267267
other_holders=sorted(set(discovered_holders)),
268-
resources_count=len(package_resources),
268+
files_count=sum(1 for package_resource in package_resources if package_resource.is_file),
269269
resources=package_resources,
270270
)
271271
yield ConsolidatedPackage(
@@ -381,7 +381,7 @@ def create_license_holders_consolidated_component(resource, codebase):
381381
c = Consolidation(
382382
core_license_expression=license_expression,
383383
core_holders=holders,
384-
resources_count=len(component_resources),
384+
files_count=sum(1 for component_resource in component_resources if component_resource.is_file),
385385
resources=component_resources,
386386
)
387387
return ConsolidatedComponent(
@@ -434,7 +434,7 @@ def combine_license_holders_consolidated_components(components):
434434
c = Consolidation(
435435
core_license_expression=component_license_expression,
436436
core_holders=component_holders,
437-
resources_count=len(component_resources),
437+
files_count=sum(1 for component_resource in component_resources if component_resource.is_file),
438438
resources=component_resources,
439439
)
440440
yield ConsolidatedComponent(

plugins/scancode-consolidate-scan/tests/data/plugin_consolidate/clear-summary-expected.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
],
3535
"other_license_expression": null,
3636
"other_holders": [],
37-
"resources_count": 4
37+
"files_count": 3
3838
}
3939
],
4040
"consolidated_packages": [],

plugins/scancode-consolidate-scan/tests/data/plugin_consolidate/component-package-expected.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
],
3535
"other_license_expression": null,
3636
"other_holders": [],
37-
"resources_count": 4
37+
"files_count": 3
3838
}
3939
],
4040
"consolidated_packages": [
@@ -90,7 +90,7 @@
9090
"The Apache Software",
9191
"The Apache Software Foundation"
9292
],
93-
"resources_count": 4
93+
"files_count": 3
9494
}
9595
],
9696
"files": [

plugins/scancode-consolidate-scan/tests/data/plugin_consolidate/license-holder-rollup-expected.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
],
3535
"other_license_expression": null,
3636
"other_holders": [],
37-
"resources_count": 2
37+
"files_count": 1
3838
},
3939
{
4040
"type": "license-holders",
@@ -50,7 +50,7 @@
5050
],
5151
"other_license_expression": null,
5252
"other_holders": [],
53-
"resources_count": 2
53+
"files_count": 1
5454
},
5555
{
5656
"type": "license-holders",
@@ -66,7 +66,7 @@
6666
],
6767
"other_license_expression": null,
6868
"other_holders": [],
69-
"resources_count": 3
69+
"files_count": 1
7070
}
7171
],
7272
"consolidated_packages": [],

plugins/scancode-consolidate-scan/tests/data/plugin_consolidate/multiple-same-holder-and-license-expected.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@
3535
],
3636
"other_license_expression": null,
3737
"other_holders": [],
38-
"resources_count": 3
38+
"files_count": 2
3939
}
4040
],
4141
"consolidated_packages": [],

plugins/scancode-consolidate-scan/tests/data/plugin_consolidate/package-files-not-counted-in-license-holders-expected.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@
7171
"The Apache Software",
7272
"The Apache Software Foundation"
7373
],
74-
"resources_count": 5
74+
"files_count": 4
7575
}
7676
],
7777
"files": [

plugins/scancode-consolidate-scan/tests/data/plugin_consolidate/package-fileset-expected.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@
6969
"other_holders": [
7070
"The Apache Software Foundation"
7171
],
72-
"resources_count": 5
72+
"files_count": 4
7373
}
7474
],
7575
"files": [

plugins/scancode-consolidate-scan/tests/data/plugin_consolidate/package-manifest-expected.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@
6565
"core_holders": [],
6666
"other_license_expression": "apache-2.0",
6767
"other_holders": [],
68-
"resources_count": 2
68+
"files_count": 1
6969
}
7070
],
7171
"files": [

plugins/scancode-consolidate-scan/tests/data/plugin_consolidate/return-nested-local-majority-expected.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@
3434
],
3535
"other_license_expression": null,
3636
"other_holders": [],
37-
"resources_count": 2
37+
"files_count": 1
3838
},
3939
{
4040
"type": "license-holders",
@@ -50,7 +50,7 @@
5050
],
5151
"other_license_expression": null,
5252
"other_holders": [],
53-
"resources_count": 5
53+
"files_count": 3
5454
}
5555
],
5656
"consolidated_packages": [],

0 commit comments

Comments
 (0)