ryan-crabbe-berri e9d40a8f73 test: enforce F811 so a duplicate definition cannot silently replace the first ... A name bound twice keeps only the second binding. In `tests/` that is nearly always a repeated import, harmless but misleading, and the same rule is what catches the cases that are not harmless: a local that shadows an import the module still calls, and a second `def test_x` that quietly replaces the first. 311 of the 344 sites were repeated imports and came out with ruff's own fix. The remaining 33 needed a decision. Four modules imported a name they never used because a local definition below already shadowed it. Two comprehensions bound `call` over `unittest.mock.call`, which those modules import and use. One test rebound the two module handles its nested reload closure had captured. One class attribute shadowed an unused `status` import. The load-test fixtures move to a conftest, which is how pytest is meant to share them, so the test module no longer imports three fixture names it never calls. The nine `prisma_client` parameters keep a narrow `noqa`: pytest resolves that fixture by name before the body runs, so the parameter never shadows anything.		2026-08-21 12:06:19 -07:00
..
rag
base_vector_store_test.py	test: enforce F811 so a duplicate definition cannot silently replace the first	2026-08-21 12:06:19 -07:00
conftest.py	test: enforce F811 so a duplicate definition cannot silently replace the first	2026-08-21 12:06:19 -07:00
test_azure_ai_vector_store.py
test_azure_vector_store.py
test_bedrock_vector_store.py
test_gemini_vector_store.py
test_milvus_vector_store.py
test_openai_vector_store.py
test_ragflow_vector_store.py	test(lint): ban blind pytest.raises(Exception) with ruff B017 (#37731)	2026-08-20 18:09:42 -07:00
test_s3_vectors_vector_store.py
test_vertex_ai_search_api_vector_store.py
test_vertex_ai_vector_store.py