test: ignore code coverage

Awabalii · Dec 20, 2022 · 986fdae · 986fdae
1 parent 1cfa6f2
commit 986fdae
Showing 1 changed file with 3 additions and 9 deletions.
diff --git a/mava/environment_loop.py b/mava/environment_loop.py
@@ -25,7 +25,6 @@
 import jax
 import jax.numpy as jnp
 import numpy as np
-import pytest
 from acme.utils import counting, loggers
 
 import mava
@@ -203,10 +202,7 @@ def run_episode_and_log(self) -> loggers.LoggingData:
         self._logger.write(results)
         return results
 
-    # TODO (Omayma): remove this condition when we fix the coverage bot to be able
-    # to use the integration tests or when we create a unit test for the env loop
-    @pytest.mark.skipif(True, reason="Skip this function for code coverage")
-    def run(self) -> None:  # noqa: C901
+    def run(self) -> None:  # pragma: no cover # noqa: C901
         """Run the environment loop."""
 
         def should_run_loop(eval_interval_condition: Tuple) -> bool:
@@ -251,8 +247,7 @@ def should_run_loop(eval_interval_condition: Tuple) -> bool:
             )
             evaluation_duration = eval_duration_condition[1]
 
-        @pytest.mark.skipif(True, reason="Skip this function for code coverage")
-        def run_evaluation(results: Any) -> None:
+        def run_evaluation(results: Any) -> None:  # pragma: no cover
             """Calculate the absolute metric"""
 
             normalisation = self._executor.has(BaseNormalisation) and (
@@ -330,8 +325,7 @@ def run_evaluation(results: Any) -> None:
                 {"terminate": True}
             )
 
-        @pytest.mark.skipif(True, reason="Skip this function for code coverage")
-        def step_executor() -> None:
+        def step_executor() -> None:  # pragma: no cover
             if (not environment_loop_schedule) or (
                 should_run_loop(eval_interval_condition)
             ):