Merge branch 'img-source' of https://github.com/BiocPy/SpatialExperiment into img-source

Author: keviny2

src/spatialexperiment/SpatialExperiment.py

@@ -921,7 +921,7 @@ def img_source(
         path=False,
     ):
         """Retrieve the source(s) for images stored in the SpatialExperiment object.
-        
+
         Args:
             sample_id:
                 - `sample_id=True`: Matches all samples.
@@ -932,9 +932,9 @@ def img_source(
                 - `image_id=True`: Matches all images for the specified sample(s).
                 - `image_id=None`: Matches the first image for the sample(s).
                 - `image_id="<str>"`: Matches image(s) by its(their) id.
-            
+
             path: If True, returns path as string. Defaults to False.
-        
+
         Returns:
             The image source(s) for the matching criteria. Returns `None` if `img_data` is `None`.
             When a single image matches, returns its source as a `str`, `Path`, or `None`.

tests/test_img_source.py

@@ -13,14 +13,14 @@
 def test_loaded_spatial_image_img_source():
     image = Image.open("tests/images/sample_image2.png")
     spi_loaded = construct_spatial_image_class(image, is_url=False)
-    
+
     assert isinstance(spi_loaded, LoadedSpatialImage)
     assert spi_loaded.img_source() is None
     assert spi_loaded.img_source(as_path=True) is None
-    
+
     np_image = np.zeros((100, 100, 3), dtype=np.uint8)
     spi_loaded_np = construct_spatial_image_class(np_image)
-    
+
     assert isinstance(spi_loaded_np, LoadedSpatialImage)
     assert spi_loaded_np.img_source() is None
     assert spi_loaded_np.img_source(as_path=True) is None
@@ -29,26 +29,26 @@ def test_loaded_spatial_image_img_source():
 def test_stored_spatial_image_img_source():
     image_path = "tests/images/sample_image1.jpg"
     spi_stored = construct_spatial_image_class(image_path, is_url=False)
-    
+
     assert isinstance(spi_stored, StoredSpatialImage)
-    
+
     source_path = spi_stored.img_source()
     assert isinstance(source_path, Path)
     assert image_path in str(source_path)
-    
+
     source_str = spi_stored.img_source(as_path=True)
     assert isinstance(source_str, str)
     assert image_path in source_str
-    
+
     assert str(source_path) == str(spi_stored.path)
 
 
 def test_remote_spatial_image_img_source():
     image_url = "https://example.com/test_image.jpg"
     spi_remote = construct_spatial_image_class(image_url, is_url=True)
-    
+
     assert isinstance(spi_remote, RemoteSpatialImage)
-    
+
     source = spi_remote.img_source()
     assert isinstance(source, str)
     assert source == image_url
@@ -57,17 +57,17 @@ def test_remote_spatial_image_img_source():
 def test_remote_spatial_image_img_source_with_mock(monkeypatch):
     image_url = "https://example.com/test_image.jpg"
     spi_remote = construct_spatial_image_class(image_url, is_url=True)
-    
+
     assert isinstance(spi_remote, RemoteSpatialImage)
-    
+
     # Mock the _download_image method to return a fixed path
     mock_path = Path("/tmp/image.jpg")
     monkeypatch.setattr(spi_remote, "_download_image", lambda: mock_path)
-    
+
     # Test with as_path=True (returns the cached path)
     source_path = spi_remote.img_source(as_path=True)
     assert source_path == str(mock_path)
-    
+
     # Test default behavior returns URL
     assert spi_remote.img_source() == image_url