sneaky from_existing test

Author: rbs333

redisvl/extensions/router/semantic.py

@@ -101,6 +101,8 @@ def __init__(
             routes=routes,
             vectorizer=vectorizer,
             routing_config=routing_config,
+            redis_url=redis_url,
+            redis_client=redis_client,
         )
 
         self._initialize_index(redis_client, redis_url, overwrite, **connection_kwargs)
@@ -130,7 +132,9 @@ def from_existing(
             )
 
         router_dict = redis_client.json().get(f"{name}:route_config")  # type: ignore
-        return cls.from_dict(router_dict)
+        return cls.from_dict(
+            router_dict, redis_url=redis_url, redis_client=redis_client
+        )
 
     @deprecated_argument("dtype")
     def _initialize_index(

tests/integration/test_threshold_optimizer.py

@@ -113,7 +113,7 @@ def test_routes_different_distance_thresholds_optimizer_default(
 
     # now run optimizer
     router_optimizer = RouterThresholdOptimizer(router, test_data_optimization)
-    router_optimizer.optimize(max_iterations=10, search_step=0.5)
+    router_optimizer.optimize(max_iterations=20, search_step=0.5)
 
     # test that it updated thresholds beyond the null case
     for route in routes:
@@ -150,7 +150,7 @@ def test_routes_different_distance_thresholds_optimizer_precision(
     router_optimizer = RouterThresholdOptimizer(
         router, test_data_optimization, eval_metric="precision"
     )
-    router_optimizer.optimize(max_iterations=10, search_step=0.5)
+    router_optimizer.optimize(max_iterations=20, search_step=0.5)
 
     # test that it updated thresholds beyond the null case
     for route in routes:
@@ -186,7 +186,7 @@ def test_routes_different_distance_thresholds_optimizer_recall(
     router_optimizer = RouterThresholdOptimizer(
         router, test_data_optimization, eval_metric="recall"
     )
-    router_optimizer.optimize(max_iterations=10, search_step=0.5)
+    router_optimizer.optimize(max_iterations=20, search_step=0.5)
 
     # test that it updated thresholds beyond the null case
     for route in routes: