From d38fb1e49dfc037d379b8841add915f0964b63f1 Mon Sep 17 00:00:00 2001 From: Osmozy Date: Tue, 17 Nov 2015 00:28:50 +0300 Subject: [PATCH] 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 --- .../DefaultAuthorizationServiceTests.cs | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/test/Microsoft.AspNet.Authorization.Test/DefaultAuthorizationServiceTests.cs b/test/Microsoft.AspNet.Authorization.Test/DefaultAuthorizationServiceTests.cs index 968f1151bb..63f1168991 100644 --- a/test/Microsoft.AspNet.Authorization.Test/DefaultAuthorizationServiceTests.cs +++ b/test/Microsoft.AspNet.Authorization.Test/DefaultAuthorizationServiceTests.cs @@ -45,7 +45,7 @@ namespace Microsoft.AspNet.Authorization.Test 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 var allowed = await authorizationService.AuthorizeAsync(user, "Basic"); @@ -62,7 +62,10 @@ namespace Microsoft.AspNet.Authorization.Test { 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"));