Skip to content

Commit 901737b

Browse files
committed
Return package_uids in for_packages #444
* This is so we are consistent with scancode-toolkit JSON output Signed-off-by: Jono Yang <[email protected]>
1 parent 1264008 commit 901737b

4 files changed

+44
-325
lines changed

scanpipe/models.py

+4-1
Original file line numberDiff line numberDiff line change
@@ -1759,7 +1759,10 @@ def for_packages(self):
17591759
"""
17601760
Returns the list of all discovered packages associated to this resource.
17611761
"""
1762-
return [str(package) for package in self.discovered_packages.all()]
1762+
return [
1763+
package.package_uid if package.package_uid else str(package)
1764+
for package in self.discovered_packages.all()
1765+
]
17631766

17641767

17651768
class DiscoveredPackageQuerySet(PackageURLQuerySetMixin, ProjectRelatedQuerySet):

scanpipe/tests/data/asgiref-3.3.0_load_inventory_expected.json

+40-10
Original file line numberDiff line numberDiff line change
@@ -847,7 +847,8 @@
847847
"is_text": true,
848848
"is_archive": false,
849849
"is_key_file": false,
850-
"is_media": false
850+
"is_media": false,
851+
"package_data": []
851852
},
852853
{
853854
"for_packages": [],
@@ -873,7 +874,8 @@
873874
"is_text": true,
874875
"is_archive": false,
875876
"is_key_file": false,
876-
"is_media": false
877+
"is_media": false,
878+
"package_data": []
877879
},
878880
{
879881
"for_packages": [],
@@ -899,7 +901,8 @@
899901
"is_text": true,
900902
"is_archive": false,
901903
"is_key_file": false,
902-
"is_media": false
904+
"is_media": false,
905+
"package_data": []
903906
},
904907
{
905908
"for_packages": [],
@@ -930,9 +933,9 @@
930933
},
931934
{
932935
"for_packages": [],
933-
"path": "asgiref-3.3.0-py3-none-any.whl-extract/asgiref-3.3.0.dist-info/top_level.txt",
934-
"sha1": "612390bd0d0227c009f9c99b479878adf7ac2f23",
935-
"md5": "680e61db4d95c8d9501b7a49fa2bf0b2",
936+
"path": "asgiref-3.3.0-py3-none-any.whl-extract/asgiref/compatibility.py",
937+
"sha1": "9c74e64e9a71903bb227907ea1806eac77e52434",
938+
"md5": "5231077fd0628314246fcba7817b561e",
936939
"extra_data": {},
937940
"copyrights": [],
938941
"holders": [],
@@ -944,10 +947,37 @@
944947
"status": "",
945948
"tag": "",
946949
"type": "file",
947-
"name": "top_level.txt",
948-
"extension": ".txt",
949-
"programming_language": "",
950-
"mime_type": "text/plain",
950+
"name": "compatibility.py",
951+
"extension": ".py",
952+
"programming_language": "Python",
953+
"mime_type": "text/x-script.python",
954+
"is_binary": false,
955+
"is_text": true,
956+
"is_archive": false,
957+
"is_key_file": false,
958+
"is_media": false,
959+
"package_data": []
960+
},
961+
{
962+
"for_packages": [],
963+
"path": "asgiref-3.3.0-py3-none-any.whl-extract/asgiref/current_thread_executor.py",
964+
"sha1": "aacf7e5e2e5ba78ccfb67fa10e9e6b22c3935c9b",
965+
"md5": "b4c45f37055d88dd11b15eb4de51b074",
966+
"extra_data": {},
967+
"copyrights": [],
968+
"holders": [],
969+
"authors": [],
970+
"licenses": [],
971+
"license_expressions": [],
972+
"emails": [],
973+
"urls": [],
974+
"status": "",
975+
"tag": "",
976+
"type": "file",
977+
"name": "current_thread_executor.py",
978+
"extension": ".py",
979+
"programming_language": "Python",
980+
"mime_type": "text/x-script.python",
951981
"is_binary": false,
952982
"is_text": true,
953983
"is_archive": false,

scanpipe/tests/data/centos_scan_codebase.json

-52
Original file line numberDiff line numberDiff line change
@@ -186626,58 +186626,6 @@
186626186626
"is_media": false,
186627186627
"package_data": []
186628186628
},
186629-
{
186630-
"for_packages": [],
186631-
"path": "centos.tar.gz-extract/a10cf747c363a52be048f884c084a25e03280d54a7ac02e17dbd8c5ad160e9bd/var/lib/rpm/.dbenv.lock",
186632-
"sha1": "",
186633-
"md5": "",
186634-
"extra_data": {},
186635-
"copyrights": [],
186636-
"holders": [],
186637-
"authors": [],
186638-
"licenses": [],
186639-
"license_expressions": [],
186640-
"emails": [],
186641-
"urls": [],
186642-
"status": "ignored-empty-file",
186643-
"tag": "img-c967b7-layer-01-a10cf7",
186644-
"type": "file",
186645-
"name": ".dbenv.lock",
186646-
"extension": ".lock",
186647-
"programming_language": "",
186648-
"mime_type": "inode/x-empty",
186649-
"is_binary": false,
186650-
"is_text": true,
186651-
"is_archive": false,
186652-
"is_key_file": false,
186653-
"is_media": false
186654-
},
186655-
{
186656-
"for_packages": [],
186657-
"path": "centos.tar.gz-extract/a10cf747c363a52be048f884c084a25e03280d54a7ac02e17dbd8c5ad160e9bd/var/lib/rpm/.rpm.lock",
186658-
"sha1": "",
186659-
"md5": "",
186660-
"extra_data": {},
186661-
"copyrights": [],
186662-
"holders": [],
186663-
"authors": [],
186664-
"licenses": [],
186665-
"license_expressions": [],
186666-
"emails": [],
186667-
"urls": [],
186668-
"status": "ignored-empty-file",
186669-
"tag": "img-c967b7-layer-01-a10cf7",
186670-
"type": "file",
186671-
"name": ".rpm.lock",
186672-
"extension": ".lock",
186673-
"programming_language": "",
186674-
"mime_type": "inode/x-empty",
186675-
"is_binary": false,
186676-
"is_text": true,
186677-
"is_archive": false,
186678-
"is_key_file": false,
186679-
"is_media": false
186680-
},
186681186629
{
186682186630
"for_packages": [
186683186631
"pkg:rpm/[email protected]?arch=x86_64&uuid=fixed-uid-done-for-testing-5642512d1758"

0 commit comments

Comments
 (0)