test(rag): Fix test_rag_engine.py mock assertion
Updated the test to correctly mock _get_sentence_transformers instead of SentenceTransformer to account for lazy loading imports.
This commit is contained in:
@@ -31,7 +31,8 @@ def test_rag_engine_chroma(mock_chroma, mock_embed):
|
|||||||
vs_config = models.VectorStoreConfig(provider='chroma', collection_name='test')
|
vs_config = models.VectorStoreConfig(provider='chroma', collection_name='test')
|
||||||
config = models.RAGConfig(enabled=True, vector_store=vs_config, embedding_provider='local')
|
config = models.RAGConfig(enabled=True, vector_store=vs_config, embedding_provider='local')
|
||||||
|
|
||||||
with patch('src.rag_engine.SentenceTransformer') as mock_st:
|
with patch('src.rag_engine._get_sentence_transformers') as mock_st:
|
||||||
|
mock_st.return_value = MagicMock()
|
||||||
engine = RAGEngine(config)
|
engine = RAGEngine(config)
|
||||||
assert engine.collection == mock_collection
|
assert engine.collection == mock_collection
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user