set default metadata when using load_result, to avoid client vs backend inconsistencies

Author: jdries

openeo/rest/connection.py

@@ -23,7 +23,7 @@
 from openeo.internal.jupyter import VisualDict, VisualList
 from openeo.internal.processes.builder import ProcessBuilderBase
 from openeo.internal.warnings import legacy_alias, deprecated
-from openeo.metadata import CollectionMetadata
+from openeo.metadata import CollectionMetadata, SpatialDimension, TemporalDimension, BandDimension, Band
 from openeo.rest import OpenEoClientException, OpenEoApiError, OpenEoRestError
 from openeo.rest.auth.auth import NullAuth, BearerAuth, BasicBearerAuth, OidcBearerAuth, OidcRefreshInfo
 from openeo.rest.auth.config import RefreshTokenStore, AuthConfig
@@ -1035,15 +1035,18 @@ def load_result(
         if self._api_version.below("1.0.0"):
             raise OpenEoClientException(
                 "This method requires support for at least version 1.0.0 in the openEO backend.")
-        return self.datacube_from_process(
-            process_id="load_result",
-            id=id,
-            **dict_no_none(
-                spatial_extent=spatial_extent,
-                temporal_extent=temporal_extent and DataCube._get_temporal_extent(temporal_extent),
-                bands=bands
-            )
-        )
+        metadata = CollectionMetadata({}, dimensions=[
+            SpatialDimension(name="x", extent=[]),
+            SpatialDimension(name="y", extent=[]),
+            TemporalDimension(name='t', extent=[]),
+            BandDimension(name="bands", bands=[Band("unknown")]),
+        ])
+        cube = self.datacube_from_process(process_id="load_result", id=id,
+                                             **dict_no_none(spatial_extent=spatial_extent,
+                                                            temporal_extent=temporal_extent and DataCube._get_temporal_extent(
+                                                                temporal_extent), bands=bands))
+        cube.metadata = metadata
+        return cube
 
     def load_ml_model(self, id: Union[str, BatchJob]) -> "MlModel":
         """

tests/rest/test_connection.py

@@ -1761,6 +1761,9 @@ def test_load_result(requests_mock):
     requests_mock.get(API_URL, json={"api_version": "1.0.0"})
     con = Connection(API_URL)
     cube = con.load_result("j0bi6")
+    assert cube.metadata.has_band_dimension()
+    assert cube.metadata.has_temporal_dimension()
+    assert len(cube.metadata.spatial_dimensions)==2
     assert cube.flat_graph() == {
         "loadresult1": {"process_id": "load_result", "arguments": {"id": "j0bi6"}, "result": True}
     }