Consistently raise ImportError for missing dependencies

Author: vishal-bala

redisvl/index/index.py

@@ -288,7 +288,7 @@ def _validate_hybrid_query(self, query: Any) -> None:
 
             from redisvl.query.hybrid import HybridQuery
         except (ImportError, ModuleNotFoundError):
-            raise NotImplementedError(_HYBRID_SEARCH_ERROR_MESSAGE)
+            raise ImportError(_HYBRID_SEARCH_ERROR_MESSAGE)
 
         if not isinstance(query, HybridQuery):
             raise TypeError(f"query must be of type HybridQuery, got {type(query)}")
@@ -1054,7 +1054,7 @@ def _hybrid_search(self, query: HybridQuery, **kwargs) -> List[Dict[str, Any]]:
         self._validate_hybrid_query(query)
 
         if not hasattr(index, "hybrid_search"):
-            raise NotImplementedError(_HYBRID_SEARCH_ERROR_MESSAGE)
+            raise ImportError(_HYBRID_SEARCH_ERROR_MESSAGE)
 
         results = index.hybrid_search(
             query=query.query,
@@ -1928,7 +1928,7 @@ async def _hybrid_search(
         self._validate_hybrid_query(query)
 
         if not hasattr(index, "hybrid_search"):
-            raise NotImplementedError(_HYBRID_SEARCH_ERROR_MESSAGE)
+            raise ImportError(_HYBRID_SEARCH_ERROR_MESSAGE)
 
         results = await index.hybrid_search(
             query=query.query,

tests/integration/test_hybrid.py

@@ -512,5 +512,5 @@ def test_hybrid_query_not_available(index):
     REDIS_HYBRID_AVAILABLE, reason="Requires hybrid search to NOT be available"
 )
 def test_hybrid_search_method_missing(index):
-    with pytest.raises(NotImplementedError):
+    with pytest.raises(ImportError):
         index.query(None)