Replace usage of NullLogger with LoggerFactory with console output in AmazonSQSQueryTests

Author: ramonsmits

src/Directory.Packages.props

@@ -27,6 +27,7 @@
     <PackageVersion Include="Microsoft.Extensions.Logging" Version="8.0.1" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Configuration" Version="8.0.1" />
     <PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.3" />
+    <PackageVersion Include="Microsoft.Extensions.Logging.Console" Version="8.0.1" />
     <PackageVersion Include="Microsoft.Extensions.TimeProvider.Testing" Version="8.10.0" />
     <PackageVersion Include="Microsoft.NET.Test.Sdk" Version="18.0.1" />
     <PackageVersion Include="Microsoft-WindowsAPICodePack-Shell" Version="1.1.5" />

src/ServiceControl.Transports.SQS.Tests/AmazonSQSQueryTests.cs

@@ -6,7 +6,7 @@ namespace ServiceControl.Transport.Tests;
 using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
-using Microsoft.Extensions.Logging.Abstractions;
+using Microsoft.Extensions.Logging;
 using Microsoft.Extensions.Time.Testing;
 using NUnit.Framework;
 using Particular.Approvals;
@@ -35,7 +35,9 @@ public void Initialise()
             MaxConcurrency = 1,
             EndpointName = Guid.NewGuid().ToString("N")
         };
-        query = new AmazonSQSQuery(NullLogger<AmazonSQSQuery>.Instance, provider, transportSettings);
+        var loggerFactory = LoggerFactory.Create(builder => builder.AddSimpleConsole().SetMinimumLevel(LogLevel.Trace));
+        var logger = loggerFactory.CreateLogger<AmazonSQSQuery>();
+        query = new AmazonSQSQuery(logger, provider, transportSettings);
     }
 
     [Test]
@@ -125,7 +127,7 @@ public async Task RunScenario()
 
         query.Initialize(dictionary.AsReadOnly());
 
-        DateTime startDate = provider.GetUtcNow().DateTime;
+        var startDate = DateOnly.FromDateTime(provider.GetUtcNow().DateTime);
         provider.Advance(TimeSpan.FromDays(1));
 
         while (!TestContext.CurrentContext.CancellationToken.IsCancellationRequested)
@@ -147,7 +149,7 @@ public async Task RunScenario()
 
             long total = 0L;
 
-            await foreach (QueueThroughput queueThroughput in query.GetThroughputPerDay(queue, DateOnly.FromDateTime(startDate), TestContext.CurrentContext.CancellationToken))
+            await foreach (QueueThroughput queueThroughput in query.GetThroughputPerDay(queue, startDate, TestContext.CurrentContext.CancellationToken))
             {
                 total += queueThroughput.TotalThroughput;
             }

src/ServiceControl.Transports.SQS.Tests/ServiceControl.Transports.SQS.Tests.csproj

@@ -13,6 +13,7 @@
 
   <ItemGroup>
     <PackageReference Include="GitHubActionsTestLogger" />
+    <PackageReference Include="Microsoft.Extensions.Logging.Console" />
     <PackageReference Include="Microsoft.Extensions.TimeProvider.Testing" />
     <PackageReference Include="Microsoft.NET.Test.Sdk" />
     <PackageReference Include="NServiceBus.AcceptanceTesting" />