Skip to content

Commit 49a33cb

Browse files
committed
Use sync Send overload
1 parent 8350448 commit 49a33cb

File tree

5 files changed

+11
-15
lines changed

5 files changed

+11
-15
lines changed

src/libraries/Common/tests/System/Net/Http/HttpClientHandlerTest.AutoRedirect.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -184,7 +184,7 @@ await LoopbackServer.CreateServerAsync(async (origServer, origUrl) =>
184184
{
185185
await LoopbackServer.CreateServerAsync(async (redirectServer, redirectUrl) =>
186186
{
187-
Task<HttpResponseMessage> getResponseTask = client.GetAsync(origUrl);
187+
Task<HttpResponseMessage> getResponseTask = client.GetAsync(TestAsync, origUrl);
188188

189189
Task redirectTask = redirectServer.AcceptConnectionSendResponseAndCloseAsync();
190190

src/libraries/Common/tests/System/Net/Http/HttpClientHandlerTestBase.cs

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -261,5 +261,10 @@ public static Task<byte[]> GetByteArrayAsync(this HttpClient client, bool async,
261261
return client.GetByteArrayAsync(uri);
262262
#endif
263263
}
264+
265+
public static Task<HttpResponseMessage> GetAsync(this HttpClient client, bool async, Uri uri, HttpCompletionOption completionOption = default, CancellationToken cancellationToken = default)
266+
{
267+
return SendAsync(client, async, new HttpRequestMessage(HttpMethod.Get, uri), completionOption, cancellationToken);
268+
}
264269
}
265270
}

src/libraries/Common/tests/System/Net/Http/HttpProtocolTests.cs

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -266,10 +266,7 @@ await LoopbackServer.CreateServerAsync(async (server, url) =>
266266
{
267267
using (HttpClient client = CreateHttpClient())
268268
{
269-
HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, url);
270-
request.Version = HttpVersion.Version11;
271-
272-
Task<HttpResponseMessage> getResponseTask = client.SendAsync(TestAsync, request);
269+
Task<HttpResponseMessage> getResponseTask = client.GetAsync(TestAsync, url);
273270

274271
await TestHelper.WhenAllCompletedOrAnyFailed(
275272
getResponseTask,
@@ -360,10 +357,7 @@ await LoopbackServer.CreateServerAsync(async (server, url) =>
360357
Task ignoredServerTask = server.AcceptConnectionSendCustomResponseAndCloseAsync(
361358
responseString + "\r\nConnection: close\r\nContent-Length: 0\r\n\r\n");
362359

363-
HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, url);
364-
request.Version = HttpVersion.Version11;
365-
366-
await Assert.ThrowsAsync<HttpRequestException>(() => client.SendAsync(TestAsync, request));
360+
await Assert.ThrowsAsync<HttpRequestException>(() => client.GetAsync(TestAsync, url));
367361
}
368362
}, new LoopbackServer.Options { StreamWrapper = GetStream });
369363
}
@@ -459,10 +453,7 @@ await LoopbackServer.CreateClientAndServerAsync(async url =>
459453
{
460454
using HttpClient client = CreateHttpClient();
461455

462-
HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Get, url);
463-
request.Version = HttpVersion.Version11;
464-
465-
using HttpResponseMessage response = await client.SendAsync(TestAsync, request)
456+
using HttpResponseMessage response = await client.GetAsync(TestAsync, url)
466457
.WaitAsync(TestHelper.PassingTestTimeout);
467458

468459
Assert.Equal(200, (int)response.StatusCode);

src/libraries/Common/tests/System/Net/Http/IdnaProtocolTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ await LoopbackServer.CreateServerAsync(async (server, serverUrl) =>
4040

4141
using (HttpClient client = CreateHttpClient(handler))
4242
{
43-
Task<HttpResponseMessage> getResponseTask = client.GetAsync(uri);
43+
Task<HttpResponseMessage> getResponseTask = client.GetAsync(TestAsync, uri);
4444
Task<List<string>> serverTask = server.AcceptConnectionSendResponseAndCloseAsync();
4545

4646
await TestHelper.WhenAllCompletedOrAnyFailed(getResponseTask, serverTask);

src/libraries/System.Net.Http/tests/FunctionalTests/MetricsTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -719,7 +719,7 @@ protected override void Dispose(bool disposing)
719719

720720
protected Task<HttpResponseMessage> SendAsync(HttpMessageInvoker invoker, HttpRequestMessage request, CancellationToken cancellationToken = default) =>
721721
TestHttpMessageInvoker ?
722-
invoker.SendAsync(request, cancellationToken) :
722+
(TestAsync ? invoker.SendAsync(request, cancellationToken) : Task.Run(() => invoker.Send(request, cancellationToken))) :
723723
((HttpClient)invoker).SendAsync(TestAsync, request, cancellationToken);
724724

725725
protected HttpMessageInvoker CreateHttpMessageInvoker(HttpMessageHandler? handler = null) =>

0 commit comments

Comments
 (0)