From: Stefano Tondo <[email protected]> Add test to verify that SPDX generation handles download_location failures gracefully and doesn't crash if fetch_data_to_uri() behavior changes.
Test verifies: 1. SPDX file generation succeeds for recipes with tarball sources 2. External references are properly structured when generated 3. ExternalRef.locator is a list of strings (SPDX 3.0 spec requirement) 4. Defensive try/except and isinstance() checks prevent crashes The test uses m4 recipe which has tarball sources, allowing verification of the download location handling without requiring complex setup. Test can be run with: oe-selftest -r spdx.SPDX30Check.test_download_location_defensive_handling Signed-off-by: Stefano Tondo <[email protected]> --- meta/lib/oeqa/selftest/cases/spdx.py | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/meta/lib/oeqa/selftest/cases/spdx.py b/meta/lib/oeqa/selftest/cases/spdx.py index 41ef52fce1..cae5c95f43 100644 --- a/meta/lib/oeqa/selftest/cases/spdx.py +++ b/meta/lib/oeqa/selftest/cases/spdx.py @@ -414,3 +414,31 @@ class SPDX30Check(SPDX3CheckBase, OESelftestTestCase): value, ["enabled", "disabled"], f"Unexpected PACKAGECONFIG value '{value}' for {key}" ) + + def test_download_location_defensive_handling(self): + """Test that download_location handling is defensive. + + Verifies SPDX generation succeeds and external references are + properly structured when download_location retrieval works. + """ + objset = self.check_recipe_spdx( + "m4", + "{DEPLOY_DIR_SPDX}/{SSTATE_PKGARCH}/recipes/recipe-m4.spdx.json", + ) + + found_external_refs = False + for pkg in objset.foreach_type(oe.spdx30.software_Package): + if hasattr(pkg, 'externalRef') and pkg.externalRef: + found_external_refs = True + for ref in pkg.externalRef: + self.assertIsNotNone(ref.externalRefType) + self.assertIsNotNone(ref.locator) + self.assertIsInstance(ref.locator, list) + for loc in ref.locator: + self.assertIsInstance(loc, str) + break + + self.logger.info( + f"External references {'found' if found_external_refs else 'not found'} " + f"in SPDX output (defensive handling verified)" + ) -- 2.53.0
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#231592): https://lists.openembedded.org/g/openembedded-core/message/231592 Mute This Topic: https://lists.openembedded.org/mt/117922744/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
