From da9b6b4c812dae049a038aa38aa9032c0b384c3a Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 21 Jul 2025 14:40:22 +0000 Subject: [PATCH 1/6] Initial plan From 587f68eac5f0f9cd4376b492df0fcc80f3e3b1da Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 21 Jul 2025 14:46:53 +0000 Subject: [PATCH 2/6] Update Azure packages and resolve security vulnerability Co-authored-by: jasonshave <28457536+jasonshave@users.noreply.github.com> --- .../AzureStorage.QueueService.csproj | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/AzureStorage.QueueService/AzureStorage.QueueService.csproj b/src/AzureStorage.QueueService/AzureStorage.QueueService.csproj index ed89303..c6cd144 100644 --- a/src/AzureStorage.QueueService/AzureStorage.QueueService.csproj +++ b/src/AzureStorage.QueueService/AzureStorage.QueueService.csproj @@ -24,13 +24,13 @@ - - - - - - - + + + + + + + From a326f91af82aea45bde71be0899216eef40a3686 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 21 Jul 2025 14:50:46 +0000 Subject: [PATCH 3/6] Update test framework packages to latest versions Co-authored-by: jasonshave <28457536+jasonshave@users.noreply.github.com> --- .../AzureStorage.QueueClient.IntegrationTests.csproj | 12 ++++++------ .../AzureStorage.QueueService.Tests.csproj | 12 ++++++------ 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/AzureStorage.QueueClient.IntegrationTests/AzureStorage.QueueClient.IntegrationTests.csproj b/tests/AzureStorage.QueueClient.IntegrationTests/AzureStorage.QueueClient.IntegrationTests.csproj index 590ca5b..d92f80d 100644 --- a/tests/AzureStorage.QueueClient.IntegrationTests/AzureStorage.QueueClient.IntegrationTests.csproj +++ b/tests/AzureStorage.QueueClient.IntegrationTests/AzureStorage.QueueClient.IntegrationTests.csproj @@ -10,12 +10,12 @@ - - - - - - + + + + + + diff --git a/tests/AzureStorage.QueueService.Tests/AzureStorage.QueueService.Tests.csproj b/tests/AzureStorage.QueueService.Tests/AzureStorage.QueueService.Tests.csproj index 3848130..1bcfc8b 100644 --- a/tests/AzureStorage.QueueService.Tests/AzureStorage.QueueService.Tests.csproj +++ b/tests/AzureStorage.QueueService.Tests/AzureStorage.QueueService.Tests.csproj @@ -11,16 +11,16 @@ - - - + + + - - + + runtime; build; native; contentfiles; analyzers; buildtransitive all - + runtime; build; native; contentfiles; analyzers; buildtransitive all From 9af43d6f6bb0f026e6743e2f02a78da337f545c9 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 21 Jul 2025 14:53:18 +0000 Subject: [PATCH 4/6] Final NuGet package updates to latest compatible versions Co-authored-by: jasonshave <28457536+jasonshave@users.noreply.github.com> --- .../AzureStorage.QueueService.csproj | 8 ++++---- .../AzureStorage.QueueService.Tests.csproj | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/AzureStorage.QueueService/AzureStorage.QueueService.csproj b/src/AzureStorage.QueueService/AzureStorage.QueueService.csproj index c6cd144..157259b 100644 --- a/src/AzureStorage.QueueService/AzureStorage.QueueService.csproj +++ b/src/AzureStorage.QueueService/AzureStorage.QueueService.csproj @@ -26,10 +26,10 @@ - - - - + + + + diff --git a/tests/AzureStorage.QueueService.Tests/AzureStorage.QueueService.Tests.csproj b/tests/AzureStorage.QueueService.Tests/AzureStorage.QueueService.Tests.csproj index 1bcfc8b..d864feb 100644 --- a/tests/AzureStorage.QueueService.Tests/AzureStorage.QueueService.Tests.csproj +++ b/tests/AzureStorage.QueueService.Tests/AzureStorage.QueueService.Tests.csproj @@ -12,7 +12,7 @@ - + From 17a919bff6f11dd6128249f7a476fbf69d3e4970 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 21 Jul 2025 15:10:51 +0000 Subject: [PATCH 5/6] Fix async/await warnings in test server endpoints Co-authored-by: jasonshave <28457536+jasonshave@users.noreply.github.com> --- tests/AzureStorage.QueueService.Tests.Server/Endpoints.cs | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/AzureStorage.QueueService.Tests.Server/Endpoints.cs b/tests/AzureStorage.QueueService.Tests.Server/Endpoints.cs index 56503c3..5702dc6 100644 --- a/tests/AzureStorage.QueueService.Tests.Server/Endpoints.cs +++ b/tests/AzureStorage.QueueService.Tests.Server/Endpoints.cs @@ -21,13 +21,15 @@ public static async ValueTask, BadRequest>> SendMessage(Perso Person? response = null; await client.ReceiveMessagesAsync(HandleMessage, HandleException, 10); - async ValueTask HandleMessage(Person? message, IDictionary? metadata) + ValueTask HandleMessage(Person? message, IDictionary? metadata) { response = message; + return ValueTask.CompletedTask; } - async ValueTask HandleException(Exception ex, IDictionary? metadata) + ValueTask HandleException(Exception ex, IDictionary? metadata) { + return ValueTask.CompletedTask; } if (response is not null) From c6adfec14212c07a1eca9174176d156972901709 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 21 Jul 2025 15:23:27 +0000 Subject: [PATCH 6/6] Fix all remaining test warnings for nullable properties and null references Co-authored-by: jasonshave <28457536+jasonshave@users.noreply.github.com> --- .../TestWebApplicationFactory.cs | 2 +- tests/AzureStorage.QueueService.Tests.Server/Person.cs | 2 +- .../AzureStorageQueueServiceTests.cs | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/AzureStorage.QueueClient.IntegrationTests/TestWebApplicationFactory.cs b/tests/AzureStorage.QueueClient.IntegrationTests/TestWebApplicationFactory.cs index fd7199b..2a4a4f7 100644 --- a/tests/AzureStorage.QueueClient.IntegrationTests/TestWebApplicationFactory.cs +++ b/tests/AzureStorage.QueueClient.IntegrationTests/TestWebApplicationFactory.cs @@ -9,7 +9,7 @@ namespace AzureStorage.QueueClient.IntegrationTests; public class TestWebApplicationFactory : WebApplicationFactory where TProgram : class { - public ITestOutputHelper TestOutputHelper { get; set; } + public ITestOutputHelper TestOutputHelper { get; set; } = null!; protected override IHost CreateHost(IHostBuilder builder) { diff --git a/tests/AzureStorage.QueueService.Tests.Server/Person.cs b/tests/AzureStorage.QueueService.Tests.Server/Person.cs index d4e904b..52d9ff0 100644 --- a/tests/AzureStorage.QueueService.Tests.Server/Person.cs +++ b/tests/AzureStorage.QueueService.Tests.Server/Person.cs @@ -2,5 +2,5 @@ public class Person { - public string FirstName { get; set; } + public string FirstName { get; set; } = string.Empty; } \ No newline at end of file diff --git a/tests/AzureStorage.QueueService.Tests/AzureStorageQueueServiceTests.cs b/tests/AzureStorage.QueueService.Tests/AzureStorageQueueServiceTests.cs index 8c1fa62..3150de4 100644 --- a/tests/AzureStorage.QueueService.Tests/AzureStorageQueueServiceTests.cs +++ b/tests/AzureStorage.QueueService.Tests/AzureStorageQueueServiceTests.cs @@ -107,7 +107,7 @@ public async Task Receive_Messages_Throws_Collection() QueueMessage[] peekedMessages = { queueMessage }; var response = Response.FromValue(peekedMessages, mockResponse); - var mockQueueClient = new Mock(_queueClientSettings.ConnectionString, _queueClientSettings.QueueName); + var mockQueueClient = new Mock(_queueClientSettings.ConnectionString!, _queueClientSettings.QueueName!); mockQueueClient.Setup(x => x.ReceiveMessagesAsync(1, null, CancellationToken.None)).ReturnsAsync(response); mockQueueClient.Setup(x => x.DeleteMessageAsync(It.IsAny(), It.IsAny(), CancellationToken.None)); @@ -135,7 +135,7 @@ public async Task Can_Send_Message() var fixture = new Fixture(); var testObject = fixture.Create(); - var mockQueueClient = new Mock(_queueClientSettings.ConnectionString, _queueClientSettings.QueueName); + var mockQueueClient = new Mock(_queueClientSettings.ConnectionString!, _queueClientSettings.QueueName!); var mockResponse = Mock.Of(); var mockMessageConverter = new Mock();