Skip to content

Commit 2688709

Browse files
committed
Fix ChallengeResultAssertions tests.
1 parent 297b9ce commit 2688709

File tree

2 files changed

+84
-97
lines changed

2 files changed

+84
-97
lines changed

src/FluentAssertions.AspNetCore.Mvc/ChallengeResultAssertions.cs

Lines changed: 16 additions & 68 deletions
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,8 @@ public ChallengeResultAssertions WithAuthenticationProperties(AuthenticationProp
6565
Execute.Assertion
6666
.ForCondition(actualAuthenticationProperties == expectedAuthenticationProperties)
6767
.BecauseOf(reason, reasonArgs)
68-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties", expectedAuthenticationProperties, actualAuthenticationProperties));
68+
.WithDefaultIdentifier("ChallengeResult.AuthenticationProperties")
69+
.FailWith(FailureMessages.CommonFailMessage, expectedAuthenticationProperties, actualAuthenticationProperties);
6970

7071
return this;
7172
}
@@ -88,7 +89,8 @@ public ChallengeResultAssertions WithIsPersistent(bool expectedIsPersistent, str
8889
Execute.Assertion
8990
.ForCondition(actualIsPersistent == expectedIsPersistent)
9091
.BecauseOf(reason, reasonArgs)
91-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.IsPersistent", expectedIsPersistent, actualIsPersistent));
92+
.WithDefaultIdentifier("ChallengeResult.AuthenticationProperties.IsPersistent")
93+
.FailWith(FailureMessages.CommonFailMessage, expectedIsPersistent, actualIsPersistent);
9294

9395
return this;
9496
}
@@ -111,7 +113,8 @@ public ChallengeResultAssertions WithRedirectUri(string expectedRedirectUri, str
111113
Execute.Assertion
112114
.ForCondition(string.Equals(actualRedirectUri, expectedRedirectUri))
113115
.BecauseOf(reason, reasonArgs)
114-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.RedirectUri", expectedRedirectUri, actualRedirectUri));
116+
.WithDefaultIdentifier("ChallengeResult.AuthenticationProperties.RedirectUri")
117+
.FailWith(FailureMessages.CommonFailMessage, expectedRedirectUri, actualRedirectUri);
115118

116119
return this;
117120
}
@@ -131,41 +134,13 @@ public ChallengeResultAssertions WithIssuedUtc(DateTimeOffset? expectedIssuedUtc
131134
{
132135
var actualResult = IssuedUtc;
133136

134-
var expectedIssuedUtcAsString = expectedIssuedUtc?.ToString("r", (IFormatProvider) CultureInfo.InvariantCulture);
135-
136-
var expectedResult = DateTimeOffset.TryParseExact(expectedIssuedUtcAsString, "r", (IFormatProvider)CultureInfo.InvariantCulture, DateTimeStyles.RoundtripKind, out var result)
137-
? new DateTimeOffset?(result)
138-
: new DateTimeOffset?();
139-
140-
if (actualResult == null && expectedResult == null)
141-
{
142-
return this;
143-
}
144-
145-
if (actualResult == null)
146-
{
147-
Execute.Assertion
148-
.ForCondition(false)
149-
.BecauseOf(reason, reasonArgs)
150-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.IssuedUtc", expectedResult, null));
151-
152-
return this;
153-
}
154-
155-
if (expectedResult == null)
156-
{
157-
Execute.Assertion
158-
.ForCondition(false)
159-
.BecauseOf(reason, reasonArgs)
160-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.IssuedUtc", null, actualResult));
161-
162-
return this;
163-
}
137+
var expectedResult = AssertionHelpers.RoundToSeconds(expectedIssuedUtc);
164138

165139
Execute.Assertion
166-
.ForCondition(DateTimeOffset.Compare(expectedResult.Value, actualResult.Value) == 0)
140+
.ForCondition(expectedResult == actualResult)
167141
.BecauseOf(reason, reasonArgs)
168-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.IssuedUtc", expectedResult.Value, actualResult.Value));
142+
.WithDefaultIdentifier("ChallengeResult.AuthenticationProperties.IssuedUtc")
143+
.FailWith(FailureMessages.CommonFailMessage, expectedResult, actualResult);
169144

170145
return this;
171146
}
@@ -185,41 +160,13 @@ public ChallengeResultAssertions WithExpiresUtc(DateTimeOffset? expectedExpiresU
185160
{
186161
var actualResult = ExpiresUtc;
187162

188-
var expectedExpiresUtcAsString = expectedExpiresUtc?.ToString("r", (IFormatProvider)CultureInfo.InvariantCulture);
189-
190-
var expectedResult = DateTimeOffset.TryParseExact(expectedExpiresUtcAsString, "r", (IFormatProvider)CultureInfo.InvariantCulture, DateTimeStyles.RoundtripKind, out var result)
191-
? new DateTimeOffset?(result)
192-
: new DateTimeOffset?();
193-
194-
if (actualResult == null && expectedResult == null)
195-
{
196-
return this;
197-
}
198-
199-
if (actualResult == null)
200-
{
201-
Execute.Assertion
202-
.ForCondition(false)
203-
.BecauseOf(reason, reasonArgs)
204-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.ExpiresUtc", expectedResult, null));
205-
206-
return this;
207-
}
208-
209-
if (expectedResult == null)
210-
{
211-
Execute.Assertion
212-
.ForCondition(false)
213-
.BecauseOf(reason, reasonArgs)
214-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.ExpiresUtc", null, actualResult));
215-
216-
return this;
217-
}
163+
var expectedResult = AssertionHelpers.RoundToSeconds(expectedExpiresUtc);
218164

219165
Execute.Assertion
220-
.ForCondition(DateTimeOffset.Compare(expectedResult.Value, actualResult.Value) == 0)
166+
.ForCondition(expectedResult == actualResult)
221167
.BecauseOf(reason, reasonArgs)
222-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.ExpiresUtc", expectedResult.Value, actualResult.Value));
168+
.WithDefaultIdentifier("ChallengeResult.AuthenticationProperties.ExpiresUtc")
169+
.FailWith(FailureMessages.CommonFailMessage, expectedResult, actualResult);
223170

224171
return this;
225172
}
@@ -243,7 +190,8 @@ public ChallengeResultAssertions WithAllowRefresh(bool? expectedAllowRefresh, st
243190
Execute.Assertion
244191
.ForCondition(actualAllowRefresh == expectedAllowRefresh)
245192
.BecauseOf(reason, reasonArgs)
246-
.FailWith(string.Format(FailureMessages.CommonFailMessage, "ChallengeResult.AuthenticationProperties.AllowRefresh", expectedAllowRefresh, actualAllowRefresh));
193+
.WithDefaultIdentifier("ChallengeResult.AuthenticationProperties.AllowRefresh")
194+
.FailWith(FailureMessages.CommonFailMessage, expectedAllowRefresh, actualAllowRefresh);
247195

248196
return this;
249197
}

0 commit comments

Comments
 (0)