fix feature unit test and enable all unit tests

Author: AhmetNSimsek

pytest.ini

@@ -1,18 +1,4 @@
 [pytest]
 pythonpath = .
 addopts = -rf
-testpaths = 
-    # slowly add tests, until all tests are added 
-    test/test_siibra.py
-    test/retrieval/
-    test/core/
-    test/volumes/
-    test/features/external/
-    test/features/test_cells.py
-    test/features/test_connectivity.py
-    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/
+testpaths = test/

siibra/features/feature.py

@@ -624,14 +624,25 @@ def _wrap_livequery_feature(feature: 'Feature', fid: str):
 
         See docstring of serialize_query_context for further context.
         """
-        class ProxyFeature(feature.__class__, do_not_index=True):
+        class ProxyFeature(feature.__class__):
 
             # override __class__ property
             # some instances of features accesses inst.__class__
             @property
             def __class__(self):
                 return self.inst.__class__
 
+            def __init_subclass__(
+                cls,
+                configuration_folder=None,
+                category=None,
+                do_not_index=True,
+                **kwargs,
+            ):
+                return super().__init_subclass__(
+                    configuration_folder, category, do_not_index, **kwargs
+                )
+
             def __init__(self, inst: Feature, fid: str):
                 self.inst = inst
                 self.fid = fid

test/features/test_feature.py

@@ -13,6 +13,7 @@ class FooFeatureBase:
     def __init__(self, id) -> None:
         self.id = id
         self.other = "bla"
+        self.category = None
 
 
 class FooFeature(FooFeatureBase):
@@ -67,7 +68,7 @@ def test_serialize_query_context(feature, concept, ExCls, expected_id):
 
 @pytest.fixture
 def mock_parse_featuretype():
-    with patch.object(Feature, "parse_featuretype") as mock:
+    with patch.object(Feature, "_parse_featuretype") as mock:
         mock.return_value = [FooFeature, FooFeatureBase]
         yield mock
 
@@ -104,12 +105,12 @@ def mock_all(
 
 
 def test_mock_featuretype(mock_parse_featuretype):
-    feature_types = Feature.parse_featuretype()
+    feature_types = Feature._parse_featuretype()
     mock_parse_featuretype.assert_called_once()
     assert feature_types == [FooFeature, FooFeatureBase]
 
     mock_parse_featuretype.return_value = "foo"
-    feature_types = Feature.parse_featuretype()
+    feature_types = Feature._parse_featuretype()
     assert feature_types == "foo"
 
 
@@ -245,7 +246,8 @@ def test_deserialize_query_context(
 
 
 def test_wrap_feature():
-    new_feat = Feature._wrap_livequery_feature(feature_inst, "helloworld")
+    feature_inst.category = "vategory_before_wrapping"
+    new_feat = Feature._wrap_livequery_feature(feature_inst, fid="helloworld")
     assert new_feat.other == feature_inst.other
     assert new_feat.id != feature_inst.id
     assert new_feat.id == "helloworld"