From 61a40e50ed0b86f3c5a73a7e08f276937fcb7edf Mon Sep 17 00:00:00 2001 From: Safia Abdalla Date: Tue, 6 May 2025 17:43:33 -0700 Subject: [PATCH 1/2] Update AzureUserAssignedIdentityTests.cs --- .../AzureUserAssignedIdentityTests.cs | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/tests/Aspire.Hosting.Azure.Tests/AzureUserAssignedIdentityTests.cs b/tests/Aspire.Hosting.Azure.Tests/AzureUserAssignedIdentityTests.cs index 3e4a2967b7d..aa2f80ec026 100644 --- a/tests/Aspire.Hosting.Azure.Tests/AzureUserAssignedIdentityTests.cs +++ b/tests/Aspire.Hosting.Azure.Tests/AzureUserAssignedIdentityTests.cs @@ -32,8 +32,7 @@ public async Task AddAzureUserAssignedIdentity_GeneratesExpectedResourcesAndBice var (_, bicep) = await GetManifestWithBicep(resource); await Verifier.Verify(bicep, extension: "bicep") - .UseHelixAwareDirectory("Snapshots") - .AutoVerify(); + .UseHelixAwareDirectory("Snapshots"); } [Fact] @@ -52,8 +51,7 @@ public async Task AddAzureUserAssignedIdentity_PublishAsExisting_Works() var (_, bicep) = await GetManifestWithBicep(resource); await Verifier.Verify(bicep, extension: "bicep") - .UseHelixAwareDirectory("Snapshots") - .AutoVerify(); + .UseHelixAwareDirectory("Snapshots"); } [Fact] @@ -96,7 +94,6 @@ public async Task AddAzureUserAssignedIdentity_WithRoleAssignments_Works() new Target("bicep", identityRoleAssignmentsBicep) ]; await Verifier.Verify(targets) - .UseHelixAwareDirectory("Snapshots") - .AutoVerify(); + .UseHelixAwareDirectory("Snapshots"); } } From 36b64ab7f6924ed821405d5a48f5bacc9b18b916 Mon Sep 17 00:00:00 2001 From: David Fowler Date: Tue, 6 May 2025 22:34:18 -0700 Subject: [PATCH 2/2] Apply suggestions from code review --- .../AzureUserAssignedIdentityTests.cs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/Aspire.Hosting.Azure.Tests/AzureUserAssignedIdentityTests.cs b/tests/Aspire.Hosting.Azure.Tests/AzureUserAssignedIdentityTests.cs index aa2f80ec026..3a52386bad5 100644 --- a/tests/Aspire.Hosting.Azure.Tests/AzureUserAssignedIdentityTests.cs +++ b/tests/Aspire.Hosting.Azure.Tests/AzureUserAssignedIdentityTests.cs @@ -32,7 +32,7 @@ public async Task AddAzureUserAssignedIdentity_GeneratesExpectedResourcesAndBice var (_, bicep) = await GetManifestWithBicep(resource); await Verifier.Verify(bicep, extension: "bicep") - .UseHelixAwareDirectory("Snapshots"); + .UseHelixAwareDirectory(); } [Fact] @@ -51,7 +51,7 @@ public async Task AddAzureUserAssignedIdentity_PublishAsExisting_Works() var (_, bicep) = await GetManifestWithBicep(resource); await Verifier.Verify(bicep, extension: "bicep") - .UseHelixAwareDirectory("Snapshots"); + .UseHelixAwareDirectory(); } [Fact] @@ -94,6 +94,6 @@ public async Task AddAzureUserAssignedIdentity_WithRoleAssignments_Works() new Target("bicep", identityRoleAssignmentsBicep) ]; await Verifier.Verify(targets) - .UseHelixAwareDirectory("Snapshots"); + .UseHelixAwareDirectory(); } }