Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(194)

Unified Diff: Src/GoogleApis.Tests/Apis/Http/ConfigurableMessageHandlerTest.cs

Issue 12772043: Issues 373 (Execute Bug), 374 (Remove Tests.Utility) and 375 (Clean warnings) (Closed) Base URL: https://google-api-dotnet-client.googlecode.com/hg/
Patch Set: Error in uploading the first time Created 10 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Src/GoogleApis.Tests/Apis/Http/ConfigurableMessageHandlerTest.cs
===================================================================
--- a/Src/GoogleApis.Tests/Apis/Http/ConfigurableMessageHandlerTest.cs
b/Src/GoogleApis.Tests/Apis/Http/ConfigurableMessageHandlerTest.cs
@@ -51,10 51,12 @@
/// <summary> Message handler which returns a new successful (and empty) response. </summary>
private class MockMessageHandler : HttpMessageHandler
{
- protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request,
protected override Task<HttpResponseMessage> SendAsync(HttpRequestMessage request,
CancellationToken cancellationToken)
{
- return new HttpResponseMessage();
TaskCompletionSource<HttpResponseMessage> tcs = new TaskCompletionSource<HttpResponseMessage>();
tcs.SetResult(new HttpResponseMessage());
return tcs.Task;
}
}
@@ -180,11 182,14 @@
const string InjectedHeader = "Some-Header";
const string InjectedValue = "123";
- protected override async Task<HttpResponseMessage> SendAsyncCore(HttpRequestMessage request,
protected override Task<HttpResponseMessage> SendAsyncCore(HttpRequestMessage request,
CancellationToken cancellationToken)
{
Assert.That(request.Headers.GetValues(InjectedHeader).First(), Is.EqualTo(InjectedValue));
- return InjectedResponseMessage;
TaskCompletionSource<HttpResponseMessage> tcs = new TaskCompletionSource<HttpResponseMessage>();
tcs.SetResult(InjectedResponseMessage);
return tcs.Task;
}
/// <summary> A mock interceptor which inject a header to a request. </summary>
@@ -288,7 293,7 @@
/// </summary>
public int CancelRequestNum { get; set; }
- protected override async Task<HttpResponseMessage> SendAsyncCore(HttpRequestMessage request,
protected override Task<HttpResponseMessage> SendAsyncCore(HttpRequestMessage request,
CancellationToken cancellationToken)
{
if (Calls == CancelRequestNum)
@@ -296,7 301,9 @@
CancellationTokenSource.Cancel();
}
- return new HttpResponseMessage { StatusCode = ResponseStatusCode };
TaskCompletionSource<HttpResponseMessage> tcs = new TaskCompletionSource<HttpResponseMessage>();
tcs.SetResult(new HttpResponseMessage { StatusCode = ResponseStatusCode });
return tcs.Task;
}
/// <summary> Unsuccessful response handler which "handles" only service unavailable responses. </summary>
@@ -416,7 423,7 @@
/// </summary>
public const string ExceptionMessage = "Exception from execute";
- protected override async Task<HttpResponseMessage> SendAsyncCore(HttpRequestMessage request,
protected override Task<HttpResponseMessage> SendAsyncCore(HttpRequestMessage request,
CancellationToken cancellationToken)
{
if (ThrowException)
@@ -424,7 431,9 @@
throw Exception;
}
- return new HttpResponseMessage();
TaskCompletionSource<HttpResponseMessage> tcs = new TaskCompletionSource<HttpResponseMessage>();
tcs.SetResult(new HttpResponseMessage());
return tcs.Task;
}
/// <summary> Mock Exception handler which "handles" the exception. </summary>

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b