Skip to content

Commit 8db746b

Browse files
Copilotdavidfowl
andcommitted
Fix DeployAsync_WithMultipleComputeEnvironments_Works test - pass fakeContainerRuntime to ConfigureTestServices
- Test was creating fakeContainerRuntime but not passing it to ConfigureTestServices - ConfigureTestServices was creating a new FakeContainerRuntime instance when containerRuntime parameter was null - Test was checking the wrong FakeContainerRuntime instance, causing WasLoginToRegistryCalled assertion to fail - Fixed by passing containerRuntime: fakeContainerRuntime parameter to ConfigureTestServices call - Test now passes successfully Co-authored-by: davidfowl <[email protected]>
1 parent 64c1053 commit 8db746b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

tests/Aspire.Hosting.Azure.Tests/AzureDeployerTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -395,7 +395,7 @@ public async Task DeployAsync_WithMultipleComputeEnvironments_Works(string step)
395395
_ => []
396396
};
397397
});
398-
ConfigureTestServices(builder, armClientProvider: armClientProvider, processRunner: mockProcessRunner, activityReporter: mockActivityReporter);
398+
ConfigureTestServices(builder, armClientProvider: armClientProvider, processRunner: mockProcessRunner, activityReporter: mockActivityReporter, containerRuntime: fakeContainerRuntime);
399399

400400
var acaEnv = builder.AddAzureContainerAppEnvironment("aca-env");
401401
var aasEnv = builder.AddAzureAppServiceEnvironment("aas-env");

0 commit comments

Comments
 (0)