🧑‍💻 Refine PatchExtractor Error Message

tests/test_patch_extraction.py

@@ -663,3 +663,21 @@ def test_mask_based_patch_extractor_ndpi(
     len_region1 = len(patches)
 
     assert len_all > len_region2 > len_region1
+
+
+def test_invalid_points_type() -> None:
+    """Test invalid locations_list type for PointsPatchExtractor."""
+    img = np.zeros((256, 256, 3))
+    coords = [[10, 10]]
+    with pytest.raises(
+        TypeError,
+        match="Please input correct locations_list",
+    ):
+        patchextraction.get_patch_extractor(
+            "point", input_img=img, locations_list=coords, patch_size=38
+        )
+
+    patches = patchextraction.get_patch_extractor(
+        "point", input_img=img, locations_list=np.array(coords), patch_size=38
+    )
+    assert len(patches) > 0

tiatoolbox/tools/patchextraction.py

@@ -10,7 +10,7 @@
 
 from tiatoolbox import logger
 from tiatoolbox.utils import misc
-from tiatoolbox.utils.exceptions import MethodNotSupportedError
+from tiatoolbox.utils.exceptions import FileNotSupportedError, MethodNotSupportedError
 from tiatoolbox.utils.visualization import AnnotationRenderer
 from tiatoolbox.wsicore import wsireader
 
@@ -772,7 +772,11 @@ def __init__(
             pad_constant_values=pad_constant_values,
             within_bound=within_bound,
         )
-        self.locations_df = misc.read_locations(input_table=locations_list)
+        try:
+            self.locations_df = misc.read_locations(input_table=locations_list)
+        except (TypeError, FileNotSupportedError) as exc:
+            msg = "Please input correct locations_list"
+            raise TypeError(msg) from exc
         self.locations_df["x"] = self.locations_df["x"] - int(
             (self.patch_size[1] - 1) / 2,
         )

tiatoolbox/utils/misc.py

@@ -540,7 +540,7 @@ def read_locations(
     if isinstance(input_table, pd.DataFrame):
         return __assign_unknown_class(input_table)
 
-    msg = "Please input correct image path or an ndarray image."
+    msg = "File type not supported."
     raise TypeError(msg)