Refactor to use ** instead of catching

Author: christopher-wild

causal_testing/testing/causal_test_case.py

@@ -30,7 +30,7 @@ def __init__(
         control_value: Any = None,
         treatment_value: Any = None,
         estimate_type: str = "ate",
-        estimate_params: dict = "None",
+        estimate_params: dict = None,
         effect_modifier_configuration: dict[Variable:Any] = None,
     ):
         """
@@ -48,7 +48,8 @@ def __init__(
         self.treatment_variable = base_test_case.treatment_variable
         self.treatment_value = treatment_value
         self.estimate_type = estimate_type
-        self.estimate_params = estimate_params
+        if estimate_params is None:
+            self.estimate_params = {}
         self.effect = base_test_case.effect
 
         if effect_modifier_configuration:

causal_testing/testing/causal_test_engine.py

@@ -162,10 +162,8 @@ def _return_causal_test_results(self, estimator, causal_test_case):
             )
         elif causal_test_case.estimate_type == "risk_ratio":
             logger.debug("calculating risk_ratio")
-            try:
-                risk_ratio, confidence_intervals = estimator.estimate_risk_ratio(causal_test_case.estimate_params)
-            except TypeError:
-                risk_ratio, confidence_intervals = estimator.estimate_risk_ratio()
+            risk_ratio, confidence_intervals = estimator.estimate_risk_ratio(**causal_test_case.estimate_params)
+
             causal_test_result = CausalTestResult(
                 estimator=estimator,
                 test_value=TestValue("risk_ratio", risk_ratio),