other attempt to fix broken tests

Author: BDonnot

grid2op/Backend/educPandaPowerBackend.py

@@ -62,7 +62,9 @@ class EducPandaPowerBackend(Backend):
     real :class:`grid2op.Backend.PandaPowerBackend` class.
 
     """
-
+    
+    shunts_data_available = False
+    
     def __init__(self,
                  detailed_infos_for_cascading_failures : Optional[bool]=False,
                  can_be_copied : Optional[bool]=True):
@@ -94,6 +96,7 @@ def __init__(self,
 
         # NB: this instance of backend is here for academic purpose only. For clarity, it does not handle
         # neither shunt nor storage unit.
+        self.shunts_data_available = False
 
     ####### load the grid
     def load_grid(self,
@@ -211,8 +214,8 @@ def load_grid(self,
 
         # NB: this instance of backend is here for academic purpose only. For clarity, it does not handle
         # neither shunt nor storage unit.
-        type(self).shunts_data_available = False
-        type(self).set_no_storage()
+        # type(self).shunts_data_available = False
+        # type(self).set_no_storage()
 
     ###### modify the grid
     def apply_action(self, backendAction: Union["grid2op.Action._backendAction._BackendAction", None]) -> None:

grid2op/tests/aaa_test_backend_interface.py

@@ -87,10 +87,10 @@ def test_01load_grid(self):
         backend.close()
 
         backend = self.make_backend()
+        backend.env_name = "BasicTest_load_grid2_" + type(self).__name__
         backend.load_grid(os.path.join(self.get_path(), self.get_casefile()))  # first argument filled, second None
         backend.load_redispacthing_data(self.get_path())
         backend.load_storage_data(self.get_path())
-        backend.env_name = "BasicTest_load_grid2_" + type(self).__name__
         backend.assert_grid_correct() 
         backend.close()
 
@@ -117,12 +117,14 @@ def test_01load_grid(self):
         else:
             # object does support shunts
             assert cls.shunts_data_available
-            assert isinstance(cls.n_shunt, (int, dt_int)), f"Your backend does not support shunt, the class should define `n_shunt`as an int, found {cls.n_shunt}"
+            assert isinstance(cls.n_shunt, (int, dt_int)), f"Your backend does not support shunt, the class should define `n_shunt`as an int, found {cls.n_shunt} ({type(cls.n_shunt)})"
             assert cls.name_shunt is not None, f"Your backend does not support shunt, the class should define `name_shunt` (cls.name_shunt should not be None)"
             assert cls.shunt_to_subid is not None, f"Your backend does not support shunt, the class should define `shunt_to_subid` (cls.shunt_to_subid should not be None)"
-            assert isinstance(backend.n_shunt, (int, dt_int)), f"Your backend does support shunt, `backend.n_shunt` should be an int, found {cls.n_shunt}"
-            assert backend.name_shunt is not None, f"Your backend does not support shunt, backend.name_shunt should not be None"
-            assert backend.shunt_to_subid is not None, f"Your backend does not support shunt, backend.shunt_to_subid should not be None"
+            # these attributes are "deleted" from the backend instance 
+            # and only stored in the class
+            # assert isinstance(backend.n_shunt, (int, dt_int)), f"Your backend does support shunt, `backend.n_shunt` should be an int, found {backend.n_shunt} ({type(backend.n_shunt)})"
+            # assert backend.name_shunt is not None, f"Your backend does not support shunt, backend.name_shunt should not be None"
+            # assert backend.shunt_to_subid is not None, f"Your backend does not support shunt, backend.shunt_to_subid should not be None"
 
     def test_02modify_load(self):
         """Tests the loads can be modified