Fixed some comments from review

Author: jmafoster1

causal_testing/json_front/json_class.py

@@ -278,8 +278,6 @@ def _execute_test_case(
         if "coverage" in test and test["coverage"]:
             adequacy_metric = DataAdequacy(causal_test_case, estimation_model, self.data_collector)
             adequacy_metric.measure_adequacy()
-            # self._append_to_file(f"KURTOSIS: {effect_estimate.mean()}", logging.INFO)
-            # self._append_to_file(f"PASSING: {sum(outcomes)}/{len(outcomes)}", logging.INFO)
             causal_test_result.adequacy = adequacy_metric
 
         if causal_test_result.ci_low() is not None and causal_test_result.ci_high() is not None:

causal_testing/testing/causal_test_outcome.py

@@ -58,12 +58,6 @@ def apply(self, res: CausalTestResult) -> bool:
             ci_high = res.ci_high() if isinstance(res.ci_high(), Iterable) else [res.ci_high()]
             value = res.test_value.value if isinstance(res.ci_high(), Iterable) else [res.test_value.value]
 
-            # if not all(ci_low < 0 < ci_high for ci_low, ci_high in zip(ci_low, ci_high)):
-            #     print(
-            #         "FAILING ON",
-            #         [(ci_low, ci_high) for ci_low, ci_high in zip(ci_low, ci_high) if not ci_low < 0 < ci_high],
-            #     )
-
             return (
                 sum(
                     not ((ci_low < 0 < ci_high) or abs(v) < self.atol)

tests/testing_tests/test_causal_test_adequacy.py

@@ -17,11 +17,10 @@
 from causal_testing.specification.causal_specification import CausalSpecification
 
 
-class TestJsonClass(unittest.TestCase):
-    """Test the JSON frontend for the Causal Testing Framework (CTF)
-
-    The JSON frontend is an alternative interface for the CTF where tests are specified in JSON format and ingested
-    with the frontend. Tests involve testing that this correctly interfaces with the framework with some dummy data
+class TestCausalTestAdequacy(unittest.TestCase):
+    """
+    Test the causal test adequacy metrics. These provide metrics determine how comprehensive a test set is and,
+    ultimately whether we can stop testing.
     """
 
     def setUp(self) -> None: