Fixed tests

Author: HaoK

src/Microsoft.AspNetCore.Server.HttpSys/AuthenticationHandler.cs

@@ -3,7 +3,6 @@
 
 using System;
 using System.Security.Claims;
-using System.Security.Principal;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Authentication;
 using Microsoft.AspNetCore.Http;
@@ -19,12 +18,12 @@ internal class AuthenticationHandler : IAuthenticationHandler
 
         public Task<AuthenticateResult> AuthenticateAsync(AuthenticateContext context)
         {
-            var identity = (ClaimsIdentity)_requestContext.User?.Identity;
+            var identity = _requestContext.User?.Identity;
             var success = identity != null 
                 && identity.IsAuthenticated 
                 && string.Equals(context.AuthenticationScheme, identity.AuthenticationType, StringComparison.Ordinal);
             return Task.FromResult(success
-                ? AuthenticateResult.Success(new AuthenticationTicket(new ClaimsPrincipal(identity), properties: null, authenticationScheme: context.AuthenticationScheme))
+                ? AuthenticateResult.Success(new AuthenticationTicket(_requestContext.User, properties: null, authenticationScheme: context.AuthenticationScheme))
                 : AuthenticateResult.None());
         }
 

test/Microsoft.AspNetCore.Server.HttpSys.FunctionalTests/AuthenticationTests.cs

@@ -17,30 +17,6 @@ public class AuthenticationTests
         private static bool AllowAnoymous = true;
         private static bool DenyAnoymous = false;
 
-        [ConditionalTheory]
-        [InlineData(AuthenticationSchemes.Negotiate)]
-        [InlineData(AuthenticationSchemes.NTLM)]
-        [InlineData(AuthenticationSchemes.Basic)]
-        [InlineData(AuthenticationSchemes.Negotiate | AuthenticationSchemes.NTLM | /*AuthenticationSchemes.Digest |*/ AuthenticationSchemes.Basic)]
-        public async Task CanAuthenticate(AuthenticationSchemes authType)
-        {
-            var authTypeList = authType.ToString().Split(new char[] { ',', ' ' }, StringSplitOptions.RemoveEmptyEntries);
-            using (var server = Utilities.CreateDynamicHost(string.Empty, authType, AllowAnoymous, out var address, out var baseAddress, async httpContext =>
-            {
-                foreach (var scheme in authTypeList)
-                {
-                    var result = await httpContext.AuthenticateAsync(scheme);
-                    Assert.True(result.Succeeded);
-                }
-                httpContext.Response.StatusCode = 200;
-            }))
-            {
-                var response = await SendRequestAsync(address);
-                Assert.NotNull(response);
-                Assert.Equal(HttpStatusCode.OK, response.StatusCode);
-            }
-        }
-
         [ConditionalTheory]
         [InlineData(AuthenticationSchemes.None)]
         [InlineData(AuthenticationSchemes.Negotiate)]
@@ -176,7 +152,7 @@ public async Task AuthTypes_AllowAnonymousButSpecify401_Success(AuthenticationSc
         [InlineData(AuthenticationSchemes.Negotiate | AuthenticationSchemes.NTLM | /* AuthenticationSchemes.Digest |*/ AuthenticationSchemes.Basic)]
         public async Task AuthTypes_RequireAuth_Success(AuthenticationSchemes authType)
         {
-            using (var server = Utilities.CreateDynamicHost(string.Empty, authType, AllowAnoymous, out var address, out var baseAddress, httpContext =>
+            using (var server = Utilities.CreateDynamicHost(string.Empty, authType, DenyAnoymous, out var address, out var baseAddress, httpContext =>
             {
                 Assert.NotNull(httpContext.User);
                 Assert.NotNull(httpContext.User.Identity);