[MAINTENANCE] Ensure that DataContextConfig has a consistent shape when args are omitted

great_expectations/data_context/types/base.py

@@ -2308,7 +2308,7 @@ class DataContextConfig(BaseYamlConfig):
             globally, at the ExpectationSuite or ExpectationValidationResults-level.
     """
 
-    def __init__(  # noqa: C901, PLR0912, PLR0913, PLR0915
+    def __init__(  # noqa: C901, PLR0912, PLR0913
         self,
         batch_configs: Optional[Dict] = None,
         config_version: Optional[float] = None,
@@ -2327,7 +2327,6 @@ def __init__(  # noqa: C901, PLR0912, PLR0913, PLR0915
         plugins_directory: Optional[str] = None,
         validation_operators=None,
         stores: Optional[Dict] = None,
-        notebooks: Optional[Any] = None,
         data_docs_sites: Optional[Dict] = None,
         config_variables_file_path: Optional[str] = None,
         anonymous_usage_statistics: Optional[AnonymizedUsageStatisticsConfig] = None,
@@ -2337,9 +2336,6 @@ def __init__(  # noqa: C901, PLR0912, PLR0913, PLR0915
         progress_bars: Optional[ProgressBarsConfig] = None,
         include_rendered_content: Optional[IncludeRenderedContentConfig] = None,
     ) -> None:
-        if notebooks:
-            warnings.warn("The `notebooks` parameter no longer supported.", UserWarning)
-
         # Set defaults
         if config_version is None:
             config_version = DataContextConfigDefaults.DEFAULT_CONFIG_VERSION.value
@@ -2373,13 +2369,10 @@ def __init__(  # noqa: C901, PLR0912, PLR0913, PLR0915
         self.expectations_store_name = expectations_store_name
         self.validations_store_name = validations_store_name
         self.evaluation_parameter_store_name = evaluation_parameter_store_name
-        if checkpoint_store_name is not None:
-            self.checkpoint_store_name = checkpoint_store_name
-        if profiler_store_name is not None:
-            self.profiler_store_name = profiler_store_name
+        self.checkpoint_store_name = checkpoint_store_name
+        self.profiler_store_name = profiler_store_name
         self.plugins_directory = plugins_directory
-        if validation_operators is not None:
-            self.validation_operators = validation_operators
+        self.validation_operators = validation_operators
         self.stores = stores or {}
         self.data_docs_sites = data_docs_sites
         self.config_variables_file_path = config_variables_file_path

tests/data_context/store/test_store_backends.py

@@ -1359,8 +1359,10 @@ def test_InlineStoreBackend(empty_data_context) -> None:
         ("fluent_datasources",),
         ("include_rendered_content",),
         ("plugins_directory",),
+        ("profiler_store_name",),
         ("progress_bars",),
         ("stores",),
+        ("validation_operators",),
         ("validations_store_name",),
     ]
 

tests/data_context/test_data_context_config_ui.py

@@ -1,6 +1,7 @@
 import copy
 import os
 from typing import Dict, Final, Optional
+from unittest import mock
 
 import pytest
 
@@ -1560,3 +1561,38 @@ def test_DataContextConfig_with_InMemoryStoreBackendDefaults(
         ),
         DataContextConfig,
     )
+
+
+@pytest.mark.unit
+def test_data_context_config_defaults():
+    config = DataContextConfig()
+    assert config.to_json_dict() == {
+        "anonymous_usage_statistics": {
+            "data_context_id": mock.ANY,
+            "enabled": True,
+            "explicit_id": False,
+            "explicit_url": False,
+            "usage_statistics_url": "https://stats.greatexpectations.io/great_expectations/v1/usage_statistics",
+        },
+        "batch_configs": {},
+        "checkpoint_store_name": None,
+        "concurrency": None,
+        "config_variables_file_path": None,
+        "config_version": 3,
+        "data_docs_sites": None,
+        "datasources": {},
+        "evaluation_parameter_store_name": None,
+        "expectations_store_name": None,
+        "fluent_datasources": {},
+        "include_rendered_content": {
+            "expectation_suite": False,
+            "expectation_validation_result": False,
+            "globally": False,
+        },
+        "plugins_directory": None,
+        "profiler_store_name": None,
+        "progress_bars": None,
+        "stores": {},
+        "validation_operators": None,
+        "validations_store_name": None,
+    }

tests/data_context/test_data_context_variables.py

@@ -466,6 +466,9 @@ def test_data_context_variables_save_config(
         ):
             expected_config_dict[attr] = data_context_config_dict[attr]
 
+        # Add null values for stores that are not configured
+        expected_config_dict["profiler_store_name"] = None
+
         assert mock_put.call_count == 1
         mock_put.assert_called_with(
             mock.ANY,  # requests.Session object