diff options
author | Andrew Arnott <andrewarnott@gmail.com> | 2009-03-10 20:31:50 -0700 |
---|---|---|
committer | Andrew Arnott <andrewarnott@gmail.com> | 2009-03-10 20:31:50 -0700 |
commit | dd49ae4a865b21f0722efab87a4b0723ec1a6950 (patch) | |
tree | 379098bde760f95a452d995bdfdb364dec7bbb7e | |
parent | bd3b3f6b07dbd1d43c2ffdff4504fd5eec1eb388 (diff) | |
download | DotNetOpenAuth-dd49ae4a865b21f0722efab87a4b0723ec1a6950.zip DotNetOpenAuth-dd49ae4a865b21f0722efab87a4b0723ec1a6950.tar.gz DotNetOpenAuth-dd49ae4a865b21f0722efab87a4b0723ec1a6950.tar.bz2 |
Renamed all the Channel virtual methods that end with Internal to end with Core.
10 files changed, 22 insertions, 22 deletions
diff --git a/src/DotNetOpenAuth.Test/Mocks/CoordinatingChannel.cs b/src/DotNetOpenAuth.Test/Mocks/CoordinatingChannel.cs index 4598a9f..53e0b3c 100644 --- a/src/DotNetOpenAuth.Test/Mocks/CoordinatingChannel.cs +++ b/src/DotNetOpenAuth.Test/Mocks/CoordinatingChannel.cs @@ -148,7 +148,7 @@ namespace DotNetOpenAuth.Test.Mocks { } } - protected override IProtocolMessage RequestInternal(IDirectedProtocolMessage request) { + protected override IProtocolMessage RequestCore(IDirectedProtocolMessage request) { this.ProcessMessageFilter(request, true); // Drop the outgoing message in the other channel's in-slot and let them know it's there. @@ -183,7 +183,7 @@ namespace DotNetOpenAuth.Test.Mocks { return this.SendDirectMessageResponse(message); } - protected override IDirectedProtocolMessage ReadFromRequestInternal(HttpRequestInfo request) { + protected override IDirectedProtocolMessage ReadFromRequestCore(HttpRequestInfo request) { if (request.Message != null) { this.ProcessMessageFilter(request.Message, false); } @@ -191,9 +191,9 @@ namespace DotNetOpenAuth.Test.Mocks { return request.Message; } - protected override IDictionary<string, string> ReadFromResponseInternal(DirectWebResponse response) { + protected override IDictionary<string, string> ReadFromResponseCore(DirectWebResponse response) { Channel_Accessor accessor = Channel_Accessor.AttachShadow(this.wrappedChannel); - return accessor.ReadFromResponseInternal(response); + return accessor.ReadFromResponseCore(response); } protected override void VerifyMessageAfterReceiving(IProtocolMessage message) { diff --git a/src/DotNetOpenAuth.Test/Mocks/CoordinatingOAuthChannel.cs b/src/DotNetOpenAuth.Test/Mocks/CoordinatingOAuthChannel.cs index f00c356..4911a9c 100644 --- a/src/DotNetOpenAuth.Test/Mocks/CoordinatingOAuthChannel.cs +++ b/src/DotNetOpenAuth.Test/Mocks/CoordinatingOAuthChannel.cs @@ -62,7 +62,7 @@ namespace DotNetOpenAuth.Test.Mocks { return new HttpRequestInfo(directedMessage, directedMessage.HttpMethods); } - protected override IProtocolMessage RequestInternal(IDirectedProtocolMessage request) { + protected override IProtocolMessage RequestCore(IDirectedProtocolMessage request) { HttpRequestInfo requestInfo = this.SpoofHttpMethod(request); // Drop the outgoing message in the other channel's in-slot and let them know it's there. this.RemoteChannel.incomingMessage = requestInfo.Message; @@ -82,7 +82,7 @@ namespace DotNetOpenAuth.Test.Mocks { return this.SendDirectMessageResponse(message); } - protected override IDirectedProtocolMessage ReadFromRequestInternal(HttpRequestInfo request) { + protected override IDirectedProtocolMessage ReadFromRequestCore(HttpRequestInfo request) { return request.Message; } diff --git a/src/DotNetOpenAuth.Test/Mocks/TestBadChannel.cs b/src/DotNetOpenAuth.Test/Mocks/TestBadChannel.cs index faa5870..86108fb 100644 --- a/src/DotNetOpenAuth.Test/Mocks/TestBadChannel.cs +++ b/src/DotNetOpenAuth.Test/Mocks/TestBadChannel.cs @@ -37,7 +37,7 @@ namespace DotNetOpenAuth.Test.Mocks { return base.ReadFromRequest(request); } - protected override IDictionary<string, string> ReadFromResponseInternal(DirectWebResponse response) { + protected override IDictionary<string, string> ReadFromResponseCore(DirectWebResponse response) { throw new NotImplementedException(); } diff --git a/src/DotNetOpenAuth.Test/Mocks/TestChannel.cs b/src/DotNetOpenAuth.Test/Mocks/TestChannel.cs index 01dad98..a65b39c 100644 --- a/src/DotNetOpenAuth.Test/Mocks/TestChannel.cs +++ b/src/DotNetOpenAuth.Test/Mocks/TestChannel.cs @@ -19,7 +19,7 @@ namespace DotNetOpenAuth.Test.Mocks { : base(messageTypeProvider, bindingElements) { } - protected override IDictionary<string, string> ReadFromResponseInternal(DirectWebResponse response) { + protected override IDictionary<string, string> ReadFromResponseCore(DirectWebResponse response) { throw new NotImplementedException("ReadFromResponseInternal"); } diff --git a/src/DotNetOpenAuth.Test/OAuth/ChannelElements/OAuthChannelTests.cs b/src/DotNetOpenAuth.Test/OAuth/ChannelElements/OAuthChannelTests.cs index fd901c1..3444192 100644 --- a/src/DotNetOpenAuth.Test/OAuth/ChannelElements/OAuthChannelTests.cs +++ b/src/DotNetOpenAuth.Test/OAuth/ChannelElements/OAuthChannelTests.cs @@ -113,7 +113,7 @@ namespace DotNetOpenAuth.Test.ChannelElements { writer.Flush(); ms.Seek(0, SeekOrigin.Begin); Channel_Accessor channelAccessor = Channel_Accessor.AttachShadow(this.channel); - IDictionary<string, string> deserializedFields = channelAccessor.ReadFromResponseInternal(new CachedDirectWebResponse { CachedResponseStream = ms }); + IDictionary<string, string> deserializedFields = channelAccessor.ReadFromResponseCore(new CachedDirectWebResponse { CachedResponseStream = ms }); Assert.AreEqual(fields.Count, deserializedFields.Count); foreach (string key in fields.Keys) { Assert.AreEqual(fields[key], deserializedFields[key]); diff --git a/src/DotNetOpenAuth.Test/OpenId/ChannelElements/OpenIdChannelTests.cs b/src/DotNetOpenAuth.Test/OpenId/ChannelElements/OpenIdChannelTests.cs index 0897a06..b6a5d29 100644 --- a/src/DotNetOpenAuth.Test/OpenId/ChannelElements/OpenIdChannelTests.cs +++ b/src/DotNetOpenAuth.Test/OpenId/ChannelElements/OpenIdChannelTests.cs @@ -99,7 +99,7 @@ namespace DotNetOpenAuth.Test.OpenId.ChannelElements { var response = new CachedDirectWebResponse { CachedResponseStream = new MemoryStream(KeyValueFormEncoding.GetBytes(fields)), }; - Assert.IsTrue(MessagingUtilities.AreEquivalent(fields, this.accessor.ReadFromResponseInternal(response))); + Assert.IsTrue(MessagingUtilities.AreEquivalent(fields, this.accessor.ReadFromResponseCore(response))); } /// <summary> diff --git a/src/DotNetOpenAuth/Messaging/Channel.cs b/src/DotNetOpenAuth/Messaging/Channel.cs index 5c17485..f9bdb55 100644 --- a/src/DotNetOpenAuth/Messaging/Channel.cs +++ b/src/DotNetOpenAuth/Messaging/Channel.cs @@ -326,7 +326,7 @@ namespace DotNetOpenAuth.Messaging { /// <returns>The deserialized message, if one is found. Null otherwise.</returns> public IDirectedProtocolMessage ReadFromRequest(HttpRequestInfo httpRequest) { Contract.Requires(httpRequest != null); - IDirectedProtocolMessage requestMessage = this.ReadFromRequestInternal(httpRequest); + IDirectedProtocolMessage requestMessage = this.ReadFromRequestCore(httpRequest); if (requestMessage != null) { Logger.DebugFormat("Incoming request received: {0}", requestMessage); this.VerifyMessageAfterReceiving(requestMessage); @@ -370,7 +370,7 @@ namespace DotNetOpenAuth.Messaging { this.PrepareMessageForSending(requestMessage); Logger.DebugFormat("Sending request: {0}", requestMessage); - var responseMessage = this.RequestInternal(requestMessage); + var responseMessage = this.RequestCore(requestMessage); ErrorUtilities.VerifyProtocol(responseMessage != null, MessagingStrings.ExpectedMessageNotReceived, typeof(IProtocolMessage).Name); Logger.DebugFormat("Received message response: {0}", responseMessage); @@ -458,7 +458,7 @@ namespace DotNetOpenAuth.Messaging { /// behavior. However in non-HTTP frameworks, such as unit test mocks, it may be appropriate to override /// this method to eliminate all use of an HTTP transport. /// </remarks> - protected virtual IProtocolMessage RequestInternal(IDirectedProtocolMessage request) { + protected virtual IProtocolMessage RequestCore(IDirectedProtocolMessage request) { Contract.Requires(request != null); HttpWebRequest webRequest = this.CreateHttpRequest(request); IDictionary<string, string> responseFields; @@ -469,7 +469,7 @@ namespace DotNetOpenAuth.Messaging { return null; } - responseFields = this.ReadFromResponseInternal(response); + responseFields = this.ReadFromResponseCore(response); responseMessage = this.MessageFactory.GetNewResponseMessage(request, responseFields); if (responseMessage == null) { @@ -499,7 +499,7 @@ namespace DotNetOpenAuth.Messaging { /// </summary> /// <param name="request">The request to search for an embedded message.</param> /// <returns>The deserialized message, if one is found. Null otherwise.</returns> - protected virtual IDirectedProtocolMessage ReadFromRequestInternal(HttpRequestInfo request) { + protected virtual IDirectedProtocolMessage ReadFromRequestCore(HttpRequestInfo request) { Contract.Requires(request != null); ErrorUtilities.VerifyArgumentNotNull(request, "request"); @@ -641,7 +641,7 @@ namespace DotNetOpenAuth.Messaging { /// <param name="response">The response that is anticipated to contain an protocol message.</param> /// <returns>The deserialized message parts, if found. Null otherwise.</returns> /// <exception cref="ProtocolException">Thrown when the response is not valid.</exception> - protected abstract IDictionary<string, string> ReadFromResponseInternal(DirectWebResponse response); + protected abstract IDictionary<string, string> ReadFromResponseCore(DirectWebResponse response); /// <summary> /// Prepares an HTTP request that carries a given message. @@ -649,7 +649,7 @@ namespace DotNetOpenAuth.Messaging { /// <param name="request">The message to send.</param> /// <returns>The <see cref="HttpWebRequest"/> prepared to send the request.</returns> /// <remarks> - /// This method must be overridden by a derived class, unless the <see cref="RequestInternal"/> method + /// This method must be overridden by a derived class, unless the <see cref="RequestCore"/> method /// is overridden and does not require this method. /// </remarks> protected virtual HttpWebRequest CreateHttpRequest(IDirectedProtocolMessage request) { diff --git a/src/DotNetOpenAuth/Messaging/ChannelContract.cs b/src/DotNetOpenAuth/Messaging/ChannelContract.cs index 6be7d31..d5db0ad 100644 --- a/src/DotNetOpenAuth/Messaging/ChannelContract.cs +++ b/src/DotNetOpenAuth/Messaging/ChannelContract.cs @@ -29,7 +29,7 @@ namespace DotNetOpenAuth.Messaging { /// The deserialized message parts, if found. Null otherwise. /// </returns> /// <exception cref="ProtocolException">Thrown when the response is not valid.</exception> - protected override IDictionary<string, string> ReadFromResponseInternal(DirectWebResponse response) { + protected override IDictionary<string, string> ReadFromResponseCore(DirectWebResponse response) { Contract.Requires(response != null); throw new NotImplementedException(); } diff --git a/src/DotNetOpenAuth/OAuth/ChannelElements/OAuthChannel.cs b/src/DotNetOpenAuth/OAuth/ChannelElements/OAuthChannel.cs index 9211118..89b01c1 100644 --- a/src/DotNetOpenAuth/OAuth/ChannelElements/OAuthChannel.cs +++ b/src/DotNetOpenAuth/OAuth/ChannelElements/OAuthChannel.cs @@ -103,7 +103,7 @@ namespace DotNetOpenAuth.OAuth.ChannelElements { /// </summary> /// <param name="request">The HTTP request to search.</param> /// <returns>A dictionary of data in the request. Should never be null, but may be empty.</returns> - protected override IDirectedProtocolMessage ReadFromRequestInternal(HttpRequestInfo request) { + protected override IDirectedProtocolMessage ReadFromRequestCore(HttpRequestInfo request) { ErrorUtilities.VerifyArgumentNotNull(request, "request"); // First search the Authorization header. Use it exclusively if it's present. @@ -133,7 +133,7 @@ namespace DotNetOpenAuth.OAuth.ChannelElements { } // We didn't find an OAuth authorization header. Revert to other payload methods. - IDirectedProtocolMessage message = base.ReadFromRequestInternal(request); + IDirectedProtocolMessage message = base.ReadFromRequestCore(request); // Add receiving HTTP transport information required for signature generation. var signedMessage = message as ITamperResistantOAuthMessage; @@ -152,7 +152,7 @@ namespace DotNetOpenAuth.OAuth.ChannelElements { /// <returns> /// The deserialized message parts, if found. Null otherwise. /// </returns> - protected override IDictionary<string, string> ReadFromResponseInternal(DirectWebResponse response) { + protected override IDictionary<string, string> ReadFromResponseCore(DirectWebResponse response) { ErrorUtilities.VerifyArgumentNotNull(response, "response"); string body = response.GetResponseReader().ReadToEnd(); diff --git a/src/DotNetOpenAuth/OpenId/ChannelElements/OpenIdChannel.cs b/src/DotNetOpenAuth/OpenId/ChannelElements/OpenIdChannel.cs index b6c45b7..e146898 100644 --- a/src/DotNetOpenAuth/OpenId/ChannelElements/OpenIdChannel.cs +++ b/src/DotNetOpenAuth/OpenId/ChannelElements/OpenIdChannel.cs @@ -186,7 +186,7 @@ namespace DotNetOpenAuth.OpenId.ChannelElements { /// The deserialized message parts, if found. Null otherwise. /// </returns> /// <exception cref="ProtocolException">Thrown when the response is not valid.</exception> - protected override IDictionary<string, string> ReadFromResponseInternal(DirectWebResponse response) { + protected override IDictionary<string, string> ReadFromResponseCore(DirectWebResponse response) { if (response == null) { throw new ArgumentNullException("response"); } |