Fix tests to work with new default
This commit is contained in:
parent
d433f034fa
commit
38d33cfbf2
|
|
@ -1247,7 +1247,7 @@ namespace Microsoft.AspNetCore.Authentication.Cookies
|
||||||
=> CreateServerWithServices(s =>
|
=> CreateServerWithServices(s =>
|
||||||
{
|
{
|
||||||
s.AddSingleton<ISystemClock>(_clock);
|
s.AddSingleton<ISystemClock>(_clock);
|
||||||
s.AddAuthentication().AddCookie(configureOptions);
|
s.AddAuthentication(o => o.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme).AddCookie(configureOptions);
|
||||||
s.AddSingleton<IClaimsTransformation, ClaimsTransformer>();
|
s.AddSingleton<IClaimsTransformation, ClaimsTransformer>();
|
||||||
}, testpath, baseAddress);
|
}, testpath, baseAddress);
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -86,24 +86,6 @@ namespace Microsoft.AspNetCore.Authentication
|
||||||
await Assert.ThrowsAsync<InvalidOperationException>(() => server.SendAsync("http://example.com/auth/One"));
|
await Assert.ThrowsAsync<InvalidOperationException>(() => server.SendAsync("http://example.com/auth/One"));
|
||||||
}
|
}
|
||||||
|
|
||||||
[Fact]
|
|
||||||
public async Task VerifyDefaultBehavior()
|
|
||||||
{
|
|
||||||
var server = CreateServer();
|
|
||||||
|
|
||||||
await Assert.ThrowsAsync<InvalidOperationException>(() => server.SendAsync("http://example.com/auth"));
|
|
||||||
|
|
||||||
var response = await server.CreateClient().GetAsync("http://example.com/add/One");
|
|
||||||
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
|
||||||
var transaction = await server.SendAsync("http://example.com/auth");
|
|
||||||
Assert.Equal("One", transaction.FindClaimValue(ClaimTypes.NameIdentifier, "One"));
|
|
||||||
response = await server.CreateClient().GetAsync("http://example.com/add/Two");
|
|
||||||
Assert.Equal(HttpStatusCode.OK, response.StatusCode);
|
|
||||||
|
|
||||||
// Default will blow up since now there's two
|
|
||||||
await Assert.ThrowsAsync<InvalidOperationException>(() => server.SendAsync("http://example.com/auth"));
|
|
||||||
}
|
|
||||||
|
|
||||||
public class TestOptions : AuthenticationSchemeOptions
|
public class TestOptions : AuthenticationSchemeOptions
|
||||||
{
|
{
|
||||||
public Singleton Instance { get; set; }
|
public Singleton Instance { get; set; }
|
||||||
|
|
|
||||||
|
|
@ -81,8 +81,7 @@ namespace Microsoft.AspNetCore.Authentication.Facebook
|
||||||
{
|
{
|
||||||
services.AddAuthentication(options =>
|
services.AddAuthentication(options =>
|
||||||
{
|
{
|
||||||
options.DefaultSignInScheme = "External";
|
options.DefaultScheme = "External";
|
||||||
options.DefaultAuthenticateScheme = "External";
|
|
||||||
})
|
})
|
||||||
.AddCookie("External", o => { })
|
.AddCookie("External", o => { })
|
||||||
.AddFacebook(o =>
|
.AddFacebook(o =>
|
||||||
|
|
@ -221,7 +220,7 @@ namespace Microsoft.AspNetCore.Authentication.Facebook
|
||||||
{
|
{
|
||||||
services.AddAuthentication(options =>
|
services.AddAuthentication(options =>
|
||||||
{
|
{
|
||||||
options.DefaultAuthenticateScheme = CookieAuthenticationDefaults.AuthenticationScheme;
|
options.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme;
|
||||||
})
|
})
|
||||||
.AddCookie()
|
.AddCookie()
|
||||||
.AddFacebook(o =>
|
.AddFacebook(o =>
|
||||||
|
|
|
||||||
|
|
@ -1050,7 +1050,7 @@ namespace Microsoft.AspNetCore.Authentication.Google
|
||||||
services.AddTransient<IClaimsTransformation, ClaimsTransformer>();
|
services.AddTransient<IClaimsTransformation, ClaimsTransformer>();
|
||||||
services.AddAuthentication(o =>
|
services.AddAuthentication(o =>
|
||||||
{
|
{
|
||||||
o.DefaultAuthenticateScheme = TestExtensions.CookieAuthenticationScheme;
|
o.DefaultScheme = TestExtensions.CookieAuthenticationScheme;
|
||||||
o.DefaultChallengeScheme = GoogleDefaults.AuthenticationScheme;
|
o.DefaultChallengeScheme = GoogleDefaults.AuthenticationScheme;
|
||||||
});
|
});
|
||||||
services.AddAuthentication()
|
services.AddAuthentication()
|
||||||
|
|
|
||||||
|
|
@ -748,7 +748,7 @@ namespace Microsoft.AspNetCore.Authentication.JwtBearer
|
||||||
}
|
}
|
||||||
});
|
});
|
||||||
})
|
})
|
||||||
.ConfigureServices(services => services.AddAuthentication().AddJwtBearer(options));
|
.ConfigureServices(services => services.AddAuthentication(o => o.DefaultScheme = JwtBearerDefaults.AuthenticationScheme).AddJwtBearer(options));
|
||||||
|
|
||||||
return new TestServer(builder);
|
return new TestServer(builder);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -226,7 +226,7 @@ namespace Microsoft.AspNetCore.Authentication.Tests.MicrosoftAccount
|
||||||
{
|
{
|
||||||
services.AddAuthentication(o =>
|
services.AddAuthentication(o =>
|
||||||
{
|
{
|
||||||
o.DefaultAuthenticateScheme = TestExtensions.CookieAuthenticationScheme;
|
o.DefaultScheme = TestExtensions.CookieAuthenticationScheme;
|
||||||
});
|
});
|
||||||
services.AddAuthentication()
|
services.AddAuthentication()
|
||||||
.AddCookie(TestExtensions.CookieAuthenticationScheme, o => { })
|
.AddCookie(TestExtensions.CookieAuthenticationScheme, o => { })
|
||||||
|
|
|
||||||
|
|
@ -111,7 +111,7 @@ namespace Microsoft.AspNetCore.Authentication.Test.OpenIdConnect
|
||||||
{
|
{
|
||||||
services.AddAuthentication(o =>
|
services.AddAuthentication(o =>
|
||||||
{
|
{
|
||||||
o.DefaultAuthenticateScheme = CookieAuthenticationDefaults.AuthenticationScheme;
|
o.DefaultScheme = CookieAuthenticationDefaults.AuthenticationScheme;
|
||||||
});
|
});
|
||||||
services.AddAuthentication()
|
services.AddAuthentication()
|
||||||
.AddCookie()
|
.AddCookie()
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue