Skip to content

Commit 86c802d

Browse files
committed
Refactor AcceptedAtActionResultAssertions to use ObjectResultAssertionsBase.
1 parent ae13c8b commit 86c802d

File tree

1 file changed

+5
-28
lines changed

1 file changed

+5
-28
lines changed

src/FluentAssertions.AspNetCore.Mvc/AcceptedAtActionResultAssertions.cs

Lines changed: 5 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ namespace FluentAssertions.AspNetCore.Mvc
1010
/// Contains a number of methods to assert that a <see cref="AcceptedAtActionResult"/> is in the expected state.
1111
/// </summary>
1212
[DebuggerNonUserCode]
13-
public class AcceptedAtActionResultAssertions : ObjectAssertions
13+
public class AcceptedAtActionResultAssertions : ObjectResultAssertionsBase<AcceptedAtActionResult, AcceptedAtActionResultAssertions>
1414
{
1515
/// <summary>
1616
/// Initializes a new instance of the <see cref="AcceptedAtActionResultAssertions" /> class.
@@ -30,7 +30,7 @@ public AcceptedAtActionResultAssertions(AcceptedAtActionResult subject) : base(s
3030
/// </param>
3131
public AcceptedAtActionResultAssertions WithActionName(string expectedActionName, string reason = "", params object[] reasonArgs)
3232
{
33-
string actualActionName = (Subject as AcceptedAtActionResult)?.ActionName;
33+
string actualActionName = ObjectResultSubject?.ActionName;
3434

3535
Execute.Assertion
3636
.ForCondition(string.Equals(actualActionName, expectedActionName, StringComparison.OrdinalIgnoreCase))
@@ -54,7 +54,7 @@ public AcceptedAtActionResultAssertions WithActionName(string expectedActionName
5454
/// </param>
5555
public AcceptedAtActionResultAssertions WithControllerName(string expectedControllerName, string reason = "", params object[] reasonArgs)
5656
{
57-
string actualControllerName = (Subject as AcceptedAtActionResult)?.ControllerName;
57+
string actualControllerName = ObjectResultSubject?.ControllerName;
5858

5959
Execute.Assertion
6060
.ForCondition(string.Equals(actualControllerName, expectedControllerName, StringComparison.OrdinalIgnoreCase))
@@ -79,35 +79,12 @@ public AcceptedAtActionResultAssertions WithControllerName(string expectedContro
7979
/// </param>
8080
public AcceptedAtActionResultAssertions WithRouteValue(string key, object expectedValue, string reason = "", params object[] reasonArgs)
8181
{
82-
var subjectTyped = Subject as AcceptedAtActionResult;
82+
var actualRouteValues = ObjectResultSubject.RouteValues;
8383

84-
AssertionHelpers.AssertStringObjectDictionary(subjectTyped.RouteValues,
84+
AssertionHelpers.AssertStringObjectDictionary(actualRouteValues,
8585
"AcceptedAtActionResult.RouteValues", key, expectedValue, reason, reasonArgs);
8686

8787
return this;
8888
}
89-
90-
/// <summary>
91-
/// Asserts the value is of the expected type.
92-
/// </summary>
93-
/// <typeparam name="TValue">The expected type.</typeparam>
94-
/// <returns>The typed value.</returns>
95-
public TValue ValueAs<TValue>()
96-
{
97-
var subjectTyped = Subject as AcceptedAtActionResult;
98-
var value = subjectTyped.Value;
99-
100-
if (value == null)
101-
Execute.Assertion
102-
.WithDefaultIdentifier("AcceptedAtActionResult.Value")
103-
.FailWith(FailureMessages.CommonNullWasSuppliedFailMessage, typeof(TValue));
104-
105-
Execute.Assertion
106-
.ForCondition(value is TValue)
107-
.WithDefaultIdentifier("AcceptedAtActionResult.Value")
108-
.FailWith(FailureMessages.CommonTypeFailMessage, typeof(TValue), value.GetType());
109-
110-
return (TValue)value;
111-
}
11289
}
11390
}

0 commit comments

Comments
 (0)