Fix new set of bugs related to tolerance

Seddryck · Dec 30, 2018 · f1e37d2 · f1e37d2
1 parent c8797e8
commit f1e37d2
Show file tree

Hide file tree

Showing 4 changed files with 49 additions and 2 deletions.
diff --git a/NBi.Core/Scalar/Comparer/ToleranceFactory.cs b/NBi.Core/Scalar/Comparer/ToleranceFactory.cs
@@ -21,7 +21,8 @@ public static Tolerance Instantiate(IColumnDefinition columnDefinition)
 
         public static Tolerance Instantiate(ColumnType type, string value)
         {
-
+            if (string.IsNullOrEmpty(value) || string.IsNullOrWhiteSpace(value))
+                return None(type);
 
             Tolerance tolerance=null;
             switch (type)

diff --git a/NBi.Testing/NBi.Testing.csproj b/NBi.Testing/NBi.Testing.csproj
@@ -333,6 +333,7 @@
     <Compile Include="Unit\Core\ResultSet\Lookup\LookupExistsAnalyzerTest.cs" />
     <Compile Include="Unit\Core\Scalar\Casting\NumericCasterTest.cs" />
     <Compile Include="Unit\Core\Scalar\Comparer\CellComparerTest.cs" />
+    <Compile Include="Unit\Core\Scalar\Comparer\ToleranceFactoryTest.cs" />
     <Compile Include="Unit\Core\Scalar\Comparer\TextToleranceFactoryTest.cs" />
     <Compile Include="Unit\Core\Query\Resolver\ReportDataSetQueryResolverTest.cs" />
     <Compile Include="Unit\Core\Query\Resolver\SharedDataSetQueryResolverTest.cs" />

diff --git a/NBi.Testing/Unit/Core/Scalar/Comparer/ToleranceFactoryTest.cs b/NBi.Testing/Unit/Core/Scalar/Comparer/ToleranceFactoryTest.cs
@@ -0,0 +1,45 @@
+using NBi.Core.ResultSet;
+using NBi.Core.Scalar.Comparer;
+using NBi.Xml.Items.ResultSet;
+using NUnit.Framework;
+using System;
+using System.Collections.Generic;
+using System.Globalization;
+using System.Linq;
+using System.Text;
+using System.Threading;
+using System.Threading.Tasks;
+
+namespace NBi.Testing.Unit.Core.Scalar.Comparer
+{
+    public class ToleranceFactoryTest
+    {
+        [Test]
+        [TestCase(ColumnRole.Key)]
+        [TestCase(ColumnRole.Value)]
+        [TestCase(ColumnRole.Ignore)]
+        public void Instantiate_NoToleranceDefined_InstantiatedToNullOrNone(ColumnRole columnRole)
+        {
+            var colDef = new ColumnDefinitionXml()
+            {
+                Index = 0,
+                Role = columnRole,
+                Tolerance = string.Empty
+            };
+            var tolerance = ToleranceFactory.Instantiate(colDef);
+
+            Assert.That(Tolerance.IsNullOrNone(tolerance), Is.True);
+        }
+
+        [Test]
+        [TestCase(ColumnType.Text)]
+        [TestCase(ColumnType.Numeric)]
+        [TestCase(ColumnType.DateTime)]
+        [TestCase(ColumnType.Boolean)]
+        public void Instantiate_NoToleranceDefined_InstantiatedToNullOrNone(ColumnType columnType)
+        {
+            var tolerance = ToleranceFactory.Instantiate(columnType, string.Empty);
+            Assert.That(Tolerance.IsNullOrNone(tolerance), Is.True);
+        }
+    }
+}
diff --git a/NBi.Xml/Items/ResultSet/ColumnDefinitionXml.cs b/NBi.Xml/Items/ResultSet/ColumnDefinitionXml.cs
@@ -41,7 +41,7 @@ public class ColumnDefinitionXml: IColumnDefinition
         [XmlIgnore()]
         public bool IsToleranceSpecified
         {
-            get => string.IsNullOrEmpty(Tolerance);
+            get => !string.IsNullOrEmpty(Tolerance);
         }
 
         [XmlIgnore()]