T3D header not correct

hydrolib/core/io/ini/util.py

@@ -462,3 +462,28 @@ def is_valid_coordinates_with_num_coordinates_specification() -> bool:
         raise ValueError(error)
 
     return root_validator(allow_reuse=True)(validate_location_specification)
+
+
+def get_key_renaming_root_validator(keys_to_rename: Dict[str, List[str]]):
+    """
+    Gets a root validator that renames the provided keys to support backwards compatibility.
+
+    Args:
+            keys_to_rename Dict[str, List[str]]: Dictionary of keys and a list of old keys that
+            should be converted to the current key.
+    """
+
+    def rename_keys(cls, values: Dict) -> Dict:
+        for current_keyword, old_keywords in keys_to_rename.items():
+            if current_keyword in values:
+                continue
+
+            for old_keyword in old_keywords:
+                if (value := values.get(old_keyword)) is not None:
+                    values[current_keyword] = value
+                    del values[old_keyword]
+                    break
+
+        return values
+
+    return root_validator(allow_reuse=True, pre=True)(rename_keys)

hydrolib/core/io/mdu/models.py

@@ -30,6 +30,7 @@ class AutoStartOption(IntEnum):
     Enum class containing the valid values for the AutoStart
     attribute in the [General][hydrolib.core.io.mdu.models.General] class.
     """
+
     no = 0
     autostart = 1
     autostartstop = 2

tests/data/reference/bc/t3d_backwards_compatibility.bc

@@ -0,0 +1,43 @@
+# written by HYDROLIB-core 0.3.0
+
+[General]
+fileVersion = 1.01
+fileType    = boundConds
+
+[Forcing]
+name               = boundary_timeseries
+function           = timeseries
+Time Interpolation = blockTo
+offset             = 1.23
+factor             = 2.34
+quantity           = time
+unit               = minutes since 2015-01-01 00:00:00
+quantity           = dischargebnd
+unit               = m³/s
+0.0      1.23
+60.0     2.34
+120.0    3.45
+
+[Forcing]
+name                            = boundary_t3d
+function                        = t3d
+offset                          = 1.23
+factor                          = 2.34
+Vertical Position Specification = 3.45 4.56 5.67
+Vertical Interpolation          = log
+Vertical Position Type          = percBed
+Time Interpolation              = linear
+quantity                        = time
+unit                            = m
+quantity                        = salinitybnd
+unit                            = ppt
+Vertical Position               = 1
+quantity                        = salinitybnd
+unit                            = ppt
+Vertical Position               = 2
+quantity                        = salinitybnd
+unit                            = ppt
+Vertical Position               = 3
+0.0      1.0    2.0    3.0
+60.0     4.0    5.0    6.0
+120.0    7.0    8.0    9.0

tests/data/reference/bc/test.bc

@@ -44,21 +44,25 @@ unit     = deg
 60.0    3.45    4.56
 
 [Forcing]
-name                  = boundary_t3d
-function              = t3d
-offset                = 1.23
-factor                = 2.34
-verticalPositions     = 3.45 4.56 5.67
-verticalInterpolation = log
-verticalPositionType  = percBed
-quantity              = time
-unit                  = m
-quantity              = salinitybnd
-unit                  = ppt
-quantity              = salinitybnd
-unit                  = ppt
-quantity              = salinitybnd
-unit                  = ppt
+name              = boundary_t3d
+function          = t3d
+offset            = 1.23
+factor            = 2.34
+vertPositions     = 3.45 4.56 5.67
+vertInterpolation = log
+vertPositionType  = percBed
+timeInterpolation = linear
+quantity          = time
+unit              = m
+quantity          = salinitybnd
+unit              = ppt
+vertPositionIndex = 1
+quantity          = salinitybnd
+unit              = ppt
+vertPositionIndex = 2
+quantity          = salinitybnd
+unit              = ppt
+vertPositionIndex = 3
 0.0      1.0    2.0    3.0
 60.0     4.0    5.0    6.0
 120.0    7.0    8.0    9.0

tests/io/ini/test_util.py

@@ -1,12 +1,14 @@
 from typing import Dict, List, Optional
 
 import pytest
+from pydantic import Extra
 from pydantic.error_wrappers import ValidationError
 
 from hydrolib.core.basemodel import BaseModel
 from hydrolib.core.io.ini.util import (
     LocationValidationConfiguration,
     LocationValidationFieldNames,
+    get_key_renaming_root_validator,
     get_location_specification_rootvalidator,
 )
 
@@ -198,3 +200,42 @@ def test_correct_1d_fields_locationtype_is_added(
             values
         )
         assert validated_values == expected_values
+
+
+class TestGetKeyRenamingRootValidator:
+    class DummyModel(BaseModel):
+        """Dummy model to test the validation of the location specification."""
+
+        randomproperty: str
+
+        validator = get_key_renaming_root_validator(
+            {
+                "randomproperty": [
+                    "randomProperty",
+                    "random_property",
+                    "oldRandomProperty",
+                ],
+            }
+        )
+
+        class Config:
+            extra = Extra.allow
+
+    @pytest.mark.parametrize(
+        "old_key", ["randomProperty", "random_property", "oldRandomProperty"]
+    )
+    def test_old_keys_are_correctly_renamed_to_current_keyword(self, old_key: str):
+        values = {old_key: "randomString"}
+
+        model = TestGetKeyRenamingRootValidator.DummyModel(**values)
+
+        assert model.randomproperty == "randomString"
+
+    def test_unknown_key_still_raises_error(self):
+        values = {"randomKeyThatNeverExisted": "randomString"}
+
+        with pytest.raises(ValidationError) as error:
+            TestGetKeyRenamingRootValidator.DummyModel(**values)
+
+        expected_message = "field required"
+        assert expected_message in str(error.value)