style: fix formatting and linting issues in conftest.py

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>

tests/conftest.py

@@ -1,10 +1,12 @@
 """
 Pytest configuration and shared fixtures for DocuElevate tests.
 """
+
 import os
 import tempfile
-import pytest
 from typing import Generator
+
+import pytest
 from fastapi.testclient import TestClient
 from sqlalchemy import create_engine
 from sqlalchemy.orm import sessionmaker
@@ -22,10 +24,11 @@ os.environ["WORKDIR"] = "/tmp"
 os.environ["AUTH_ENABLED"] = "False"
 os.environ["SESSION_SECRET"] = "test_secret_key_for_testing_must_be_at_least_32_characters_long"
 
-from app.database import Base, get_db
+from app.database import Base  # noqa: E402
-from app.main import app as fastapi_app
+from app.main import app as fastapi_app  # noqa: E402
+
 # Import models to register them with SQLAlchemy Base
-from app.models import DocumentMetadata, FileRecord, ProcessingLog
+from app.models import DocumentMetadata, FileRecord, ProcessingLog  # noqa: F401, E402
 
 
 @pytest.fixture(scope="session")
@@ -44,14 +47,14 @@ def db_session():
         connect_args={"check_same_thread": False},
         poolclass=StaticPool,
     )
-    
+
     # Create all tables
     Base.metadata.create_all(bind=engine)
-    
+
     # Create a session
     TestingSessionLocal = sessionmaker(autocommit=False, autoflush=False, bind=engine)
     session = TestingSessionLocal()
-    
+
     try:
         yield session
     finally:
@@ -62,24 +65,24 @@ def db_session():
 @pytest.fixture(scope="function")
 def client(db_session) -> TestClient:
     """Create a test client with a fresh database."""
-    
+
     # Import the canonical get_db function
     from app.database import get_db
-    
+
     # Override the get_db dependency to use our test database
     def override_get_db():
         try:
             yield db_session
         finally:
             pass
-    
+
     # Override the single canonical get_db dependency
     fastapi_app.dependency_overrides[get_db] = override_get_db
-    
+
     # Use base_url to satisfy TrustedHostMiddleware
     with TestClient(fastapi_app, base_url="http://localhost") as test_client:
         yield test_client
-    
+
     # Clean up
     fastapi_app.dependency_overrides.clear()
 
@@ -88,7 +91,7 @@ def client(db_session) -> TestClient:
 def sample_pdf_path(test_workdir) -> str:
     """Create a sample PDF file for testing."""
     pdf_path = os.path.join(test_workdir, "test.pdf")
-    
+
     # Create a minimal valid PDF
     pdf_content = b"""%PDF-1.4
 1 0 obj
@@ -126,10 +129,10 @@ startxref
 197
 %%EOF
 """
-    
+
-    with open(pdf_path, 'wb') as f:
+    with open(pdf_path, "wb") as f:
         f.write(pdf_content)
-    
+
     return pdf_path
 
 
@@ -137,10 +140,10 @@ startxref
 def sample_text_file(test_workdir) -> str:
     """Create a sample text file for testing."""
     text_path = os.path.join(test_workdir, "test.txt")
-    
+
-    with open(text_path, 'w') as f:
+    with open(text_path, "w") as f:
         f.write("This is a test document.\nWith multiple lines.\n")
-    
+
     return text_path
 
 
@@ -148,46 +151,29 @@ def sample_text_file(test_workdir) -> str:
 def mock_openai_response():
     """Mock OpenAI API response for testing."""
     return {
-        "choices": [{
+        "choices": [
-            "message": {
+            {
-                "content": '{"document_type": "invoice", "summary": "Test invoice", "tags": ["test", "invoice"]}'
+                "message": {
+                    "content": '{"document_type": "invoice", "summary": "Test invoice", "tags": ["test", "invoice"]}'
+                }
             }
-        }]
+        ]
     }
 
 
 @pytest.fixture
 def mock_azure_response():
     """Mock Azure Document Intelligence API response for testing."""
-    return {
+    return {"analyzeResult": {"content": "Test document content extracted by OCR", "pages": [{"pageNumber": 1}]}}
-        "analyzeResult": {
-            "content": "Test document content extracted by OCR",
-            "pages": [{"pageNumber": 1}]
-        }
-    }
 
 
 # Markers for categorizing tests
 def pytest_configure(config):
     """Configure custom pytest markers."""
-    config.addinivalue_line(
+    config.addinivalue_line("markers", "unit: Unit tests for individual functions/methods")
-        "markers", "unit: Unit tests for individual functions/methods"
+    config.addinivalue_line("markers", "integration: Integration tests for API endpoints and workflows")
-    )
+    config.addinivalue_line("markers", "slow: Tests that take significant time to run")
-    config.addinivalue_line(
+    config.addinivalue_line("markers", "security: Security-related tests")
-        "markers", "integration: Integration tests for API endpoints and workflows"
+    config.addinivalue_line("markers", "requires_external: Tests requiring external services")
-    )
+    config.addinivalue_line("markers", "requires_db: Tests requiring database")
-    config.addinivalue_line(
+    config.addinivalue_line("markers", "requires_redis: Tests requiring Redis")
-        "markers", "slow: Tests that take significant time to run"
-    )
-    config.addinivalue_line(
-        "markers", "security: Security-related tests"
-    )
-    config.addinivalue_line(
-        "markers", "requires_external: Tests requiring external services"
-    )
-    config.addinivalue_line(
-        "markers", "requires_db: Tests requiring database"
-    )
-    config.addinivalue_line(
-        "markers", "requires_redis: Tests requiring Redis"
-    )