_job_splitting: raise error when user specified non-positive tile_size #883

Author: VincentVerelst

openeo/extra/job_management/_job_splitting.py

@@ -122,6 +122,8 @@ def __init__(self, *, epsg: int, size: float):
         if not isinstance(epsg, int):
             raise JobSplittingFailure(f"Only integer EPSG codes are supported for tile grid splitting, got {epsg!r}.")
         self._epsg = epsg
+        if size <= 0:
+            raise JobSplittingFailure(f"Tile size must be positive, got {size!r}.")
         self.size = size
 
     @property

tests/extra/job_management/test_job_splitting.py

@@ -28,6 +28,11 @@ def test_constructor_rejects_unknown_epsg(self):
         with pytest.raises(JobSplittingFailure, match="Failed to normalize EPSG code"):
             _SizeBasedTileGrid(epsg=999999, size=1.0)
 
+    @pytest.mark.parametrize("size", [0, -1, -0.5])
+    def test_constructor_rejects_non_positive_size(self, size):
+        """Tile size must be strictly positive."""
+        with pytest.raises(JobSplittingFailure, match="Tile size must be positive"):
+            _SizeBasedTileGrid(epsg=4326, size=size)
     def test_get_tiles_raises_exception(self):
         """test get_tiles when the input geometry is not a dict or shapely.geometry.Polygon"""
         tile_grid = _SizeBasedTileGrid(epsg=4326, size=0.1)