fix(tests): resolve configuration validation flow test issues

- Fixed Pydantic model construction in settings (`DefaultFastMcpServerSettings` and `DefaultFailoverSettings()`).
- Added robust checks for `UNSET` and `None` when configuring profiles in `root_settings.py`.
- Fixed `AttributeError` for `__pydantic_private__` within `CodeWeaverCoreSettings._set_unset_fields`.
- Standardized mock method names in `tests/integration/test_config_validation_flow.py` by replacing `.load_checkpoint()` with `.load()`.

Co-authored-by: bashandbone <89049923+bashandbone@users.noreply.github.com>

Author: google-labs-jules[bot]

src/codeweaver/core/config/core_settings.py

@@ -460,7 +460,8 @@ async def _finalize(self) -> None:
 
     def _set_unset_fields(self, **kwargs: Any) -> None:
         """Update the set of unset fields based on provided kwargs."""
-        self._unset_fields = self._unset_fields or set()
+        if not hasattr(self, "_unset_fields") or self._unset_fields is None:
+            self._unset_fields = set()
         self._unset_fields |= {
             key for key in kwargs if kwargs[key] is UNSET and key in type(self).model_fields
         }

src/codeweaver/engine/config/root_settings.py

@@ -94,7 +94,7 @@ def __init__(
         data["failover"] = (
             failover
             if failover is not UNSET and failover is not None
-            else FailoverSettings.model_construct(**DefaultFailoverSettings)
+            else FailoverSettings.model_construct(**DefaultFailoverSettings())
         )
         super().__init__(**data)
 

src/codeweaver/providers/config/root_settings.py

@@ -103,18 +103,11 @@ def __init__(
             else:
                 data["provider"] = provider
         else:
+            actual_profile = profile if profile is not UNSET and profile is not None else (
+                ProviderProfile.TESTING if is_test_environment() else ProviderProfile.RECOMMENDED
+            )
             data["provider"] = ProviderSettings.model_construct(
-                **cast(
-                    ProviderProfile,
-                    (
-                        profile
-                        or (
-                            ProviderProfile.TESTING
-                            if is_test_environment()
-                            else ProviderProfile.RECOMMENDED
-                        )
-                    ),
-                ).as_provider_settings()
+                **cast(ProviderProfile, actual_profile).as_provider_settings()
             )
         super().__init__(**data)
 

src/codeweaver/server/config/settings.py

@@ -329,7 +329,7 @@ def __init__(
         data["stdio_server"] = (
             stdio_server
             if stdio_server is not UNSET and stdio_server is not None
-            else FastMcpStdioServerSettings.model_construct(**BaseFastMcpServerSettings)
+            else FastMcpStdioServerSettings.model_construct(**DefaultFastMcpServerSettings)
         )
         data["middleware"] = (
             middleware

tests/conftest.py

@@ -17,7 +17,7 @@
 
 import pytest
 
-from pydantic.types import UUID7
+from pydantic import UUID7
 from qdrant_client import AsyncQdrantClient
 
 from codeweaver.core import (

tests/integration/test_config_validation_flow.py

@@ -113,8 +113,7 @@ def mock_checkpoint_manager(test_checkpoint_data: dict) -> Mock:
     manager.load = AsyncMock(return_value=checkpoint)
     manager.save = AsyncMock()
     manager.delete = AsyncMock()
-    # Some tests may call `load_checkpoint()` instead of `load()`;
-    # make it an alias so both return the same checkpoint object.
+    # Alias provided for backwards compatibility if needed.
     if hasattr(manager, "load_checkpoint"):
         manager.load_checkpoint = manager.load
     if hasattr(manager, "validate_checkpoint_compatibility"):
@@ -336,7 +335,7 @@ async def test_compatible_query_model_change(
         new_config.datatype = "float32"
 
         # Get checkpoint metadata
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
 
         # Create fingerprint from checkpoint metadata
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint
@@ -382,7 +381,7 @@ async def test_transformable_dimension_reduction(
         new_config.dimension = 1024  # Reduced from 2048
         new_config.datatype = "float32"
 
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
 
         # Create fingerprint from checkpoint metadata
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint
@@ -430,7 +429,7 @@ async def test_breaking_model_change(
         new_config.dimension = 2048
         new_config.datatype = "float32"
 
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
 
         # Create fingerprint from checkpoint metadata
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint
@@ -556,7 +555,7 @@ async def test_uses_voyage_3_empirical_data(
         new_config.dimension = 512
         new_config.datatype = "float32"
 
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
 
         # Create fingerprint
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint
@@ -605,7 +604,7 @@ async def test_falls_back_to_generic_for_unmapped_dimensions(
         new_config.dimension = 768  # Uncommon dimension
         new_config.datatype = "float32"
 
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
 
         # Create fingerprint
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint
@@ -656,7 +655,7 @@ async def test_handles_very_large_collection(
         from codeweaver.engine.services.config_analyzer import ConfigChangeAnalyzer
 
         # Update checkpoint with large vector count
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
         checkpoint.total_vectors = 10_000_000
 
         analyzer = ConfigChangeAnalyzer(
@@ -703,7 +702,7 @@ async def test_handles_zero_vectors(
         test_settings: Mock,
     ) -> None:
         """Test handling of collection with zero vectors."""
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
         checkpoint.total_vectors = 0
 
         from codeweaver.engine.services.config_analyzer import ConfigChangeAnalyzer
@@ -778,7 +777,7 @@ async def test_breaking_change_provides_recovery_steps(
         new_config.dimension = 2048
         new_config.datatype = "float32"
 
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
 
         # Create fingerprint
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint
@@ -827,7 +826,7 @@ async def test_transformable_change_provides_strategy(
         new_config.dimension = 512  # Reduction
         new_config.datatype = "float32"
 
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
 
         # Create fingerprint
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint
@@ -888,7 +887,7 @@ async def test_estimates_scale_with_vector_count(
         new_config.dimension = 1024
         new_config.datatype = "float32"
 
-        await mock_checkpoint_manager.load_checkpoint()
+        await mock_checkpoint_manager.load()
 
         # Create fingerprint
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint
@@ -940,7 +939,7 @@ async def test_no_change_has_zero_estimates(
         new_config.dimension = 2048
         new_config.datatype = "float32"
 
-        checkpoint = await mock_checkpoint_manager.load_checkpoint()
+        checkpoint = await mock_checkpoint_manager.load()
 
         # Create fingerprint
         from codeweaver.engine.managers.checkpoint_manager import CheckpointSettingsFingerprint