diff options
| -rw-r--r-- | samples/OpenIdProviderMvc/Controllers/OpenIdController.cs | 2 | ||||
| -rw-r--r-- | src/DotNetOpenAuth.AspNet.Test/OAuth2ClientTest.cs | 17 |
2 files changed, 14 insertions, 5 deletions
diff --git a/samples/OpenIdProviderMvc/Controllers/OpenIdController.cs b/samples/OpenIdProviderMvc/Controllers/OpenIdController.cs index bd6de1b..2f45ee8 100644 --- a/samples/OpenIdProviderMvc/Controllers/OpenIdController.cs +++ b/samples/OpenIdProviderMvc/Controllers/OpenIdController.cs @@ -111,7 +111,7 @@ namespace OpenIdProviderMvc.Controllers { !this.UserControlsIdentifier(ProviderEndpoint.PendingAuthenticationRequest)) { // The user shouldn't have gotten this far without controlling the identifier we'd send an assertion for. - return new HttpStatusCodeResult((int)HttpStatusCode.BadRequest); + throw new InvalidOperationException(); } if (ProviderEndpoint.PendingAnonymousRequest != null) { diff --git a/src/DotNetOpenAuth.AspNet.Test/OAuth2ClientTest.cs b/src/DotNetOpenAuth.AspNet.Test/OAuth2ClientTest.cs index 89a483c..e60df01 100644 --- a/src/DotNetOpenAuth.AspNet.Test/OAuth2ClientTest.cs +++ b/src/DotNetOpenAuth.AspNet.Test/OAuth2ClientTest.cs @@ -47,7 +47,16 @@ namespace DotNetOpenAuth.AspNet.Test { var client = new MockOAuth2Client(); // Act && Assert - Assert.Throws<ArgumentNullException>(() => client.VerifyAuthentication(null)); + Assert.Throws<ArgumentNullException>(() => client.VerifyAuthentication(null, new Uri("http://me.com"))); + } + + [TestCase] + public void VerifyAuthenticationWithoutReturnUrlThrows() { + // Arrange + var client = new MockOAuth2Client(); + + // Act && Assert + Assert.Throws<InvalidOperationException>(() => client.VerifyAuthentication(new Mock<HttpContextBase>().Object)); } [TestCase] @@ -59,7 +68,7 @@ namespace DotNetOpenAuth.AspNet.Test { context.Setup(c => c.Request.QueryString).Returns(queryStrings); // Act - AuthenticationResult result = client.VerifyAuthentication(context.Object); + AuthenticationResult result = client.VerifyAuthentication(context.Object, new Uri("http://me.com")); // Assert Assert.IsFalse(result.IsSuccessful); @@ -75,7 +84,7 @@ namespace DotNetOpenAuth.AspNet.Test { context.Setup(c => c.Request.QueryString).Returns(queryStrings); // Act - AuthenticationResult result = client.VerifyAuthentication(context.Object); + AuthenticationResult result = client.VerifyAuthentication(context.Object, new Uri("http://me.com")); // Assert Assert.IsFalse(result.IsSuccessful); @@ -91,7 +100,7 @@ namespace DotNetOpenAuth.AspNet.Test { context.Setup(c => c.Request.QueryString).Returns(queryStrings); // Act - AuthenticationResult result = client.VerifyAuthentication(context.Object); + AuthenticationResult result = client.VerifyAuthentication(context.Object, new Uri("http://me.com")); // Assert Assert.True(result.IsSuccessful); |