Merge pull request #20 from synercoder/develop

Fixed value division by double

Author: synercoder

src/Synercoding.Primitives/PackageDetails.props

@@ -10,11 +10,7 @@
     <Product>Synercoding.Primitives</Product>
     <Title>Synercoding.Primitives</Title>
     <Description>Primitives with units attached (think mm, cm, in, px) that can be used for 2D graphics.</Description>
-    <PackageReleaseNotes>
-- Added support for net452 &amp; net5.0.
-- Added a ValueCreator class, supposed to be used as a static using.
-- Added AsRaw extensions for Point &amp; Size
-    </PackageReleaseNotes>
+    <PackageReleaseNotes> - Fixed value division by double</PackageReleaseNotes>
   </PropertyGroup>
 
 </Project>

src/Synercoding.Primitives/Value.cs

@@ -241,7 +241,7 @@ public bool Equals(Value other)
         /// <param name="a">The <see cref="Value"/> to divide</param>
         /// <param name="b">The <see cref="double"/> to divisor</param>
         /// <returns>The result of the division operation</returns>
-        public static double operator /(Value a, double b)
-            => a.Raw / b;
+        public static Value operator /(Value a, double b)
+            => new Value(a.Raw / b, a.Unit);
     }
 }

tests/Synercoding.Primitives.Tests/ValueTests.cs

@@ -120,6 +120,21 @@ public void DivisionOperator_1cmAnd1mm_containes10times()
             Assert.Equal(expected, result);
         }
 
+        [Fact]
+        public void DivisionOperator_1cmAnd2_Is0dot5cm()
+        {
+            // Arrange
+            var cm = new Value(1, Unit.Centimeters);
+            var amount = 2;
+            var expected = new Value(0.5, Unit.Centimeters);
+
+            // Act
+            var result = cm / amount;
+
+            // Assert
+            Assert.Equal(expected, result);
+        }
+
         [Fact]
         public void FromDesignation_AllEnumValuesExceptPx_DoesNotThrowNotImplementedException()
         {