Skip to content

Commit a984dc1

Browse files
committed
ruff, black
1 parent 52e06c4 commit a984dc1

File tree

3 files changed

+32
-17
lines changed

3 files changed

+32
-17
lines changed

tests/conftest.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ async def mock_search(self, *args, **kwargs):
6060

6161

6262
async def mock_retrieve(self, *args, **kwargs):
63-
return
63+
return
6464

6565

6666
@pytest.fixture
@@ -250,6 +250,7 @@ def patch(openai_client):
250250

251251
return patch
252252

253+
253254
@pytest.fixture
254255
def mock_acs_search(monkeypatch):
255256
monkeypatch.setattr(SearchClient, "search", mock_search)

tests/mocks.py

Lines changed: 25 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,19 @@
66
import openai.types
77
from azure.cognitiveservices.speech import ResultReason
88
from azure.core.credentials_async import AsyncTokenCredential
9+
from azure.search.documents.agent.models import (
10+
KnowledgeAgentAzureSearchDocReference,
11+
KnowledgeAgentMessage,
12+
KnowledgeAgentMessageTextContent,
13+
KnowledgeAgentModelQueryPlanningActivityRecord,
14+
KnowledgeAgentRetrievalResponse,
15+
KnowledgeAgentSearchActivityRecord,
16+
KnowledgeAgentSearchActivityRecordQuery,
17+
)
918
from azure.search.documents.models import (
1019
VectorQuery,
1120
)
1221
from azure.storage.blob import BlobProperties
13-
from azure.search.documents.agent.models import KnowledgeAgentRetrievalResponse, KnowledgeAgentMessage, KnowledgeAgentMessageTextContent, KnowledgeAgentModelQueryPlanningActivityRecord, KnowledgeAgentSearchActivityRecord, KnowledgeAgentSearchActivityRecordQuery, KnowledgeAgentAzureSearchDocReference
14-
1522

1623
MOCK_EMBEDDING_DIMENSIONS = 1536
1724
MOCK_EMBEDDING_MODEL_NAME = "text-embedding-ada-002"
@@ -209,31 +216,40 @@ def mock_computervision_response():
209216
),
210217
)
211218

219+
212220
def mock_retrieval_response():
213221
return KnowledgeAgentRetrievalResponse(
214222
response=[
215223
KnowledgeAgentMessage(
216224
role="assistant",
217-
content=[KnowledgeAgentMessageTextContent(text=r'[{"ref_id":0,"title":"Benefit_Options-2.pdf","content":"There is a whistleblower policy."}]')],
225+
content=[
226+
KnowledgeAgentMessageTextContent(
227+
text=r'[{"ref_id":0,"title":"Benefit_Options-2.pdf","content":"There is a whistleblower policy."}]'
228+
)
229+
],
218230
)
219231
],
220232
activity=[
221233
KnowledgeAgentModelQueryPlanningActivityRecord(id=0, input_tokens=10, output_tokens=20, elapsed_ms=200),
222-
KnowledgeAgentSearchActivityRecord(id=1, target_index="index", query=KnowledgeAgentSearchActivityRecordQuery(search="whistleblower query"),count=10, elapsed_ms=50)
234+
KnowledgeAgentSearchActivityRecord(
235+
id=1,
236+
target_index="index",
237+
query=KnowledgeAgentSearchActivityRecordQuery(search="whistleblower query"),
238+
count=10,
239+
elapsed_ms=50,
240+
),
223241
],
224242
references=[
225243
KnowledgeAgentAzureSearchDocReference(
226244
id=0,
227245
activity_source=1,
228246
doc_key="Benefit_Options-2.pdf",
229-
source_data={
230-
"content": "There is a whistleblower policy.",
231-
"sourcepage": "Benefit_Options-2.pdf"
232-
}
247+
source_data={"content": "There is a whistleblower policy.", "sourcepage": "Benefit_Options-2.pdf"},
233248
)
234-
]
249+
],
235250
)
236251

252+
237253
class MockAudio:
238254
def __init__(self, audio_data):
239255
self.audio_data = audio_data

tests/test_chatapproach.py

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,13 +13,14 @@
1313
MOCK_EMBEDDING_DIMENSIONS,
1414
MOCK_EMBEDDING_MODEL_NAME,
1515
MockAsyncSearchResultsIterator,
16-
mock_retrieval_response
16+
mock_retrieval_response,
1717
)
1818

1919

2020
async def mock_search(*args, **kwargs):
2121
return MockAsyncSearchResultsIterator(kwargs.get("search_text"), kwargs.get("vector_queries"))
2222

23+
2324
async def mock_retrieval(*args, **kwargs):
2425
return mock_retrieval_response()
2526

@@ -264,6 +265,7 @@ async def validate_qr_and_mock_search(*args, **kwargs):
264265
assert len(results) == 1
265266
assert query_rewrites == "generative"
266267

268+
267269
@pytest.mark.asyncio
268270
async def test_agent_retrieval_results(monkeypatch):
269271
chat_approach = ChatReadRetrieveReadApproach(
@@ -284,18 +286,14 @@ async def test_agent_retrieval_results(monkeypatch):
284286
content_field="",
285287
query_language="en-us",
286288
query_speller="lexicon",
287-
prompt_manager=PromptyManager()
289+
prompt_manager=PromptyManager(),
288290
)
289291

290292
agent_client = KnowledgeAgentRetrievalClient(endpoint="", agent_name="", credential=AzureKeyCredential(""))
291293

292294
monkeypatch.setattr(KnowledgeAgentRetrievalClient, "retrieve", mock_retrieval)
293295

294-
_, results = await chat_approach.run_agentic_retrieval(
295-
messages=[],
296-
agent_client=agent_client,
297-
search_index_name=""
298-
)
296+
_, results = await chat_approach.run_agentic_retrieval(messages=[], agent_client=agent_client, search_index_name="")
299297

300298
assert len(results) == 1
301299
assert results[0].id == "Benefit_Options-2.pdf"

0 commit comments

Comments
 (0)