langchain/libs/community/tests/unit_tests/retrievers/test_base.py
Bagatur ed58eeb9c5
community[major], core[patch], langchain[patch], experimental[patch]: Create langchain-community (#14463)
Moved the following modules to new package langchain-community in a backwards compatible fashion:

```
mv langchain/langchain/adapters community/langchain_community
mv langchain/langchain/callbacks community/langchain_community/callbacks
mv langchain/langchain/chat_loaders community/langchain_community
mv langchain/langchain/chat_models community/langchain_community
mv langchain/langchain/document_loaders community/langchain_community
mv langchain/langchain/docstore community/langchain_community
mv langchain/langchain/document_transformers community/langchain_community
mv langchain/langchain/embeddings community/langchain_community
mv langchain/langchain/graphs community/langchain_community
mv langchain/langchain/llms community/langchain_community
mv langchain/langchain/memory/chat_message_histories community/langchain_community
mv langchain/langchain/retrievers community/langchain_community
mv langchain/langchain/storage community/langchain_community
mv langchain/langchain/tools community/langchain_community
mv langchain/langchain/utilities community/langchain_community
mv langchain/langchain/vectorstores community/langchain_community
mv langchain/langchain/agents/agent_toolkits community/langchain_community
mv langchain/langchain/cache.py community/langchain_community
mv langchain/langchain/adapters community/langchain_community
mv langchain/langchain/callbacks community/langchain_community/callbacks
mv langchain/langchain/chat_loaders community/langchain_community
mv langchain/langchain/chat_models community/langchain_community
mv langchain/langchain/document_loaders community/langchain_community
mv langchain/langchain/docstore community/langchain_community
mv langchain/langchain/document_transformers community/langchain_community
mv langchain/langchain/embeddings community/langchain_community
mv langchain/langchain/graphs community/langchain_community
mv langchain/langchain/llms community/langchain_community
mv langchain/langchain/memory/chat_message_histories community/langchain_community
mv langchain/langchain/retrievers community/langchain_community
mv langchain/langchain/storage community/langchain_community
mv langchain/langchain/tools community/langchain_community
mv langchain/langchain/utilities community/langchain_community
mv langchain/langchain/vectorstores community/langchain_community
mv langchain/langchain/agents/agent_toolkits community/langchain_community
mv langchain/langchain/cache.py community/langchain_community
```

Moved the following to core
```
mv langchain/langchain/utils/json_schema.py core/langchain_core/utils
mv langchain/langchain/utils/html.py core/langchain_core/utils
mv langchain/langchain/utils/strings.py core/langchain_core/utils
cat langchain/langchain/utils/env.py >> core/langchain_core/utils/env.py
rm langchain/langchain/utils/env.py
```

See .scripts/community_split/script_integrations.sh for all changes
2023-12-11 13:53:30 -08:00

220 lines
7.8 KiB
Python

"""Test Base Retriever logic."""
from __future__ import annotations
from typing import Dict, List, Optional
import pytest
from langchain_core.callbacks import (
AsyncCallbackManagerForRetrieverRun,
CallbackManagerForRetrieverRun,
)
from langchain_core.retrievers import BaseRetriever, Document
from tests.unit_tests.callbacks.fake_callback_handler import FakeCallbackHandler
@pytest.fixture
def fake_retriever_v1() -> BaseRetriever:
with pytest.warns(
DeprecationWarning,
match="Retrievers must implement abstract "
"`_get_relevant_documents` method instead of `get_relevant_documents`",
):
class FakeRetrieverV1(BaseRetriever):
def get_relevant_documents( # type: ignore[override]
self,
query: str,
) -> List[Document]:
assert isinstance(self, FakeRetrieverV1)
return [
Document(page_content=query, metadata={"uuid": "1234"}),
]
async def aget_relevant_documents( # type: ignore[override]
self,
query: str,
) -> List[Document]:
assert isinstance(self, FakeRetrieverV1)
return [
Document(
page_content=f"Async query {query}", metadata={"uuid": "1234"}
),
]
return FakeRetrieverV1() # type: ignore[abstract]
def test_fake_retriever_v1_upgrade(fake_retriever_v1: BaseRetriever) -> None:
callbacks = FakeCallbackHandler()
assert fake_retriever_v1._new_arg_supported is False
assert fake_retriever_v1._expects_other_args is False
results: List[Document] = fake_retriever_v1.get_relevant_documents(
"Foo", callbacks=[callbacks]
)
assert results[0].page_content == "Foo"
assert callbacks.retriever_starts == 1
assert callbacks.retriever_ends == 1
assert callbacks.retriever_errors == 0
async def test_fake_retriever_v1_upgrade_async(
fake_retriever_v1: BaseRetriever,
) -> None:
callbacks = FakeCallbackHandler()
assert fake_retriever_v1._new_arg_supported is False
assert fake_retriever_v1._expects_other_args is False
results: List[Document] = await fake_retriever_v1.aget_relevant_documents(
"Foo", callbacks=[callbacks]
)
assert results[0].page_content == "Async query Foo"
assert callbacks.retriever_starts == 1
assert callbacks.retriever_ends == 1
assert callbacks.retriever_errors == 0
@pytest.fixture
def fake_retriever_v1_with_kwargs() -> BaseRetriever:
# Test for things like the Weaviate V1 Retriever.
with pytest.warns(
DeprecationWarning,
match="Retrievers must implement abstract "
"`_get_relevant_documents` method instead of `get_relevant_documents`",
):
class FakeRetrieverV1(BaseRetriever):
def get_relevant_documents( # type: ignore[override]
self, query: str, where_filter: Optional[Dict[str, object]] = None
) -> List[Document]:
assert isinstance(self, FakeRetrieverV1)
return [
Document(page_content=query, metadata=where_filter or {}),
]
async def aget_relevant_documents( # type: ignore[override]
self, query: str, where_filter: Optional[Dict[str, object]] = None
) -> List[Document]:
assert isinstance(self, FakeRetrieverV1)
return [
Document(
page_content=f"Async query {query}", metadata=where_filter or {}
),
]
return FakeRetrieverV1() # type: ignore[abstract]
def test_fake_retriever_v1_with_kwargs_upgrade(
fake_retriever_v1_with_kwargs: BaseRetriever,
) -> None:
callbacks = FakeCallbackHandler()
assert fake_retriever_v1_with_kwargs._new_arg_supported is False
assert fake_retriever_v1_with_kwargs._expects_other_args is True
results: List[Document] = fake_retriever_v1_with_kwargs.get_relevant_documents(
"Foo", callbacks=[callbacks], where_filter={"foo": "bar"}
)
assert results[0].page_content == "Foo"
assert results[0].metadata == {"foo": "bar"}
assert callbacks.retriever_starts == 1
assert callbacks.retriever_ends == 1
assert callbacks.retriever_errors == 0
async def test_fake_retriever_v1_with_kwargs_upgrade_async(
fake_retriever_v1_with_kwargs: BaseRetriever,
) -> None:
callbacks = FakeCallbackHandler()
assert fake_retriever_v1_with_kwargs._new_arg_supported is False
assert fake_retriever_v1_with_kwargs._expects_other_args is True
results: List[
Document
] = await fake_retriever_v1_with_kwargs.aget_relevant_documents(
"Foo", callbacks=[callbacks], where_filter={"foo": "bar"}
)
assert results[0].page_content == "Async query Foo"
assert results[0].metadata == {"foo": "bar"}
assert callbacks.retriever_starts == 1
assert callbacks.retriever_ends == 1
assert callbacks.retriever_errors == 0
class FakeRetrieverV2(BaseRetriever):
throw_error: bool = False
def _get_relevant_documents(
self,
query: str,
*,
run_manager: Optional[CallbackManagerForRetrieverRun] = None,
) -> List[Document]:
assert isinstance(self, FakeRetrieverV2)
assert run_manager is not None
assert isinstance(run_manager, CallbackManagerForRetrieverRun)
if self.throw_error:
raise ValueError("Test error")
return [
Document(page_content=query),
]
async def _aget_relevant_documents(
self,
query: str,
*,
run_manager: Optional[AsyncCallbackManagerForRetrieverRun] = None,
) -> List[Document]:
assert isinstance(self, FakeRetrieverV2)
assert run_manager is not None
assert isinstance(run_manager, AsyncCallbackManagerForRetrieverRun)
if self.throw_error:
raise ValueError("Test error")
return [
Document(page_content=f"Async query {query}"),
]
@pytest.fixture
def fake_retriever_v2() -> BaseRetriever:
return FakeRetrieverV2() # type: ignore[abstract]
@pytest.fixture
def fake_erroring_retriever_v2() -> BaseRetriever:
return FakeRetrieverV2(throw_error=True) # type: ignore[abstract]
def test_fake_retriever_v2(
fake_retriever_v2: BaseRetriever, fake_erroring_retriever_v2: BaseRetriever
) -> None:
callbacks = FakeCallbackHandler()
assert fake_retriever_v2._new_arg_supported is True
results = fake_retriever_v2.get_relevant_documents("Foo", callbacks=[callbacks])
assert results[0].page_content == "Foo"
assert callbacks.retriever_starts == 1
assert callbacks.retriever_ends == 1
assert callbacks.retriever_errors == 0
fake_retriever_v2.get_relevant_documents("Foo", callbacks=[callbacks])
with pytest.raises(ValueError, match="Test error"):
fake_erroring_retriever_v2.get_relevant_documents("Foo", callbacks=[callbacks])
assert callbacks.retriever_errors == 1
async def test_fake_retriever_v2_async(
fake_retriever_v2: BaseRetriever, fake_erroring_retriever_v2: BaseRetriever
) -> None:
callbacks = FakeCallbackHandler()
assert fake_retriever_v2._new_arg_supported is True
results = await fake_retriever_v2.aget_relevant_documents(
"Foo", callbacks=[callbacks]
)
assert results[0].page_content == "Async query Foo"
assert callbacks.retriever_starts == 1
assert callbacks.retriever_ends == 1
assert callbacks.retriever_errors == 0
await fake_retriever_v2.aget_relevant_documents("Foo", callbacks=[callbacks])
with pytest.raises(ValueError, match="Test error"):
await fake_erroring_retriever_v2.aget_relevant_documents(
"Foo", callbacks=[callbacks]
)