From 55f19283f487bad8c8e1989ee98919e65583980b Mon Sep 17 00:00:00 2001 From: jochen Date: Sat, 24 Oct 2026 16:42:28 +0200 Subject: [PATCH] Do not wait twice at the cleanup for an assigner that ended on its own, which made the refused-start test take ten seconds a bucket --- .../cmd/mesh-delivery-assigner/integration_test.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/modules/mesh-delivery-assigner/cmd/mesh-delivery-assigner/integration_test.go b/modules/mesh-delivery-assigner/cmd/mesh-delivery-assigner/integration_test.go index 93dc6d1..2f732fd 100644 --- a/modules/mesh-delivery-assigner/cmd/mesh-delivery-assigner/integration_test.go +++ b/modules/mesh-delivery-assigner/cmd/mesh-delivery-assigner/integration_test.go @@ -104,7 +104,10 @@ func (b *bed) startStoppable(t *testing.T) (<-chan error, func()) { env := map[string]string{"MESH_BROKER_FILE": broker} ctx, stop := context.WithCancel(b.ctx) done := make(chan error, 1) - go func() { done <- run(ctx, func(k string) string { return env[k] }, io.Discard) }() + go func() { + done <- run(ctx, func(k string) string { return env[k] }, io.Discard) + close(done) // a program that ended on its own (a refused start) is not waited for again at the cleanup + }() stopped := func() { stop() select {