Skip to content
This repository was archived by the owner on Dec 14, 2018. It is now read-only.

Commit 9861473

Browse files
committed
React to authentication API changes.
1 parent 0bfebf7 commit 9861473

File tree

4 files changed

+14
-12
lines changed

4 files changed

+14
-12
lines changed

src/Microsoft.AspNet.Mvc.Core/ActionResults/ChallengeResult.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,17 +46,17 @@ public ChallengeResult(IList<string> authenticationSchemes, AuthenticationProper
4646

4747
public override void ExecuteResult([NotNull] ActionContext context)
4848
{
49-
var response = context.HttpContext.Response;
49+
var auth = context.HttpContext.Authentication;
5050
if (AuthenticationSchemes.Count > 0)
5151
{
5252
foreach (var scheme in AuthenticationSchemes)
5353
{
54-
response.Challenge(Properties, scheme);
54+
auth.Challenge(scheme, Properties);
5555
}
5656
}
5757
else
5858
{
59-
response.Challenge(Properties);
59+
auth.Challenge(Properties);
6060
}
6161
}
6262
}

src/Microsoft.AspNet.Mvc.Core/Filters/AuthorizeFilter.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ public virtual async Task OnAuthorizationAsync([NotNull] AuthorizationContext co
3838
var newPrincipal = new ClaimsPrincipal();
3939
foreach (var scheme in Policy.ActiveAuthenticationSchemes)
4040
{
41-
var result = (await context.HttpContext.AuthenticateAsync(scheme))?.Principal;
41+
var result = (await context.HttpContext.Authentication.AuthenticateAsync(scheme))?.Principal;
4242
if (result != null)
4343
{
4444
newPrincipal.AddIdentities(result.Identities);

test/Microsoft.AspNet.Mvc.Core.Test/ActionResults/ChallengeResultTest.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ public void ChallengeResult_Execute()
1818
// Arrange
1919
var result = new ChallengeResult("", null);
2020
var httpContext = new Mock<HttpContext>();
21-
var httpResponse = new Mock<HttpResponse>();
22-
httpContext.Setup(o => o.Response).Returns(httpResponse.Object);
21+
var auth = new Mock<AuthenticationManager>();
22+
httpContext.Setup(o => o.Authentication).Returns(auth.Object);
2323

2424
var routeData = new RouteData();
2525
routeData.Routers.Add(Mock.Of<IRouter>());
@@ -32,7 +32,7 @@ public void ChallengeResult_Execute()
3232
result.ExecuteResult(actionContext);
3333

3434
// Assert
35-
httpResponse.Verify(c => c.Challenge(null, ""), Times.Exactly(1));
35+
auth.Verify(c => c.Challenge("", null), Times.Exactly(1));
3636
}
3737

3838
[Fact]
@@ -41,8 +41,8 @@ public void ChallengeResult_ExecuteNoSchemes()
4141
// Arrange
4242
var result = new ChallengeResult(new string[] { }, null);
4343
var httpContext = new Mock<HttpContext>();
44-
var httpResponse = new Mock<HttpResponse>();
45-
httpContext.Setup(o => o.Response).Returns(httpResponse.Object);
44+
var auth = new Mock<AuthenticationManager>();
45+
httpContext.Setup(o => o.Authentication).Returns(auth.Object);
4646

4747
var routeData = new RouteData();
4848
routeData.Routers.Add(Mock.Of<IRouter>());
@@ -55,7 +55,7 @@ public void ChallengeResult_ExecuteNoSchemes()
5555
result.ExecuteResult(actionContext);
5656

5757
// Assert
58-
httpResponse.Verify(c => c.Challenge((AuthenticationProperties)null), Times.Exactly(1));
58+
auth.Verify(c => c.Challenge((AuthenticationProperties)null), Times.Exactly(1));
5959
}
6060
}
6161
}

test/Microsoft.AspNet.Mvc.Core.Test/Filters/AuthorizeFilterTest.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -352,14 +352,16 @@ private AuthorizationContext GetAuthorizationContext(Action<ServiceCollection> r
352352

353353
// HttpContext
354354
var httpContext = new Mock<HttpContext>();
355+
var auth = new Mock<AuthenticationManager>();
356+
httpContext.Setup(o => o.Authentication).Returns(auth.Object);
355357
httpContext.SetupProperty(c => c.User);
356358
if (!anonymous)
357359
{
358360
httpContext.Object.User = validUser;
359361
}
360362
httpContext.SetupGet(c => c.RequestServices).Returns(serviceProvider);
361-
httpContext.Setup(c => c.AuthenticateAsync("Bearer")).ReturnsAsync(new AuthenticationResult(bearerPrincipal, new AuthenticationProperties(), new AuthenticationDescription()));
362-
httpContext.Setup(c => c.AuthenticateAsync("Basic")).ReturnsAsync(new AuthenticationResult(basicPrincipal, new AuthenticationProperties(), new AuthenticationDescription()));
363+
auth.Setup(c => c.AuthenticateAsync("Bearer")).ReturnsAsync(new AuthenticationResult(bearerPrincipal, new AuthenticationProperties(), new AuthenticationDescription()));
364+
auth.Setup(c => c.AuthenticateAsync("Basic")).ReturnsAsync(new AuthenticationResult(basicPrincipal, new AuthenticationProperties(), new AuthenticationDescription()));
363365

364366
// AuthorizationContext
365367
var actionContext = new ActionContext(

0 commit comments

Comments
 (0)