Skip to content

Commit c122c70

Browse files
danegstaCopilot
andcommitted
Bound Redis argument evaluation test
Use a bounded wait around the Redis argument evaluation regression test so the test fails promptly if endpoint resolution deadlocks again. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent c6bd90e commit c122c70

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

tests/Aspire.Hosting.Redis.Tests/AddRedisTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -850,7 +850,7 @@ public async Task RedisWithCertificateUsesTargetPortsForCommandLineArgs()
850850

851851
await builder.Eventing.PublishAsync(new BeforeStartEvent(app.Services, appModel));
852852

853-
var args = await ArgumentEvaluator.GetArgumentListAsync(redis.Resource, app.Services);
853+
var args = await ArgumentEvaluator.GetArgumentListAsync(redis.Resource, app.Services).AsTask().WaitAsync(TimeSpan.FromSeconds(60));
854854

855855
Assert.Equal("6379", args[args.IndexOf("--tls-port") + 1]);
856856
Assert.Equal("6380", args[args.IndexOf("--port") + 1]);

0 commit comments

Comments
 (0)