Allow to pass IssueBuilder to IssueChecker

cake-contrib · Apr 19, 2019 · 8da63f6 · 8da63f6
1 parent 25c4ede
commit 8da63f6
Show file tree

Hide file tree

Showing 2 changed files with 78 additions and 0 deletions.
diff --git a/src/Cake.Issues.Testing/IssueChecker.cs b/src/Cake.Issues.Testing/IssueChecker.cs
@@ -8,6 +8,23 @@
     /// </summary>
     public static class IssueChecker
     {
+        /// <summary>
+        /// Checks values of an issue.
+        /// </summary>
+        /// <param name="issueToCheck">Issue which should be checked.</param>
+        /// <param name="expectedIssue">Description of the expected issue.</param>
+        public static void Check(
+            IIssue issueToCheck,
+            IssueBuilder expectedIssue)
+        {
+            issueToCheck.NotNull(nameof(issueToCheck));
+            expectedIssue.NotNull(nameof(expectedIssue));
+
+            Check(
+                issueToCheck,
+                expectedIssue.Create());
+        }
+
         /// <summary>
         /// Checks values of an issue.
         /// </summary>

diff --git a/src/Cake.Issues.Tests/Testing/IssueCheckerTests.cs b/src/Cake.Issues.Tests/Testing/IssueCheckerTests.cs
@@ -7,6 +7,67 @@
 
     public sealed class IssueCheckerTests
     {
+        public sealed class TheCheckMethodWithIssueBuilder
+        {
+            [Fact]
+            public void Should_Throw_If_IssueToCheck_Is_Null()
+            {
+#pragma warning disable SA1123 // Do not place regions within elements
+                #region DupFinder Exclusion
+#pragma warning restore SA1123 // Do not place regions within elements
+
+                // Given
+                var fixture = new IssueBuilderFixture();
+                IIssue issueToCheck = null;
+                var expectedIssue = fixture.IssueBuilder;
+
+                // When
+                var result = Record.Exception(() =>
+                    IssueChecker.Check(issueToCheck, expectedIssue));
+
+                // Then
+                result.IsArgumentNullException("issueToCheck");
+
+                #endregion
+            }
+
+            [Fact]
+            public void Should_Throw_If_ExpectedIssue_Is_Null()
+            {
+#pragma warning disable SA1123 // Do not place regions within elements
+                #region DupFinder Exclusion
+#pragma warning restore SA1123 // Do not place regions within elements
+
+                // Given
+                var fixture = new IssueBuilderFixture();
+                var issueToCheck = fixture.IssueBuilder.Create();
+                IssueBuilder expectedIssue = null;
+
+                // When
+                var result = Record.Exception(() =>
+                    IssueChecker.Check(issueToCheck, expectedIssue));
+
+                // Then
+                result.IsArgumentNullException("expectedIssue");
+
+                #endregion
+            }
+
+            [Fact]
+            public void Should_Not_Throw_If_Issues_Are_Identical()
+            {
+                // Given
+                var fixture = new IssueCheckerFixture();
+                var issueToCheck = fixture.IssueBuilder.Create();
+                var expectedIssue = fixture.IssueBuilder;
+
+                // When
+                IssueChecker.Check(issueToCheck, expectedIssue);
+
+                // Then
+            }
+        }
+
         public sealed class TheCheckMethodComparingTwoIssues
         {
             [Fact]