Have DESIGN_MATRIX not require design_sheet and default_sheet

This commit makes the DESIGN_MATRIX keyword not require arguments for design and default sheet. Now DESIGN_SHEET defaults to `DesignSheet01` and `DEFAULT_SHEET` defaults to "DefaultSheet"

Author: jonathan-eq

src/ert/config/design_matrix.py

@@ -44,8 +44,8 @@ def __post_init__(self) -> None:
     def from_config_list(cls, config_list: list[str]) -> DesignMatrix:
         filename = Path(config_list[0])
         options = option_dict(config_list, 1)
-        design_sheet = options.get("DESIGN_SHEET")
-        default_sheet = options.get("DEFAULT_SHEET")
+        design_sheet = options.get("DESIGN_SHEET", "DesignSheet01")
+        default_sheet = options.get("DEFAULT_SHEET", "DefaultSheet")
         errors = []
         if filename.suffix not in {
             ".xlsx",
@@ -56,14 +56,6 @@ def from_config_list(cls, config_list: list[str]) -> DesignMatrix:
                     f"DESIGN_MATRIX must be of format .xls or .xlsx; is '{filename}'"
                 ).set_context(config_list)
             )
-        if design_sheet is None:
-            errors.append(
-                ErrorInfo("Missing required DESIGN_SHEET").set_context(config_list)
-            )
-        if default_sheet is None:
-            errors.append(
-                ErrorInfo("Missing required DEFAULT_SHEET").set_context(config_list)
-            )
         if design_sheet is not None and design_sheet == default_sheet:
             errors.append(
                 ErrorInfo(

src/ert/config/parsing/config_schema.py

@@ -257,7 +257,7 @@ def install_job_directory_keyword() -> SchemaItem:
 def design_matrix_keyword() -> SchemaItem:
     return SchemaItem(
         kw=ConfigKeys.DESIGN_MATRIX,
-        argc_min=3,
+        argc_min=1,
         argc_max=3,
         type_map=[
             SchemaItemType.EXISTING_PATH,

tests/ert/unit_tests/config/test_analysis_config.py

@@ -103,53 +103,21 @@ def test_invalid_min_realization_raises_config_validation_error():
         )
 
 
-def test_invalid_design_matrix_format_raises_validation_error():
-    with pytest.raises(
-        ConfigValidationError,
-        match="DESIGN_MATRIX must be of format \\.xls or \\.xlsx; is 'my_matrix\\.txt'",
-    ):
-        AnalysisConfig.from_dict(
-            {
-                ConfigKeys.NUM_REALIZATIONS: 1,
-                ConfigKeys.DESIGN_MATRIX: [
-                    [
-                        "my_matrix.txt",
-                        "DESIGN_SHEET:sheet1",
-                        "DEFAULT_SHEET:sheet2",
-                    ],
-                ],
-            }
-        )
-
-
-def test_design_matrix_without_design_sheet_raises_validation_error():
-    with pytest.raises(ConfigValidationError, match="Missing required DESIGN_SHEET"):
-        AnalysisConfig.from_dict(
-            {
-                ConfigKeys.DESIGN_MATRIX: [
-                    [
-                        "my_matrix.xlsx",
-                        "DESIGN_:design",
-                        "DEFAULT_SHEET:default",
-                    ]
-                ],
-            }
-        )
-
-
-def test_design_matrix_without_default_sheet_raises_validation_error():
-    with pytest.raises(ConfigValidationError, match="Missing required DEFAULT_SHEET"):
-        AnalysisConfig.from_dict(
-            {
-                ConfigKeys.DESIGN_MATRIX: [
-                    [
-                        "my_matrix.xlsx",
-                        "DESIGN_SHEET:design",
-                        "DEFAULT_:default",
-                    ]
-                ],
-            }
-        )
+def test_design_matrix_default_arguments():
+    config = AnalysisConfig.from_dict(
+        {
+            ConfigKeys.DESIGN_MATRIX: [
+                [
+                    "my_matrix.xlsx",
+                    "DESIGN_SHEET:design",
+                    "DEFAULT_:default",
+                ]
+            ],
+        }
+    )
+    assert config.design_matrix
+    assert config.design_matrix.design_sheet == "DesignSheet01"
+    assert config.design_matrix.default_sheet == "DefaultSheet"
 
 
 def test_invalid_min_realization_percentage_raises_config_validation_error():