Further avoid confusion of LoadParameters.temporal_extent default

Author: soxofaan

openeo_driver/ProcessGraphDeserializer.py

@@ -838,7 +838,9 @@ def _extract_load_parameters(env: EvalEnv, source_id: tuple) -> LoadParameters:
     source_constraints.remove((source_id,constraints))  # Side effect!
 
     params = LoadParameters()
-    params.temporal_extent = constraints.get("temporal_extent", None)
+    if temporal_extent := constraints.get("temporal_extent"):
+        params.temporal_extent = temporal_extent
+    # TODO: avoid duplication of default values used here and in LoadParameters definition
     labels_args = constraints.get("filter_labels", {})
     if("dimension" in labels_args and labels_args["dimension"] == "t"):
         params.filter_temporal_labels = labels_args.get("condition")

openeo_driver/_version.py

@@ -1 +1 @@
-__version__ = "0.138.0a1"
+__version__ = "0.138.0a2"

tests/test_views_execute.py

@@ -4367,7 +4367,7 @@ def test_load_stac_default_temporal_extent(api, backend_implementation):
     assert load_stac.call_count == 1
     _, kwargs = load_stac.call_args
 
-    assert kwargs["load_params"]["temporal_extent"] is None
+    assert kwargs["load_params"].temporal_extent == (None, None)
 
 
 class TestVectorCubeRunUDF: