Resource management

Author: Bart Koelman

test/JsonApiDotNetCoreTests/IntegrationTests/NonJsonApiControllers/NonJsonApiControllerTests.cs

@@ -23,10 +23,10 @@ public async Task Get_skips_middleware_and_formatters()
         // Arrange
         using var request = new HttpRequestMessage(HttpMethod.Get, "/NonJsonApi");
 
-        HttpClient client = _testContext.Factory.CreateClient();
+        using HttpClient client = _testContext.Factory.CreateClient();
 
         // Act
-        HttpResponseMessage httpResponse = await client.SendAsync(request);
+        using HttpResponseMessage httpResponse = await client.SendAsync(request);
 
         // Assert
         httpResponse.Should().HaveStatusCode(HttpStatusCode.OK);
@@ -52,10 +52,10 @@ public async Task Post_skips_middleware_and_formatters()
             }
         };
 
-        HttpClient client = _testContext.Factory.CreateClient();
+        using HttpClient client = _testContext.Factory.CreateClient();
 
         // Act
-        HttpResponseMessage httpResponse = await client.SendAsync(request);
+        using HttpResponseMessage httpResponse = await client.SendAsync(request);
 
         // Assert
         httpResponse.Should().HaveStatusCode(HttpStatusCode.OK);
@@ -72,10 +72,10 @@ public async Task Post_skips_error_handler()
         // Arrange
         using var request = new HttpRequestMessage(HttpMethod.Post, "/NonJsonApi");
 
-        HttpClient client = _testContext.Factory.CreateClient();
+        using HttpClient client = _testContext.Factory.CreateClient();
 
         // Act
-        HttpResponseMessage httpResponse = await client.SendAsync(request);
+        using HttpResponseMessage httpResponse = await client.SendAsync(request);
 
         // Assert
         httpResponse.Should().HaveStatusCode(HttpStatusCode.BadRequest);
@@ -101,10 +101,10 @@ public async Task Put_skips_middleware_and_formatters()
             }
         };
 
-        HttpClient client = _testContext.Factory.CreateClient();
+        using HttpClient client = _testContext.Factory.CreateClient();
 
         // Act
-        HttpResponseMessage httpResponse = await client.SendAsync(request);
+        using HttpResponseMessage httpResponse = await client.SendAsync(request);
 
         // Assert
         httpResponse.Should().HaveStatusCode(HttpStatusCode.OK);
@@ -121,10 +121,10 @@ public async Task Patch_skips_middleware_and_formatters()
         // Arrange
         using var request = new HttpRequestMessage(HttpMethod.Patch, "/NonJsonApi?name=Janice");
 
-        HttpClient client = _testContext.Factory.CreateClient();
+        using HttpClient client = _testContext.Factory.CreateClient();
 
         // Act
-        HttpResponseMessage httpResponse = await client.SendAsync(request);
+        using HttpResponseMessage httpResponse = await client.SendAsync(request);
 
         // Assert
         httpResponse.Should().HaveStatusCode(HttpStatusCode.OK);
@@ -141,10 +141,10 @@ public async Task Delete_skips_middleware_and_formatters()
         // Arrange
         using var request = new HttpRequestMessage(HttpMethod.Delete, "/NonJsonApi");
 
-        HttpClient client = _testContext.Factory.CreateClient();
+        using HttpClient client = _testContext.Factory.CreateClient();
 
         // Act
-        HttpResponseMessage httpResponse = await client.SendAsync(request);
+        using HttpResponseMessage httpResponse = await client.SendAsync(request);
 
         // Assert
         httpResponse.Should().HaveStatusCode(HttpStatusCode.OK);