Addressed comments

Author: jmafoster1

causal_testing/json_front/json_class.py

@@ -231,6 +231,9 @@ def _setup_test(
         """Create the necessary inputs for a single test case
         :param causal_test_case: The concrete test case to be executed
         :param test: Single JSON test definition stored in a mapping (dict)
+        :param conditions: A list of conditions which should be applied to the
+        data. Conditions should be in the query format detailed at
+        https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.query.html
         :returns:
                 - causal_test_engine - Test Engine instance for the test being run
                 - estimation_model - Estimator instance for the test being run

causal_testing/specification/causal_dag.py

@@ -521,7 +521,7 @@ def identification(self, base_test_case: BaseTestCase):
         minimal_adjustment_set = min(minimal_adjustment_sets, key=len)
         return minimal_adjustment_set
 
-    def to_dot(self) -> str:
+    def to_dot_string(self) -> str:
         """Return a string of the DOT representation of the causal DAG.
         :return DOT string of the DAG.
         """

tests/specification_tests/test_causal_dag.py

@@ -99,9 +99,9 @@ def test_empty_casual_dag(self):
         causal_dag = CausalDAG()
         assert list(causal_dag.graph.nodes) == [] and list(causal_dag.graph.edges) == []
 
-    def test_to_dot(self):
+    def test_to_dot_string(self):
         causal_dag = CausalDAG(self.dag_dot_path)
-        self.assertEqual(causal_dag.to_dot(), """digraph G {\nA -> B;\nB -> C;\nD -> A;\nD -> C;\n}""")
+        self.assertEqual(causal_dag.to_dot_string(), """digraph G {\nA -> B;\nB -> C;\nD -> A;\nD -> C;\n}""")
 
     def tearDown(self) -> None:
         remove_temp_dir_if_existent()