Skip to content

Commit 970fc9a

Browse files
committed
Fix AcceptedResultAssertions tests.
1 parent e621d1a commit 970fc9a

File tree

2 files changed

+34
-28
lines changed

2 files changed

+34
-28
lines changed

src/FluentAssertions.AspNetCore.Mvc/AcceptedResultAssertions.cs

+13-6
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,14 @@ public TValue ValueAs<TValue>()
5050
var value = AcceptedResultSubject.Value;
5151

5252
if (value == null)
53-
Execute.Assertion.FailWith(FailureMessages.CommonNullWasSuppliedFailMessage, "AcceptedResultAssertions.Value", typeof(TValue).Name);
53+
Execute.Assertion
54+
.WithDefaultIdentifier("AcceptedResultAssertions.Value")
55+
.FailWith(FailureMessages.CommonNullWasSuppliedFailMessage, typeof(TValue));
5456

5557
Execute.Assertion
5658
.ForCondition(value is TValue)
57-
.FailWith(FailureMessages.CommonTypeFailMessage, "AcceptedResultAssertions.Value", typeof(TValue).Name, value.GetType().Name);
59+
.WithDefaultIdentifier("AcceptedResultAssertions.Value")
60+
.FailWith(FailureMessages.CommonTypeFailMessage, typeof(TValue), value.GetType());
5861

5962
return (TValue)value;
6063
}
@@ -66,15 +69,17 @@ public TValue ValueAs<TValue>()
6669
/// The Uri.
6770
/// </param>
6871
/// <returns>The typed value.</returns>
69-
public AcceptedResultAssertions WithUri(Uri uri)
72+
public AcceptedResultAssertions WithUri(Uri uri, string reason = "", params object[] reasonArgs)
7073
{
7174
var expectedUri = !uri.IsAbsoluteUri
7275
? uri.GetComponents(UriComponents.SerializationInfoString, UriFormat.UriEscaped)
7376
: uri.AbsoluteUri;
7477

7578
Execute.Assertion
79+
.BecauseOf(reason, reasonArgs)
7680
.ForCondition(expectedUri == Location)
77-
.FailWith(FailureMessages.CommonFailMessage, "AcceptedResultAssertions.Uri", expectedUri, Location);
81+
.WithDefaultIdentifier("AcceptedResultAssertions.Uri")
82+
.FailWith(FailureMessages.CommonFailMessage, expectedUri, Location);
7883

7984
return this;
8085
}
@@ -86,11 +91,13 @@ public AcceptedResultAssertions WithUri(Uri uri)
8691
/// The Uri as string.
8792
/// </param>
8893
/// <returns>The typed value.</returns>
89-
public AcceptedResultAssertions WithUri(string uri)
94+
public AcceptedResultAssertions WithUri(string uri, string reason = "", params object[] reasonArgs)
9095
{
9196
Execute.Assertion
97+
.BecauseOf(reason, reasonArgs)
9298
.ForCondition(uri == Location)
93-
.FailWith(FailureMessages.CommonFailMessage, "AcceptedResultAssertions.Uri", uri, Location);
99+
.WithDefaultIdentifier("AcceptedResultAssertions.Uri")
100+
.FailWith(FailureMessages.CommonFailMessage, uri, Location);
94101

95102
return this;
96103
}

tests/FluentAssertions.AspNetCore.Mvc.Tests/AcceptedResultAssertions_Tests.cs

+21-22
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ namespace FluentAssertions.AspNetCore.Mvc.Tests
77
{
88
public class AcceptedResultAssertions_Tests
99
{
10+
public const string Reason = FailureMessageHelper.Reason;
11+
public readonly static object[] ReasonArgs = FailureMessageHelper.ReasonArgs;
12+
1013
private const string TestValue = "testValue";
1114

1215
private const string TestUriAsString = "http://localhost:5000";
@@ -19,46 +22,38 @@ public class AcceptedResultAssertions_Tests
1922
public void Value_GivenExpectedValue_ShouldPass()
2023
{
2124
var result = new TestController().Accepted(TestUri, TestValue);
22-
result.Should().BeAcceptedResult().Value.Should().Be(TestValue);
23-
}
2425

25-
[Fact]
26-
public void Value_GivenUnexpectedValue_ShouldFail()
27-
{
28-
var result = new TestController().Accepted(TestUri, TestValue);
29-
Action a = () => result.Should().BeAcceptedResult().Value.Should().Be("xyx");
30-
a.Should().Throw<Exception>();
26+
result.Should().BeAcceptedResult().Value.Should().BeSameAs(TestValue);
3127
}
3228

3329
[Fact]
3430
public void ValueAs_GivenExpectedValue_ShouldPass()
3531
{
3632
var result = new TestController().Accepted(TestUri, TestValue);
37-
result.Should().BeAcceptedResult().ValueAs<string>().Should().Be(TestValue);
38-
}
39-
40-
[Fact]
41-
public void ValueAs_GivenUnexpectedValue_ShouldFail()
42-
{
43-
var result = new TestController().Accepted(TestUri, TestValue);
44-
Action a = () => result.Should().BeAcceptedResult().ValueAs<string>().Should().Be("xyx");
45-
a.Should().Throw<Exception>();
33+
result.Should().BeAcceptedResult().ValueAs<string>().Should().BeSameAs(TestValue);
4634
}
4735

4836
[Fact]
4937
public void ValueAs_GivenWrongType_ShouldFail()
5038
{
5139
var result = new TestController().Accepted(TestUri, TestValue);
40+
var failureMessage = FailureMessageHelper.ExpectedContextTypeXButFoundY(
41+
"AcceptedResultAssertions.Value", typeof(int), typeof(string));
42+
5243
Action a = () => result.Should().BeAcceptedResult().ValueAs<int>().Should().Be(2);
44+
5345
a.Should().Throw<Exception>();
5446
}
5547

5648
[Fact]
5749
public void ValueAs_Null_ShouldFail()
5850
{
5951
ActionResult result = new AcceptedResult(TestUri, null);
60-
var failureMessage = FailureMessageHelper.Format(FailureMessages.CommonNullWasSuppliedFailMessage, "AcceptedResultAssertions.Value", typeof(object).Name);
52+
var failureMessage = FailureMessageHelper.ExpectedContextTypeXButFoundNull(
53+
"AcceptedResultAssertions.Value", typeof(object));
54+
6155
Action a = () => result.Should().BeAcceptedResult().ValueAs<object>();
56+
6257
a.Should().Throw<Exception>().WithMessage(failureMessage);
6358
}
6459

@@ -74,8 +69,10 @@ public void WithUri_GivenExpectedUri_ShouldPass()
7469
public void WithUri_GivenWrongUri_ShouldFail()
7570
{
7671
var result = new TestController().Accepted(TestWrongUri, TestValue);
77-
Action a = () => result.Should().BeAcceptedResult().WithUri(TestUri);
78-
var failureMessage = FailureMessageHelper.Format(FailureMessages.CommonFailMessage, "AcceptedResultAssertions.Uri", TestUri.ToString(), TestWrongUri.ToString());
72+
var failureMessage = FailureMessageHelper.ExpectedContextToBeXButY("AcceptedResultAssertions.Uri", TestUri.ToString(), TestWrongUri.ToString());
73+
74+
Action a = () => result.Should().BeAcceptedResult().WithUri(TestUri, Reason, ReasonArgs);
75+
7976
a.Should().Throw<Exception>().WithMessage(failureMessage);
8077
}
8178

@@ -91,8 +88,10 @@ public void WithUri_GivenExpectedUriAsString_ShouldPass()
9188
public void WithUri_GivenWrongUriAsString_ShouldFail()
9289
{
9390
var result = new TestController().Accepted(TestWrongUriAsString, TestValue);
94-
Action a = () => result.Should().BeAcceptedResult().WithUri(TestUriAsString);
95-
var failureMessage = FailureMessageHelper.Format(FailureMessages.CommonFailMessage, "AcceptedResultAssertions.Uri", TestUriAsString, TestWrongUriAsString);
91+
var failureMessage = FailureMessageHelper.ExpectedContextToBeXButY("AcceptedResultAssertions.Uri", TestUriAsString, TestWrongUriAsString);
92+
93+
Action a = () => result.Should().BeAcceptedResult().WithUri(TestUriAsString, Reason, ReasonArgs);
94+
9695
a.Should().Throw<Exception>().WithMessage(failureMessage);
9796
}
9897
}

0 commit comments

Comments
 (0)