transform url path to os path

normalize os path in tests

eodag_cube/api/product/_product.py

@@ -22,7 +22,8 @@
 from collections import UserDict
 from contextlib import contextmanager
 from typing import TYPE_CHECKING, Any, Dict, List, Optional, Union
-from urllib.parse import urlparse
+import urllib.parse
+import urllib.request
 
 import numpy as np
 import rasterio
@@ -258,8 +259,8 @@ def _get_rio_env(self, dataset_address: str) -> Dict[str, Any]:
     def _build_xarray_dict(self, **kwargs):
         logger.warning(">>>>>>>>>>>>>>>>>>")
         result = XarrayDict()
-        product_path = urlparse(self.location).path
-        logger.warning("### product_path: %s" % product_path)
+        url_path = urllib.parse.urlparse(self.location).path
+        product_path = urllib.request.url2pathname(url_path)
         for root, _dirs, filenames in os.walk(product_path):
             logger.warning("### root: %s filenames: %s" % (root, filenames))
             for filename in filenames:

tests/units/test_eoproduct.py

@@ -22,6 +22,7 @@
 import shutil
 from pathlib import Path
 from tempfile import TemporaryDirectory
+import urllib.request
 
 import numpy as np
 import xarray as xr
@@ -277,7 +278,7 @@ def test_build_xarray_dict(self):
             product = EOProduct(
                 self.provider, self.eoproduct_props, productType=self.product_type
             )
-            product.location = f"file://{tmp_dir}"
+            product.location = "file:" + urllib.request.pathname2url(tmp_dir)
             self.populate_directory_with_heterogeneous_files(tmp_dir)
 
             xarray_dict = product._build_xarray_dict()
@@ -287,3 +288,6 @@ def test_build_xarray_dict(self):
             for key, value in xarray_dict.items():
                 self.assertIn(Path(key).suffix, {".nc", ".jp2"})
                 self.assertIsInstance(value, xr.Dataset)
+
+            for ds in xarray_dict.values():
+                ds.close()

tests/units/test_eoproduct_driver_generic.py

@@ -62,15 +62,19 @@ def test_driver_get_local_dataset_address_ok(self):
         with self._filesystem_product() as product:
             band = "B01"
             address = self.product.driver.get_data_address(product, band)
-            self.assertEqual(address, self.local_band_file)
+            self.assertEqual(
+                os.path.normcase(address), os.path.normcase(self.local_band_file)
+            )
 
     def test_driver_get_local_grib_dataset_address_ok(self):
         """Driver returns a good address for a grib file"""
         with self._grib_product() as product:
 
             address = self.product.driver.get_data_address(product, TEST_GRIB_FILENAME)
 
-            self.assertEqual(address, TEST_GRIB_FILE_PATH)
+            self.assertEqual(
+                os.path.normcase(address), os.path.normcase(TEST_GRIB_FILE_PATH)
+            )
 
     def test_driver_get_http_remote_dataset_address_fail(self):
         """Driver must raise UnsupportedDatasetAddressScheme if location scheme is http or https"""

tests/units/test_eoproduct_driver_sentinel2_l1c.py

@@ -59,7 +59,9 @@ def test_driver_get_local_dataset_address_ok(self):
         with self._filesystem_product() as product:
             band = "B01"
             address = self.product.driver.get_data_address(product, band)
-            self.assertEqual(address, self.local_band_file)
+            self.assertEqual(
+                os.path.normcase(address), os.path.normcase(self.local_band_file)
+            )
 
     @mock_aws
     def test_driver_get_amazon_s3_remote_dataset_address_ok(self):