string tests

stephanstapel · Oct 8, 2024 · e31655a · e31655a
1 parent 20d363b
commit e31655a
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 8 deletions.
diff --git a/ZUGFeRD-Test/TestBase.cs b/ZUGFeRD-Test/TestBase.cs
@@ -28,7 +28,7 @@ public class TestBase
     {
         protected string _makeSurePathIsCrossPlatformCompatible(string path)
         {
-            if (string.IsNullOrEmpty(path))
+            if (string.IsNullOrWhiteSpace(path))
             {
                 return path;
             }

diff --git a/ZUGFeRD-Test/ZUGFeRD20Tests.cs b/ZUGFeRD-Test/ZUGFeRD20Tests.cs
@@ -700,7 +700,7 @@ public void TestWriteAndReadExtended()
             //Line items
             var loadedLineItem = loadedInvoice.TradeLineItems.FirstOrDefault(i => i.SellerAssignedID == "TB100A4");
             Assert.IsNotNull(loadedLineItem);
-            Assert.IsTrue(!string.IsNullOrEmpty(loadedLineItem.AssociatedDocument.LineID));
+            Assert.IsTrue(!string.IsNullOrWhiteSpace(loadedLineItem.AssociatedDocument.LineID));
             Assert.AreEqual("This is line item TB100A4", loadedLineItem.Description);
 
             Assert.AreEqual("Trennblätter A4", loadedLineItem.Name);

diff --git a/ZUGFeRD/InvoiceDescriptor22UBLWriter.cs b/ZUGFeRD/InvoiceDescriptor22UBLWriter.cs
@@ -199,7 +199,7 @@ public override void Save(InvoiceDescriptor descriptor, Stream stream, ZUGFeRDFo
                     Writer.WriteEndElement();
                 }
 
-                if (!string.IsNullOrEmpty(tradeAllowanceCharge.Reason))
+                if (!string.IsNullOrWhiteSpace(tradeAllowanceCharge.Reason))
                 {
                     Writer.WriteStartElement("cbc:AllowanceChargeReason"); // BT-97 / BT-104
                     Writer.WriteValue(tradeAllowanceCharge.Reason);

diff --git a/ZUGFeRD/InvoiceDescriptor22UblReader.cs b/ZUGFeRD/InvoiceDescriptor22UblReader.cs
@@ -673,27 +673,27 @@ private static Party _nodeAsParty(XmlNode baseNode, string xpath, XmlNamespaceMa
                 retval.ContactName = null;
             }
 
-            if (string.IsNullOrEmpty(retval.AddressLine3))
+            if (string.IsNullOrWhiteSpace(retval.AddressLine3))
             {
                 retval.AddressLine3 = String.Empty;
             }
 
-            if (string.IsNullOrEmpty(retval.CountrySubdivisionName))
+            if (string.IsNullOrWhiteSpace(retval.CountrySubdivisionName))
             {
                 retval.CountrySubdivisionName = String.Empty;
             }
 
-            if (string.IsNullOrEmpty(retval.City))
+            if (string.IsNullOrWhiteSpace(retval.City))
             {
                 retval.City = String.Empty;
             }
 
-            if (string.IsNullOrEmpty(retval.Postcode))
+            if (string.IsNullOrWhiteSpace(retval.Postcode))
             {
                 retval.Postcode = String.Empty;
             }
 
-            if (string.IsNullOrEmpty(retval.Street))
+            if (string.IsNullOrWhiteSpace(retval.Street))
             {
                 retval.Street = String.Empty;
             }