Constructors on all OpenID direct response messages now require the OpenID version of the message, allowing for RPs to receive response messages in versions differing from their request message.

Also added test to verify this, and OpenIdProvider error messages w/o requiring ASP.NET hosting.

1 files changed, 15 insertions, 14 deletions

diff --git a/src/DotNetOpenAuth.Test/OpenId/Provider/OpenIdProviderTests.cs b/src/DotNetOpenAuth.Test/OpenId/Provider/OpenIdProviderTests.cs
index bb035c3..114a135 100644
--- a/src/DotNetOpenAuth.Test/OpenId/Provider/OpenIdProviderTests.cs
+++ b/src/DotNetOpenAuth.Test/OpenId/Provider/OpenIdProviderTests.cs
@@ -101,21 +101,22 @@ namespace DotNetOpenAuth.Test.OpenId.Provider {
 			coordinator.Run();
 		}
 
-		////[TestMethod]
-		////public void BadRequestsGenerateValidErrorResponses() {
-		////    var coordinator = new OpenIdCoordinator(
-		////        rp => {
-		////            var nonOpenIdMessage = new Mocks.TestDirectedMessage();
-		////            nonOpenIdMessage.Recipient = OPUri;
-		////            nonOpenIdMessage.HttpMethods = HttpDeliveryMethods.PostRequest;
-		////            MessagingTestBase.GetStandardTestMessage(MessagingTestBase.FieldFill.AllRequired, nonOpenIdMessage);
-		////            var response = rp.Channel.Request<DirectErrorResponse>(nonOpenIdMessage);
-		////            Assert.IsNotNull(response.ErrorMessage);
-		////        },
-		////        AutoProvider);
+		[TestMethod]
+		public void BadRequestsGenerateValidErrorResponses() {
+			var coordinator = new OpenIdCoordinator(
+				rp => {
+					var nonOpenIdMessage = new Mocks.TestDirectedMessage();
+					nonOpenIdMessage.Recipient = OPUri;
+					nonOpenIdMessage.HttpMethods = HttpDeliveryMethods.PostRequest;
+					MessagingTestBase.GetStandardTestMessage(MessagingTestBase.FieldFill.AllRequired, nonOpenIdMessage);
+					var response = rp.Channel.Request<DirectErrorResponse>(nonOpenIdMessage);
+					Assert.IsNotNull(response.ErrorMessage);
+					Assert.AreEqual(Protocol.Default.Version, response.Version);
+				},
+				AutoProvider);
 
-		////    coordinator.Run();
-		////}
+			coordinator.Run();
+		}
 
 		[TestMethod]
 		public void BadRequestsGenerateValidErrorResponsesHosted() {