Expand error coverage and tests

Author: abrookins

redisvl/extensions/cache/base.py

@@ -135,8 +135,9 @@ async def _get_async_redis_client(self) -> AsyncRedisClient:
                 url = str(self.redis_kwargs["redis_url"])
                 kwargs = self.redis_kwargs.get("connection_kwargs", {})
                 if not isinstance(kwargs, Mapping):
-                    raise ValueError(
-                        f"connection_kwargs must be a mapping, got {type(kwargs)}"
+                    raise TypeError(
+                        "Expected `connection_kwargs` to be a dictionary (e.g. {'decode_responses': True}), "
+                        f"but got type: {type(kwargs).__name__}"
                     )
                 self._async_redis_client = (
                     RedisConnectionFactory.get_async_redis_connection(url, **kwargs)

redisvl/extensions/router/semantic.py

@@ -137,7 +137,9 @@ def from_existing(
 
         router_dict = redis_client.json().get(f"{name}:route_config")
         if not isinstance(router_dict, dict):
-            raise ValueError(f"No valid router config found for {name}")
+            raise ValueError(
+                f"No valid router config found for {name}. Received: {router_dict!r}"
+            )
         return cls.from_dict(
             router_dict, redis_url=redis_url, redis_client=redis_client
         )