chore: minor code style cleanups

Author: soxofaan

openeo/metadata.py

@@ -49,8 +49,10 @@ def rename_labels(self, target, source) -> 'Dimension':
 class SpatialDimension(Dimension):
     DEFAULT_CRS = 4326
 
-    def __init__(self, name: str, extent: Union[Tuple[float, float], List[float]],
-                 crs: Union[str, int, dict] = DEFAULT_CRS, step = None):
+    def __init__(
+            self, name: str, extent: Union[Tuple[float, float], List[float]],
+            crs: Union[str, int, dict] = DEFAULT_CRS, step=None
+    ):
         """
 
         @param name:
@@ -213,7 +215,9 @@ def get_or_create(cls, metadata: Union[dict, 'CollectionMetadata', None]) -> 'Co
         else:
             return cls(metadata=metadata or {})
 
-    def _clone_and_update(self, metadata: dict = None, dimensions: List[Dimension] = None, **kwargs) -> 'CollectionMetadata':
+    def _clone_and_update(
+            self, metadata: dict = None, dimensions: List[Dimension] = None, **kwargs
+    ) -> 'CollectionMetadata':
         """Create a new instance (of same class) with copied/updated fields."""
         cls = type(self)
         if dimensions == None:
@@ -436,9 +440,7 @@ def drop_dimension(self, name: str = None) -> 'CollectionMetadata':
         return self._clone_and_update(dimensions=[d for d in self._dimensions if not d.name == name])
 
     def _repr_html_(self):
-        return render_component('collection', data = self._orig_metadata)
+        return render_component('collection', data=self._orig_metadata)
 
     def __str__(self) -> str:
         return f"CollectionMetadata({self.extent} - {self.band_names} - {self.dimension_names()})"
-
-

tests/test_metadata.py

@@ -97,33 +97,32 @@ def test_band_dimension_rename_labels():
     b03 = Band("B03", "green", 0.560)
     b04 = Band("B04", "red", 0.665)
     bdim = BandDimension(name="bs", bands=[b02, b03, b04])
-    metadata = CollectionMetadata({},dimensions=[bdim])
-    newdim = metadata.rename_labels("bs",target=['1','2','3']).band_dimension
+    metadata = CollectionMetadata({}, dimensions=[bdim])
+    newdim = metadata.rename_labels("bs", target=['1', '2', '3']).band_dimension
 
-    assert metadata.band_dimension.band_names == ['B02','B03','B04']
-    assert newdim.band_names == ['1','2','3']
+    assert metadata.band_dimension.band_names == ['B02', 'B03', 'B04']
+    assert newdim.band_names == ['1', '2', '3']
 
 
 def test_band_dimension_set_labels():
-
-    bdim = BandDimension(name="bs", bands=[Band('some_name',None,None)])
-    metadata = CollectionMetadata({},dimensions=[bdim])
-    newdim = metadata.rename_labels("bs",target=['1','2','3']).band_dimension
+    bdim = BandDimension(name="bs", bands=[Band('some_name', None, None)])
+    metadata = CollectionMetadata({}, dimensions=[bdim])
+    newdim = metadata.rename_labels("bs", target=['1', '2', '3']).band_dimension
 
     assert metadata.band_dimension.band_names == ['some_name']
-    assert newdim.band_names == ['1','2','3']
+    assert newdim.band_names == ['1', '2', '3']
 
 
 def test_band_dimension_rename_labels_with_source():
     b02 = Band("B02", "blue", 0.490)
     b03 = Band("B03", "green", 0.560)
     b04 = Band("B04", "red", 0.665)
     bdim = BandDimension(name="bs", bands=[b02, b03, b04])
-    metadata = CollectionMetadata({},dimensions=[bdim])
-    newdim = metadata.rename_labels("bs",target=['2'],source=['B03']).band_dimension
+    metadata = CollectionMetadata({}, dimensions=[bdim])
+    newdim = metadata.rename_labels("bs", target=['2'], source=['B03']).band_dimension
 
-    assert metadata.band_dimension.band_names == ['B02','B03','B04']
-    assert newdim.band_names == ['B02','2','B04']
+    assert metadata.band_dimension.band_names == ['B02', 'B03', 'B04']
+    assert newdim.band_names == ['B02', '2', 'B04']
 
 
 def test_band_dimension_rename_labels_with_source_mismatch():