Make ZeitabhaengigeBeziehung.GueltigBis Optional (#610)

* Make Enddate Optional * Extend Test * add * , * Add test * cscharping * Update BO4ETestProject/TestLinkExtensions.cs * Update BO4E/Marktkommunikation/ZeitabhaengigeBeziehung.cs --------- Co-authored-by: konstantin <[email protected]>
Hochfrequenz · Nov 25, 2024 · dc3ff3c · dc3ff3c
1 parent 2910114
commit dc3ff3c
Show file tree

Hide file tree

Showing 2 changed files with 59 additions and 2 deletions.
diff --git a/BO4E/Marktkommunikation/ZeitabhaengigeBeziehung.cs b/BO4E/Marktkommunikation/ZeitabhaengigeBeziehung.cs
@@ -21,12 +21,18 @@ public class ZeitabhaengigeBeziehung
     public DateTimeOffset GueltigVon { get; set; }
 
     /// <summary>
-    /// exklusives Ende
+    /// Offene Zeitscheiben werden mit <code>GueltigBis = null</code> modelliert, _nicht_ mit <code>DateTimeOffset.MaxValue</code>.
+    /// The maximum end date is NULL, this means DateTimeOffset.MaxValue is converted to null by the setter.
     /// </summary>
     [JsonPropertyName("gueltigBis")]
     [JsonPropertyOrder(2)]
     [Newtonsoft.Json.JsonProperty(PropertyName = "gueltigBis", Order = 2)]
-    public DateTimeOffset GueltigBis { get; set; }
+    public DateTimeOffset? GueltigBis
+    {
+        get => _gueltigBis;
+        set => _gueltigBis = (value == DateTimeOffset.MaxValue) ? null : value;
+    }
+    private DateTimeOffset? _gueltigBis;
 
     /// <summary>
     /// e.g.

diff --git a/BO4ETestProject/TestLinkExtensions.cs b/BO4ETestProject/TestLinkExtensions.cs
@@ -124,4 +124,55 @@ public void Test_Zeitabhaengig_To_Links()
         actual.Should().ContainKey("x").WhoseValue.Should().HaveCount(2);
         actual.Should().ContainKey("foo").WhoseValue.Should().HaveCount(2);
     }
+
+    [TestMethod]
+    public void Test_Zeitabhaengig_To_Links_with_Null_Until()
+    {
+        var bc = new BOneyComb
+        {
+            ZeitabhaengigeLinks = new()
+            {
+                new ZeitabhaengigeBeziehung
+                {
+                    ParentId = "foo",
+                    ChildId = "bar",
+                    GueltigVon = DateTimeOffset.MinValue,
+                },
+                new ZeitabhaengigeBeziehung
+                {
+                    ParentId = "foo",
+                    ChildId = "baz",
+                    GueltigVon = DateTimeOffset.MinValue,
+                },
+                new ZeitabhaengigeBeziehung
+                {
+                    ParentId = "x",
+                    ChildId = "y",
+                    GueltigVon = DateTimeOffset.MinValue,
+                },
+                new ZeitabhaengigeBeziehung
+                {
+                    ParentId = "x",
+                    ChildId = "z",
+                    GueltigVon = DateTimeOffset.MinValue,
+                },
+            },
+        };
+        var actual = bc.ZeitabhaengigeLinks.ToTimeIndependentLinks();
+        actual.Should().ContainKey("x").WhoseValue.Should().HaveCount(2);
+        actual.Should().ContainKey("foo").WhoseValue.Should().HaveCount(2);
+    }
+
+    [TestMethod]
+    public void Test_inifinteTime_is_null()
+    {
+        ZeitabhaengigeBeziehung timeMax = new ZeitabhaengigeBeziehung()
+        {
+            ParentId = "x",
+            ChildId = "y",
+            GueltigVon = DateTimeOffset.MinValue,
+            GueltigBis = DateTimeOffset.MaxValue,
+        };
+        timeMax.GueltigBis.Should().BeNull(because: "the setter converts MaxValue to null");
+    }
 }