huahaofeng
/
aspnetcore
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Summary of the changes 

- No need to set the authenticationType in the Authorize_ShouldAllowIfClaimIsPresent unit test, since we already have another unit test for these functionality.
 - Specified the authentication scheme of the authorization policy  in the Authorize_ShouldAllowIfClaimIsPresentWithSpecifiedAuthType unit test
This commit is contained in:
Osmozy 2015-11-17 00:28:50 +03:00 committed by Hao Kung
parent a8ef4df32a
commit d38fb1e49d
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
test/Microsoft.AspNet.Authorization.Test/DefaultAuthorizationServiceTests.cs
View File

@ -45,7 +45,7 @@ namespace Microsoft.AspNet.Authorization.Test
options.AddPolicy("Basic", policy => policy.RequireClaim("Permission", "CanViewPage")); options.AddPolicy("Basic", policy => policy.RequireClaim("Permission", "CanViewPage"));
}); });
}); });
var user = new ClaimsPrincipal(new ClaimsIdentity(new Claim[] { new Claim("Permission", "CanViewPage") }, "Basic")); var user = new ClaimsPrincipal(new ClaimsIdentity(new Claim[] { new Claim("Permission", "CanViewPage") }));
// Act // Act
var allowed = await authorizationService.AuthorizeAsync(user, "Basic"); var allowed = await authorizationService.AuthorizeAsync(user, "Basic");
@ -62,7 +62,10 @@ namespace Microsoft.AspNet.Authorization.Test
{ {
services.AddAuthorization(options => services.AddAuthorization(options =>
{ {
options.AddPolicy("Basic", policy => policy.RequireClaim("Permission", "CanViewPage")); options.AddPolicy("Basic", policy => {
policy.AddAuthenticationSchemes("Basic");
policy.RequireClaim("Permission", "CanViewPage");
});
}); });
}); });
var user = new ClaimsPrincipal(new ClaimsIdentity(new Claim[] { new Claim("Permission", "CanViewPage") }, "Basic")); var user = new ClaimsPrincipal(new ClaimsIdentity(new Claim[] { new Claim("Permission", "CanViewPage") }, "Basic"));