Refactor BewitTokenExtractionMiddlewareTests; add logging support to middleware instantiation for improved debugging

Author: glucaci

test/Extensions.HotChocolate.Tests/BewitTokenExtractionMiddlewareTests.cs

@@ -1,5 +1,7 @@
 using FluentAssertions;
 using Microsoft.AspNetCore.Http;
+using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.Logging.Abstractions;
 using Microsoft.Extensions.Options;
 using Microsoft.Extensions.Primitives;
 using Xunit;
@@ -32,7 +34,8 @@ public async Task InvokeAsync_WithHeaderToken_ShouldSetContextItem()
 
                 return Task.CompletedTask;
             },
-            CreateOptions());
+            CreateOptions(),
+            NullLogger<BewitTokenExtractionMiddleware>.Instance);
 
         await middleware.InvokeAsync(context);
 
@@ -48,7 +51,8 @@ public async Task InvokeAsync_WithQueryParam_ShouldSetContextItem()
 
         var middleware = new BewitTokenExtractionMiddleware(
             _ => Task.CompletedTask,
-            CreateOptions());
+            CreateOptions(),
+            NullLogger<BewitTokenExtractionMiddleware>.Instance);
 
         await middleware.InvokeAsync(context);
 
@@ -64,7 +68,8 @@ public async Task InvokeAsync_WithHeaderAndQueryParam_ShouldPreferHeader()
 
         var middleware = new BewitTokenExtractionMiddleware(
             _ => Task.CompletedTask,
-            CreateOptions());
+            CreateOptions(),
+            NullLogger<BewitTokenExtractionMiddleware>.Instance);
 
         await middleware.InvokeAsync(context);
 
@@ -78,7 +83,8 @@ public async Task InvokeAsync_WithNoToken_ShouldNotSetContextItem()
 
         var middleware = new BewitTokenExtractionMiddleware(
             _ => Task.CompletedTask,
-            CreateOptions());
+            CreateOptions(),
+            NullLogger<BewitTokenExtractionMiddleware>.Instance);
 
         await middleware.InvokeAsync(context);
 
@@ -94,7 +100,8 @@ public async Task InvokeAsync_WithEmptyHeader_ShouldFallbackToQueryParam()
 
         var middleware = new BewitTokenExtractionMiddleware(
             _ => Task.CompletedTask,
-            CreateOptions());
+            CreateOptions(),
+            NullLogger<BewitTokenExtractionMiddleware>.Instance);
 
         await middleware.InvokeAsync(context);
 
@@ -109,7 +116,8 @@ public async Task InvokeAsync_WithCustomHeaderName_ShouldReadFromCustomHeader()
 
         var middleware = new BewitTokenExtractionMiddleware(
             _ => Task.CompletedTask,
-            CreateOptions(o => o.HeaderName = "X-My-Token"));
+            CreateOptions(o => o.HeaderName = "X-My-Token"),
+            NullLogger<BewitTokenExtractionMiddleware>.Instance);
 
         await middleware.InvokeAsync(context);
 
@@ -124,7 +132,8 @@ public async Task InvokeAsync_WithCustomQueryParamName_ShouldReadFromCustomParam
 
         var middleware = new BewitTokenExtractionMiddleware(
             _ => Task.CompletedTask,
-            CreateOptions(o => o.QueryParamName = "token"));
+            CreateOptions(o => o.QueryParamName = "token"),
+            NullLogger<BewitTokenExtractionMiddleware>.Instance);
 
         await middleware.InvokeAsync(context);
 
@@ -144,7 +153,8 @@ public async Task InvokeAsync_ShouldAlwaysCallNext()
 
                 return Task.CompletedTask;
             },
-            CreateOptions());
+            CreateOptions(),
+            NullLogger<BewitTokenExtractionMiddleware>.Instance);
 
         await middleware.InvokeAsync(context);