From 33e68452fd75cf11df756a2041092e246b83ecf9 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Fri, 13 Feb 2026 21:50:58 +0000
Subject: [PATCH] test: address code review feedback - add validate_auth_config
 callable check and improve docstrings

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
---
 tests/test_coverage_config_settings.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/tests/test_coverage_config_settings.py b/tests/test_coverage_config_settings.py
index ebe1cf83..da80cedb 100644
--- a/tests/test_coverage_config_settings.py
+++ b/tests/test_coverage_config_settings.py
@@ -38,6 +38,7 @@ class TestConfigValidatorReExports:
         assert callable(cv.validate_email_config)
         assert callable(cv.validate_storage_configs)
         assert callable(cv.validate_notification_config)
+        assert callable(cv.validate_auth_config)
         assert callable(cv.check_all_configs)
 
     @pytest.mark.unit
@@ -324,10 +325,10 @@ class TestSettingsDelete:
 
 
 class TestSettingsBulkUpdate:
-    """POST /api/settings/bulk-update - bulk update settings.
+    """Tests for bulk_update_settings handler.
 
-    Note: The /bulk-update route is defined after /{key}, so FastAPI matches
-    /{key} first.  We test the handler function directly via asyncio.
+    The /bulk-update route is defined after /{key} in the router, so FastAPI
+    matches /{key} first. We test the async handler function directly.
     """
 
     def _make_mock_db(self):