test: fix anchor unit test

Author: AhmetNSimsek

pytest.ini

@@ -13,5 +13,6 @@ testpaths =
     test/features/test_genes.py
     test/features/test_receptors.py
     test/features/test_voi.py
+    test/features/test_anchor.py
     # eventually, only use the below ini
     # test/

test/features/test_anchor.py

@@ -1,7 +1,8 @@
 import pytest
+from unittest.mock import patch, Mock
+from siibra.commons import Species
 from siibra.core.region import Region
-from siibra.features.anchor import AnatomicalAnchor, Parcellation, Species
-from unittest.mock import patch
+from siibra.features.anchor import AnatomicalAnchor
 
 
 @pytest.fixture
@@ -16,21 +17,22 @@ def fixture_teardown():
 ]
 
 
-@pytest.mark.parametrize("region,teardown", region_specs)
-def test_region_region_spec(region, teardown):
-    mock_found_regions = [Region("baz"), Region("hello world")]
+@pytest.mark.parametrize("region_spec,teardown", region_specs)
+@patch("siibra.features.anchor.find_regions")
+def test_region_region_spec(mock_find_regions: Mock, region_spec, teardown):
+    mock_find_regions.return_value = [Region("baz"), Region("hello world")]
     species = Species.UNSPECIFIED_SPECIES
-    with patch.object(
-        Parcellation, "find_regions", return_value=mock_found_regions
-    ) as mock_find_regions:
-        anchor = AnatomicalAnchor(species, region=region)
-        assert isinstance(anchor.regions, dict)
-        for _region in anchor.regions:
-            assert isinstance(_region, Region)
-
-        if isinstance(region, Region):
-            mock_find_regions.assert_not_called()
-        elif isinstance(region, str):
-            mock_find_regions.assert_called_once_with(region, species)
-        else:
-            assert False, "Cannot have region as neither str or Region"
+    for r in mock_find_regions.return_value:
+        r._species_cached = species
+
+    anchor = AnatomicalAnchor(species, region=region_spec)
+    assert isinstance(anchor.regions, dict)
+    for _region in anchor.regions:
+        assert isinstance(_region, Region)
+
+    if isinstance(region_spec, Region):
+        mock_find_regions.assert_not_called()
+    elif isinstance(region_spec, str):
+        mock_find_regions.assert_called_once_with(region_spec, filter_children=True, find_topmost=False)
+    else:
+        assert False, "Cannot have region as neither str or Region"