fix: set wekeo_main products storageTier to online (#29)

`auto_order_whitelist` setting has been added previously, and now
products from federation backends of this whitelist has `storageTier`
metadata from eodag
[EOProduct()](https://eodag.readthedocs.io/en/stable/api_reference/eoproduct.html#eodag.api.product._product.EOProduct)
set to `ONLINE` because products are available immediately after a
search. This means that `storage:tier` and `order:status` STAC
properties are respectively set to `online` and `succeeded` when these
`EOProducts` are converted to STAC items.

Author: anesson-cs

README.md

@@ -106,6 +106,7 @@ Reach to [stac-fastapi documentation](https://stac-utils.github.io/stac-fastapi/
 | `ORIGIN_URL_BLACKLIST` | Hide from clients items assets' origin URLs starting with URLs from the list. A string of comma separated values is expected. | "" |
 | `COUNT` | Whether to run a query with a count request or not. | False |
 | `FETCH_PROVIDERS` | Fetch additional collections from all EODAG providers. | False |
+| `AUTO_ORDER_WHITELIST` | List of providers for which the order should be done at the same time as the download. | ["wekeo_main"] |
 | `DOWNLOAD_BASE_URL` | Useful to expose asset download URL in a separate domain name. | "" |
 
 ### OpenTelemetry parameters

stac_fastapi/eodag/config.py

@@ -38,16 +38,17 @@ class Settings(ApiSettings):
         default=False,
         description=("Keep origin as alternate URL when data-download extension is enabled."),
     )
+
     origin_url_blacklist: Annotated[Union[str, list[str]], BeforeValidator(str2liststr)] = Field(
         default=[],
-        description=("Hide from clients items assets' origin URLs starting with URLs from the list"),
+        description=("Hide from clients items assets' origin URLs starting with URLs from the list."),
     )
 
     auto_order_whitelist: Annotated[Union[str, list[str]], BeforeValidator(str2liststr)] = Field(
         default=[
             "wekeo_main",
         ],
-        description=("Do the order at the same time as the download"),
+        description=("List of providers for which the order should be done at the same time as the download."),
     )
 
     fetch_providers: bool = Field(default=False, description="Fetch additional collections from all providers.")

stac_fastapi/eodag/models/stac_metadata.py

@@ -288,9 +288,14 @@ def create_stac_item(
         if extension_is_enabled("DataDownload")
         else None
     )
-    # create assets only if product is not offline
+
     settings = get_settings()
     auto_order_whitelist = settings.auto_order_whitelist
+    if product.provider in auto_order_whitelist:
+        # a product from a whitelisted federation backend is considered as online
+        product.properties["storageStatus"] = ONLINE_STATUS
+
+    # create assets only if product is not offline
     if (
         product.properties.get("storageStatus", ONLINE_STATUS) != OFFLINE_STATUS
         or product.provider in auto_order_whitelist

tests/test_download.py

@@ -71,11 +71,14 @@ async def test_download_auto_order_whitelist(
     and checks that the order function is triggered when the backend is present
     in the auto_order_whitelist configuration.
     """
-    get_settings().auto_order_whitelist = ["peps"]
     federation_backend = "peps"
+    # update the auto_order_whitelist setting to include "peps"
+    auto_order_whitelist = get_settings().auto_order_whitelist
+    get_settings().auto_order_whitelist = [federation_backend]
+
     collection_id = defaults.product_type
 
-    url = f"data/peps/{defaults.product_type}/dummy_id/downloadLink"
+    url = f"data/{federation_backend}/{defaults.product_type}/dummy_id/downloadLink"
 
     product = EOProduct(
         federation_backend,
@@ -88,7 +91,7 @@ async def test_download_auto_order_whitelist(
     product.product_type = collection_id
     config = PluginConfig()
     config.priority = 0
-    downloader = HTTPDownload("peps", config)
+    downloader = HTTPDownload(federation_backend, config)
 
     product.register_downloader(downloader=downloader, authenticator=None)
 
@@ -98,3 +101,6 @@ async def test_download_auto_order_whitelist(
     await request_valid_raw(url, search_result=SearchResult([product]))
 
     assert mock_order.called
+
+    # restore the original auto_order_whitelist setting
+    get_settings().auto_order_whitelist = auto_order_whitelist

tests/test_search.py

@@ -143,6 +143,20 @@ async def test_items_response(request_valid, defaults):
         == f"http://testserver/data/peps/{res[0]['collection']}/{res[0]['id']}/asset1"
     )
 
+    # check order status and storage tier properties of the "OFFLINE" item when peps is whitelisted
+    auto_order_whitelist = get_settings().auto_order_whitelist
+    get_settings().auto_order_whitelist = ["peps"]
+
+    resp_json = await request_valid(
+        f"search?collections={defaults.product_type}",
+    )
+    res = resp_json["features"]
+    assert res[1]["properties"]["order:status"] == "succeeded"
+    assert res[1]["properties"]["storage:tier"] == "online"
+
+    # restore the original auto_order_whitelist setting
+    get_settings().auto_order_whitelist = auto_order_whitelist
+
 
 async def test_items_response_unexpected_types(request_valid, defaults, mock_search_result):
     """Item properties contain values in unexpected types for processingLevel and platform