Improve Unit Tests #6

- some unit tests improvements
synergy-software · Sep 2, 2016 · 73abcf9 · 73abcf9
1 parent 509f743
commit 73abcf9
Show file tree

Hide file tree

Showing 2 changed files with 69 additions and 14 deletions.
diff --git a/Synergy.Contracts.Test/Failures/FailCastableTest.cs b/Synergy.Contracts.Test/Failures/FailCastableTest.cs
@@ -25,12 +25,24 @@ public void AsOrFail()
         }
 
         [Test]
-        [TestCase("text")]
-        [TestCase(null)]
-        public void AsOrFailSuccess(string toCast)
+        public void AsOrFailSuccess()
         {
+            // ARRANGE 
+            object somethingCastable = "text";
+
+            // ACT
+            somethingCastable.AsOrFail<string>();
+        }
+
+        [Test]
+        public void AsOrFailSuccessWithNull()
+        {
+            // ARRANGE 
+            object somethingCastable = null;
+
             // ACT
-            toCast.AsOrFail<string>();
+            // ReSharper disable once ExpressionIsAlwaysNull
+            somethingCastable.AsOrFail<string>();
         }
 
         #endregion
@@ -40,15 +52,42 @@ public void AsOrFailSuccess(string toCast)
         [Test]
         public void CastOrFail()
         {
-            Assert.Throws<DesignByContractViolationException>(
-                () => new object().CastOrFail<string>()
+            // ARRANGE
+            object somethingNotCastable = 1;
+
+            // ACT
+            var exception = Assert.Throws<DesignByContractViolationException>(
+                () => somethingNotCastable.CastOrFail<string>()
             );
 
-            Assert.Throws<DesignByContractViolationException>(
-                () => ((object) null).CastOrFail<string>()
+            // ASSERT
+            Assert.That(exception.Message, Is.EqualTo("Expected object of type 'System.String' but was '1'"));
+        }
+
+        [Test]
+        public void CastOrFailWithNull()
+        {
+            // ARRANGE
+            object somethingNotCastable = null;
+
+            // ACT
+            var exception = Assert.Throws<DesignByContractViolationException>(
+                // ReSharper disable once ExpressionIsAlwaysNull
+                () => somethingNotCastable.CastOrFail<string>()
             );
 
-            "tekst".CastOrFail<string>();
+            // ASSERT
+            Assert.That(exception.Message, Is.EqualTo("Expected object of type 'System.String' but was 'null'"));
+        }
+
+        [Test]
+        public void CastOrFailSuccess()
+        {
+            // ARRANGE
+            var somethingCastable = "text";
+
+            // ACT
+            somethingCastable.CastOrFail<string>();
         }
 
         #endregion

diff --git a/Synergy.Contracts.Test/Failures/FailDateTimeTest.cs b/Synergy.Contracts.Test/Failures/FailDateTimeTest.cs
@@ -10,14 +10,30 @@ public class FailDateTimeTest
         [Test]
         public void IfNotMidnight()
         {
-            DateTime now = DateTime.Today.AddSeconds(1000);
+            // ARRANGE
+            DateTime notMidnight = DateTime.Today.AddSeconds(1000);
 
-            Assert.Throws<DesignByContractViolationException>(
-                () => Fail.IfNotMidnight( now, "ojtam ojtam")
+            // ACT
+            var exception = Assert.Throws<DesignByContractViolationException>(
+                () => Fail.IfNotMidnight(notMidnight, "date should have no hour nor second")
                 );
 
-            Fail.IfNotMidnight( DateTime.Today, "ojtam ojtam");
-            Fail.IfNotMidnight(null, "ojtam ojtam");
+            // ASSERT
+            Assert.That(exception.Message, Is.EqualTo("date should have no hour nor second"));
+        }
+
+        [Test]
+        public void IfNotMidnightSuccess()
+        {
+            // ACT
+            Fail.IfNotMidnight(DateTime.Today, "date should have no hour nor second");
+        }
+
+        [Test]
+        public void IfNotMidnightSuccessWithNull()
+        {
+            // ACT
+            Fail.IfNotMidnight(null, "date should have no hour nor second");
         }
 
         [Test]