fix: windows support

.github/workflows/test.yml

@@ -28,8 +28,8 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        python-version: [3.8, 3.12]
-        os: [ubuntu-latest]
+        python-version: [3.8, "3.12"]
+        os: [ubuntu-latest, windows-latest]
     steps:
     - name: Checkout the repo
       uses: actions/checkout@v2

eodag_cube/api/product/drivers/generic.py

@@ -42,7 +42,7 @@ def get_data_address(self, eo_product, band):
                 try:
                     # return the first file readable by rasterio
                     rasterio.drivers.driver_from_extension(filename)
-                    return str(filename)
+                    return str(filename.resolve())
                 except ValueError:
                     pass
             raise AddressNotFound

eodag_cube/api/product/drivers/sentinel2_l1c.py

@@ -80,7 +80,7 @@ def get_data_address(self, eo_product, band):
                                     lambda f: band_file_pattern.match(f),
                                     subdataset.files,
                                 ):
-                                    return filename
+                                    return os.path.normpath(filename)
                 raise AddressNotFound
         if product_location_scheme == "s3":
             access_key, access_secret = eo_product.downloader_auth.authenticate()

tests/units/test_eoproduct.py

@@ -56,7 +56,7 @@ def test_eoproduct_driver_ok(self):
     def test_get_data_local_product_ok(self):
         """A call to get_data on a product present in the local filesystem must succeed"""  # noqa
         self.eoproduct_props.update(
-            {"downloadLink": "file://{}".format(self.local_product_abspath)}
+            {"downloadLink": "file:///{}".format(self.local_product_abspath.strip("/"))}
         )
         product = EOProduct(
             self.provider, self.eoproduct_props, productType=self.product_type

tests/units/test_eoproduct_driver_generic.py

@@ -70,7 +70,9 @@ def test_driver_get_http_remote_dataset_address_fail(self):
     def _filesystem_product(self):
         original = self.product.location
         try:
-            self.product.location = "file://{}".format(self.product.properties["title"])
+            self.product.location = "file:///{}".format(
+                self.product.properties["title"].strip("/")
+            )
             yield self.product
         finally:
             self.product.location = original

tests/units/test_eoproduct_driver_sentinel2_l1c.py

@@ -84,7 +84,9 @@ def test_driver_get_http_remote_dataset_address_fail(self):
     def _filesystem_product(self):
         original = self.product.location
         try:
-            self.product.location = "file://{}".format(self.product.properties["title"])
+            self.product.location = "file:///{}".format(
+                self.product.properties["title"].strip("/")
+            )
             yield self.product
         finally:
             self.product.location = original

tests/units/test_eoproduct_driver_stac_assets.py

@@ -34,38 +34,40 @@ def setUp(self):
             "products",
             "S2A_MSIL1C_20180101T105441_N0206_R051_T31TDH_20180101T124911.SAFE",
         )
+        product_img_data_path = os.path.join(
+            TEST_RESOURCES_PATH,
+            "products",
+            "S2A_MSIL1C_20180101T105441_N0206_R051_T31TDH_20180101T124911.SAFE",
+            "GRANULE",
+            "L1C_T31TDH_A013204_20180101T105435",
+            "IMG_DATA",
+        )
         self.product.assets = {
             "T31TDH_20180101T124911_B01.jp2": {
                 "title": "Band 1",
                 "type": "image/jp2",
                 "roles": ["data"],
                 "href": os.path.join(
-                    TEST_RESOURCES_PATH,
-                    "products",
-                    "S2A_MSIL1C_20180101T105441_N0206_R051_T31TDH_20180101T124911.SAFE/"
-                    + "GRANULE/L1C_T31TDH_A013204_20180101T105435/IMG_DATA/T31TDH_20180101T105441_B01.jp2",
+                    product_img_data_path,
+                    "T31TDH_20180101T105441_B01.jp2",
                 ),
             },
             "T31TDH_20180101T124911_B03.jp2": {
                 "title": "Band 3",
                 "type": "image/jp2",
                 "roles": ["data"],
                 "href": os.path.join(
-                    TEST_RESOURCES_PATH,
-                    "products",
-                    "S2A_MSIL1C_20180101T105441_N0206_R051_T31TDH_20180101T124911.SAFE/"
-                    + "GRANULE/L1C_T31TDH_A013204_20180101T105435/IMG_DATA/T31TDH_20180101T105441_B03.jp2",
+                    product_img_data_path,
+                    "T31TDH_20180101T105441_B03.jp2",
                 ),
             },
             "T31TDH_20180101T124911_B01.json": {
                 "title": "Band 1 metadata",
                 "type": "image/jp2",
                 "roles": ["metadata"],
                 "href": os.path.join(
-                    TEST_RESOURCES_PATH,
-                    "products",
-                    "S2A_MSIL1C_20180101T105441_N0206_R051_T31TDH_20180101T124911.SAFE/"
-                    + "GRANULE/L1C_T31TDH_A013204_20180101T105435/IMG_DATA/T31TDH_20180101T105441_B01.json",
+                    product_img_data_path,
+                    "T31TDH_20180101T105441_B01.json",
                 ),
             },
         }
@@ -100,7 +102,9 @@ def test_driver_get_local_dataset_address_ok(self):
     def _filesystem_product(self):
         original = self.product.location
         try:
-            self.product.location = "file://{}".format(self.product.properties["title"])
+            self.product.location = "file:///{}".format(
+                self.product.properties["title"].strip("/")
+            )
             yield self.product
         finally:
             self.product.location = original