Added tests that cover the situation where the subject is null to Dat…

…aTableCollectionAssertionExtensionsSpecs.cs, DataColumnCollectionAssertionExtensionsSpecs.cs and DataRowCollectionAssertionExtensionSpecs.cs. Corrected indentation error in DataRowCollectionAssertionExtensions.cs.
fluentassertions · Feb 28, 2022 · db60b38 · db60b38
1 parent 2d54c82
commit db60b38
Show file tree

Hide file tree

Showing 4 changed files with 103 additions and 1 deletion.
diff --git a/Src/FluentAssertions/DataRowCollectionAssertionExtensions.cs b/Src/FluentAssertions/DataRowCollectionAssertionExtensions.cs
@@ -151,5 +151,5 @@ public static class DataRowCollectionAssertionExtensions
 
             return new AndConstraint<GenericCollectionAssertions<DataRow>>(assertion);
         }
-        }
+    }
 }
diff --git a/...s/FluentAssertions.Specs/Collections/Data/DataColumnCollectionAssertionExtensionsSpecs.cs b/...s/FluentAssertions.Specs/Collections/Data/DataColumnCollectionAssertionExtensionsSpecs.cs
@@ -122,6 +122,23 @@ public void When_references_are_different_it_should_succeed()
 
         public class HaveSameCount
         {
+            [Fact]
+            public void When_subject_is_null_it_should_fail()
+            {
+                // Arrange
+                var subject = default(DataColumnCollection);
+
+                var expectation = new DataTable().Columns;
+
+                // Act
+                Action action =
+                    () => subject.Should().HaveSameCount(expectation, because: "we {0}", "care");
+
+                // Assert
+                action.Should().Throw<XunitException>().WithMessage(
+                    "Expected * to have the same count as * because we care, but found <null>.*");
+            }
+
             [Fact]
             public void When_expectation_is_null_it_should_fail()
             {
@@ -262,6 +279,23 @@ public void When_collection_is_compared_with_typed_collection_with_different_num
 
         public class NotHaveSameCount
         {
+            [Fact]
+            public void When_subject_is_null_it_should_fail()
+            {
+                // Arrange
+                var subject = default(DataColumnCollection);
+
+                var expectation = new DataTable().Columns;
+
+                // Act
+                Action action =
+                    () => subject.Should().NotHaveSameCount(expectation, because: "we {0}", "care");
+
+                // Assert
+                action.Should().Throw<XunitException>().WithMessage(
+                    "Expected * to not have the same count as * because we care, but found <null>.*");
+            }
+
             [Fact]
             public void When_expectation_is_null_it_should_fail()
             {

diff --git a/Tests/FluentAssertions.Specs/Collections/Data/DataRowCollectionAssertionExtensionsSpecs.cs b/Tests/FluentAssertions.Specs/Collections/Data/DataRowCollectionAssertionExtensionsSpecs.cs
@@ -163,6 +163,23 @@ public void When_references_are_different_it_should_succeed()
 
         public class HaveSameCount
         {
+            [Fact]
+            public void When_subject_is_null_it_should_fail()
+            {
+                // Arrange
+                var subject = default(DataRowCollection);
+
+                var expectation = new DataTable().Rows;
+
+                // Act
+                Action action =
+                    () => subject.Should().HaveSameCount(expectation, because: "we {0}", "care");
+
+                // Assert
+                action.Should().Throw<XunitException>().WithMessage(
+                    "Expected * to have the same count as * because we care, but found <null>.*");
+            }
+
             [Fact]
             public void When_expectation_is_null_it_should_fail()
             {
@@ -302,6 +319,23 @@ public void When_collection_is_compared_with_typed_collection_with_different_num
 
         public class NotHaveSameCount
         {
+            [Fact]
+            public void When_subject_is_null_it_should_fail()
+            {
+                // Arrange
+                var subject = default(DataRowCollection);
+
+                var expectation = new DataTable().Rows;
+
+                // Act
+                Action action =
+                    () => subject.Should().NotHaveSameCount(expectation, because: "we {0}", "care");
+
+                // Assert
+                action.Should().Throw<XunitException>().WithMessage(
+                    "Expected * to not have the same count as * because we care, but found <null>.*");
+            }
+
             [Fact]
             public void When_expectation_is_null_it_should_fail()
             {

diff --git a/Tests/FluentAssertions.Specs/Collections/Data/DataTableCollectionAssertionExtensionsSpecs.cs b/Tests/FluentAssertions.Specs/Collections/Data/DataTableCollectionAssertionExtensionsSpecs.cs
@@ -138,6 +138,23 @@ public void When_references_are_different_it_should_succeed()
 
         public class HaveSameCount
         {
+            [Fact]
+            public void When_subject_is_null_it_should_fail()
+            {
+                // Arrange
+                var subject = default(DataTableCollection);
+
+                var expectation = new DataSet().Tables;
+
+                // Act
+                Action action =
+                    () => subject.Should().HaveSameCount(expectation, because: "we {0}", "care");
+
+                // Assert
+                action.Should().Throw<XunitException>().WithMessage(
+                    "Expected * to have the same count as * because we care, but found <null>.*");
+            }
+
             [Fact]
             public void When_expectation_is_null_it_should_fail()
             {
@@ -277,6 +294,23 @@ public void When_collection_is_compared_with_typed_collection_with_different_num
 
         public class NotHaveSameCount
         {
+            [Fact]
+            public void When_subject_is_null_it_should_fail()
+            {
+                // Arrange
+                var subject = default(DataTableCollection);
+
+                var expectation = new DataSet().Tables;
+
+                // Act
+                Action action =
+                    () => subject.Should().NotHaveSameCount(expectation, because: "we {0}", "care");
+
+                // Assert
+                action.Should().Throw<XunitException>().WithMessage(
+                    "Expected * to not have the same count as * because we care, but found <null>.*");
+            }
+
             [Fact]
             public void When_expectation_is_null_it_should_fail()
             {