Change tests

Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>

Author: TG1999

src/packageurl/contrib/purl2url.py

@@ -78,7 +78,8 @@ def get_download_url(purl):
 
     # Fallback on the `download_url` qualifier when available.
     purl_data = PackageURL.from_string(purl)
-    return purl_data.qualifiers.get("download_url", None)
+    if purl_data.qualifiers:
+        return purl_data.qualifiers.get("download_url", None)
 
 
 def get_inferred_urls(purl):

tests/test_purl_spec.py

@@ -116,10 +116,6 @@ def load_spec_files(spec_dir):
     for case in cases:
         flattened_cases.append((filename, case["description"], case))
 
-def test_cran():
-    p = PackageURL(** {'name': 'URI::PackageURL', 'type': 
-                       'cran', 'namespace': None, 'qualifiers': None, 'subpath': None})
-    p.to_string()
 
 
 @pytest.mark.parametrize("filename,description,test_case", flattened_cases)
@@ -147,7 +143,7 @@ def run_test_case(case, test_type, desc):
 
     elif test_type == "roundtrip":
         purl = PackageURL.from_string(case["input"])
-        assert purl == PackageURL.from_string(case["expected_output"])
+        assert purl.to_string() == case["expected_output"]
 
     elif test_type == "build":
         input_data = case["input"]
@@ -159,4 +155,4 @@ def run_test_case(case, test_type, desc):
             qualifiers=input_data.get("qualifiers"),
             subpath=input_data.get("subpath"),
         )
-        assert purl == PackageURL.from_string(case["expected_output"])
+        assert purl.to_string() == case["expected_output"]