Determine caller name for EnumAssertions.Be

fluentassertions · Mar 6, 2022 · c950341 · c950341
1 parent 418a405
commit c950341
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/Src/FluentAssertions/Primitives/EnumAssertions.cs b/Src/FluentAssertions/Primitives/EnumAssertions.cs
@@ -57,7 +57,7 @@ public AndConstraint<TAssertions> Be(TEnum expected, string because = "", params
             Execute.Assertion
                 .ForCondition(Subject?.Equals(expected) == true)
                 .BecauseOf(because, becauseArgs)
-                .FailWith("Expected the enum to be {0}{reason}, but found {1}.",
+                .FailWith("Expected {context:the enum} to be {0}{reason}, but found {1}.",
                     expected, Subject);
 
             return new AndConstraint<TAssertions>((TAssertions)this);

diff --git a/Src/FluentAssertions/Primitives/NullableEnumAssertions.cs b/Src/FluentAssertions/Primitives/NullableEnumAssertions.cs
@@ -42,7 +42,7 @@ public NullableEnumAssertions(TEnum? subject)
             Execute.Assertion
                 .ForCondition(Subject.HasValue)
                 .BecauseOf(because, becauseArgs)
-                .FailWith("Expected {context:nullable date and time} to have a value{reason}, but found {0}.", Subject);
+                .FailWith("Expected {context:nullable enum} to have a value{reason}, but found {0}.", Subject);
 
             return new AndWhichConstraint<TAssertions, TEnum>((TAssertions)this, Subject.GetValueOrDefault());
         }
@@ -77,7 +77,7 @@ public AndConstraint<TAssertions> NotHaveValue(string because = "", params objec
             Execute.Assertion
                 .ForCondition(!Subject.HasValue)
                 .BecauseOf(because, becauseArgs)
-                .FailWith("Did not expect {context:nullable date and time} to have a value{reason}, but found {0}.", Subject);
+                .FailWith("Did not expect {context:nullable enum} to have a value{reason}, but found {0}.", Subject);
 
             return new AndConstraint<TAssertions>((TAssertions)this);
         }

diff --git a/Tests/FluentAssertions.Specs/Primitives/EnumAssertionSpecs.cs b/Tests/FluentAssertions.Specs/Primitives/EnumAssertionSpecs.cs
@@ -169,7 +169,7 @@ public void When_enums_are_unequal_it_should_throw()
 
             // Assert
             act.Should().Throw<XunitException>()
-                .WithMessage("*because we want to test the failure message*");
+                .WithMessage("*subject*because we want to test the failure message*");
         }
 
         [Theory]

diff --git a/docs/_pages/releases.md b/docs/_pages/releases.md
@@ -14,6 +14,7 @@ sidebar:
 * Added the ability to exclude fields & properties marked as non-browsable in the code editor from structural equality equivalency comparisons - [#1807](https://github.com/fluentassertions/fluentassertions/pull/1807)
 
 ### Fixes
+* `EnumAssertions.Be` did not determine the caller name - [#1833](https://github.com/fluentassertions/fluentassertions/pull/1833)
 
 ### Fixes (Extensibility)