test: address code review feedback - add validate_auth_config callable check and improve docstrings
Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
This commit is contained in:
@@ -38,6 +38,7 @@ class TestConfigValidatorReExports:
|
|||||||
assert callable(cv.validate_email_config)
|
assert callable(cv.validate_email_config)
|
||||||
assert callable(cv.validate_storage_configs)
|
assert callable(cv.validate_storage_configs)
|
||||||
assert callable(cv.validate_notification_config)
|
assert callable(cv.validate_notification_config)
|
||||||
|
assert callable(cv.validate_auth_config)
|
||||||
assert callable(cv.check_all_configs)
|
assert callable(cv.check_all_configs)
|
||||||
|
|
||||||
@pytest.mark.unit
|
@pytest.mark.unit
|
||||||
@@ -324,10 +325,10 @@ class TestSettingsDelete:
|
|||||||
|
|
||||||
|
|
||||||
class TestSettingsBulkUpdate:
|
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
|
The /bulk-update route is defined after /{key} in the router, so FastAPI
|
||||||
/{key} first. We test the handler function directly via asyncio.
|
matches /{key} first. We test the async handler function directly.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
def _make_mock_db(self):
|
def _make_mock_db(self):
|
||||||
|
|||||||
Reference in New Issue
Block a user