Fixes based on review comments

Author: m-mohr

openeo/metadata.py

@@ -508,9 +508,9 @@ class CollectionMetadata(CubeMetadata):
 
     """
 
-    def __init__(self, metadata: dict, dimensions: List[Dimension] = None, connection = None):
+    def __init__(self, metadata: dict, dimensions: List[Dimension] = None, federation = None):
         self._orig_metadata = metadata
-        self._connection = connection
+        self._federation = federation
         if dimensions is None:
             dimensions = self._parse_dimensions(self._orig_metadata)
 
@@ -630,8 +630,7 @@ def extent(self) -> dict:
         return self._orig_metadata.get("extent")
 
     def _repr_html_(self):
-        federation = self._connection.capabilities().ext_federation_backend_details()
-        return render_component("collection", data=self._orig_metadata, parameters={'federation': federation})
+        return render_component("collection", data=self._orig_metadata, parameters={'federation': self._federation})
 
     def __str__(self) -> str:
         bands = self.band_names if self.has_band_dimension() else "no bands dimension"

openeo/rest/connection.py

@@ -881,7 +881,8 @@ def collection_items(
 
     def collection_metadata(self, name) -> CollectionMetadata:
         # TODO: duplication with `Connection.describe_collection`: deprecate one or the other?
-        return CollectionMetadata(metadata=self.describe_collection(name), connection=self)
+        federation = self.capabilities().ext_federation_backend_details()
+        return CollectionMetadata(metadata=self.describe_collection(name), federation=federation)
 
     def list_processes(self, namespace: Optional[str] = None) -> ProcessListingResponse:
         """