Merge pull request #287 from christianlouis/copilot/fix-tests-module-level-patching

Fix test failures: expose helper functions at module level for patching

.github/workflows/ci.yml

@@ -99,14 +99,17 @@ jobs:
         uses: codecov/codecov-action@v5
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
-          file: ./coverage.xml
+          files: ./coverage.xml
           fail_ci_if_error: false
 
       - name: Upload test results to Codecov
         if: ${{ !cancelled() }}
-        uses: codecov/test-results-action@v1
+        uses: codecov/codecov-action@v5
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
+          files: ./junit.xml
+          report_type: test_results
+          fail_ci_if_error: false
 
       - name: Upload test artifacts
         if: ${{ !cancelled() }}

app/views/settings.py

@@ -12,7 +12,11 @@ from fastapi.responses import RedirectResponse
 from sqlalchemy.orm import Session
 
 from app.utils.config_validator.masking import mask_sensitive_value
-from app.utils.settings_service import get_setting_metadata, get_settings_by_category
+from app.utils.settings_service import (
+    get_all_settings_from_db,
+    get_setting_metadata,
+    get_settings_by_category,
+)
 from app.views.base import APIRouter, get_db, require_login, settings, templates
 
 logger = logging.getLogger(__name__)
@@ -57,8 +61,6 @@ async def settings_page(request: Request, db: Session = Depends(get_db)):
 
     try:
         # Get settings from database
-        from app.utils.settings_service import get_all_settings_from_db
-
         db_settings = get_all_settings_from_db(db)
 
         # Get settings organized by category

app/views/status.py

@@ -8,6 +8,7 @@ from datetime import datetime
 
 from fastapi import Request
 
+from app.utils.config_validator import get_provider_status, get_settings_for_display
 from app.views.base import APIRouter, require_login, settings, templates
 
 logger = logging.getLogger(__name__)
@@ -20,8 +21,6 @@ async def status_dashboard(request: Request):
     """
     Status dashboard showing all configured integration targets
     """
-    from app.utils.config_validator import get_provider_status
-
     # Get provider status
     providers = get_provider_status()
 
@@ -100,8 +99,6 @@ async def env_debug(request: Request):
     debug_enabled = settings.debug
 
     # Get settings data
-    from app.utils.config_validator import get_settings_for_display
-
     settings_data = get_settings_for_display(show_values=debug_enabled)
 
     return templates.TemplateResponse(