pass dc fns

Author: mrlevitas

reddit_decider/__init__.py

@@ -712,7 +712,7 @@ def get_bool(self, feature_name: str, default: bool = False) -> bool:
 
         :return: the boolean value of the dyanimc config if it is active/exists, :code:`default` parameter otherwise.
         """
-        return self._get_dynamic_config_value(feature_name, default, bool)
+        return self._get_dynamic_config_value(feature_name, default, bool, self._internal.get_bool)
 
     def get_int(self, feature_name: str, default: int = 0) -> int:
         """Fetch a Dynamic Configuration of int type.
@@ -724,7 +724,7 @@ def get_int(self, feature_name: str, default: int = 0) -> int:
 
         :return: the int value of the dyanimc config if it is active/exists, :code:`default` parameter otherwise.
         """
-        return self._get_dynamic_config_value(feature_name, default, int)
+        return self._get_dynamic_config_value(feature_name, default, int, self._internal.get_int)
 
     def get_float(self, feature_name: str, default: float = 0.0) -> float:
         """Fetch a Dynamic Configuration of float type.
@@ -736,7 +736,7 @@ def get_float(self, feature_name: str, default: float = 0.0) -> float:
 
         :return: the float value of the dyanimc config if it is active/exists, :code:`default` parameter otherwise.
         """
-        return self._get_dynamic_config_value(feature_name, default, float)
+        return self._get_dynamic_config_value(feature_name, default, float, self._internal.get_float)
 
     def get_string(self, feature_name: str, default: str = "") -> str:
         """Fetch a Dynamic Configuration of string type.
@@ -748,7 +748,7 @@ def get_string(self, feature_name: str, default: str = "") -> str:
 
         :return: the string value of the dyanimc config if it is active/exists, :code:`default` parameter otherwise.
         """
-        return self._get_dynamic_config_value(feature_name, default, str)
+        return self._get_dynamic_config_value(feature_name, default, str, self._internal.get_string)
 
     def get_map(self, feature_name: str, default: Optional[dict] = None) -> Optional[dict]:
         """Fetch a Dynamic Configuration of map type.
@@ -760,7 +760,7 @@ def get_map(self, feature_name: str, default: Optional[dict] = None) -> Optional
 
         :return: the map value of the dyanimc config if it is active/exists, :code:`default` parameter otherwise.
         """
-        return self._get_dynamic_config_value(feature_name, default, dict)
+        return self._get_dynamic_config_value(feature_name, default, dict, self._internal.get_map)
 
     def get_all_dynamic_configs(self) -> List[Dict[str, Any]]:
         """Return a list of dynamic configuration dicts in this format:
@@ -819,6 +819,7 @@ def _get_dynamic_config_value(
         feature_name: str,
         default: Any,
         dc_type: Type[T],
+        get_fn: Callable,
     ) -> T:
         if self._internal is None:
             logger.error("rs_decider is None--did not initialize.")
@@ -827,9 +828,7 @@ def _get_dynamic_config_value(
         ctx = self._decider_context.to_dict()
 
         try:
-            value = eval(f"self._internal.get_{dc_type.__name__}")(
-                feature_name=feature_name, context=ctx
-            )
+            value = get_fn(feature_name=feature_name, context=ctx)
         except FeatureNotFoundException as exc:
             warnings.warn(str(exc))
             return default