Fix wrong test names

fluentassertions · Apr 19, 2022 · 2733578 · 2733578
1 parent 679f0ff
commit 2733578
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 14 deletions.
diff --git a/Tests/FluentAssertions.Specs/Xml/XDocumentAssertionSpecs.cs b/Tests/FluentAssertions.Specs/Xml/XDocumentAssertionSpecs.cs
@@ -1177,7 +1177,7 @@ public void When_asserting_document_has_two_child_elements_and_it_does_it_succee
         }
 
         [Fact]
-        public void Asserting_document_inside_an_assertion_scope_it_checks_the_whole_assertion_scope_before_failing()
+        public void Asserting_document_null_inside_an_assertion_scope_it_checks_the_whole_assertion_scope_before_failing()
         {
             // Arrange
             XDocument document = null;
@@ -1197,7 +1197,7 @@ public void Asserting_document_inside_an_assertion_scope_it_checks_the_whole_ass
         }
 
         [Fact]
-        public void Asserting_document_inside_an_assertion_scope_it_checks_the_whole_assertion_scope_before_failing_()
+        public void Asserting_with_document_root_null_inside_an_assertion_scope_it_checks_the_whole_assertion_scope_before_failing()
         {
             // Arrange
             XDocument document = new();
@@ -1274,7 +1274,7 @@ public void Document_is_valid_and_expected_null_with_x_name_overload_it_fails()
         }
 
         [Fact]
-        public void Chaining_which_after_asserting_and_the_document_has_more_than_two_child_elements_it_fails()
+        public void Chaining_which_after_asserting_and_the_document_has_more_than_two_child_elements_it_passes()
         {
             // Arrange
             var document = XDocument.Parse(
@@ -1284,12 +1284,9 @@ public void Chaining_which_after_asserting_and_the_document_has_more_than_two_ch
                     <child />
                   </parent>");
 
-            // Act
-            Action act = () => document.Should().HaveElement("child", AtLeast.Twice())
+            // Act / Assert
+            document.Should().HaveElement("child", AtLeast.Twice())
                 .Which.Should().NotBeNull();
-
-            // Assert
-            act.Should().NotThrow<XunitException>();
         }
 
         [Fact]

diff --git a/Tests/FluentAssertions.Specs/Xml/XElementAssertionSpecs.cs b/Tests/FluentAssertions.Specs/Xml/XElementAssertionSpecs.cs
@@ -1324,7 +1324,7 @@ public void Element_is_valid_and_expected_null_with_x_name_overload_it_fails()
         }
 
         [Fact]
-        public void Chaining_which_after_asserting_and_the_element_has_more_than_two_child_elements_it_fails()
+        public void Chaining_which_after_asserting_and_the_element_has_more_than_two_child_elements_it_passes()
         {
             // Arrange
             var element = XElement.Parse(
@@ -1334,12 +1334,9 @@ public void Chaining_which_after_asserting_and_the_element_has_more_than_two_chi
                     <child />
                   </parent>");
 
-            // Act
-            Action act = () => element.Should().HaveElement("child", AtLeast.Twice())
+            // Act / Assert
+            element.Should().HaveElement("child", AtLeast.Twice())
                 .Which.Should().NotBeNull();
-
-            // Assert
-            act.Should().NotThrow();
         }
 
         [Fact]