chef: make cached artifact uri platform independent

Cached artifact uri was previously constructed manually using a string
representation of the artifact path. This change ensures that the uri 
is generated by pathlib instead.

Resolves: #2744

Co-authored-by: Fredrik Averpil <fredrik.averpil@ericsson.com>
Co-authored-by: Arun Babu Neelicattu <arun.neelicattu@gmail.com>

poetry/installation/chef.py

@@ -81,7 +81,7 @@ def get_cached_archives_for_link(self, link):  # type: (Link) -> List[Link]
         links = []
         for archive_type in archive_types:
             for archive in cache_dir.glob("*.{}".format(archive_type)):
-                links.append(Link("file://{}".format(str(archive))))
+                links.append(Link(archive.as_uri()))
 
         return links
 

tests/installation/test_chef.py

@@ -44,17 +44,19 @@ def test_get_cached_archives_for_link(config, mocker):
         ),
     )
 
+    distributions = Path(__file__).parent.parent.joinpath("fixtures/distributions")
     mocker.patch.object(
-        chef,
-        "get_cache_directory_for_link",
-        return_value=Path(__file__).parent.parent.joinpath("fixtures/distributions"),
+        chef, "get_cache_directory_for_link", return_value=distributions,
     )
 
     archives = chef.get_cached_archives_for_link(
         Link("https://files.python-poetry.org/demo-0.1.0.tar.gz")
     )
 
-    assert 2 == len(archives)
+    assert archives
+    assert set(archives) == {
+        Link(path.as_uri()) for path in distributions.glob("demo-0.1.0*")
+    }
 
 
 def test_get_cache_directory_for_link(config):