test: acquire lock before checking pendingRequests in controller test (…

…#42096) (#42287) fix race condition in TestQueueProducerBlocksUntilOutputIsSet (cherry picked from commit 57eae3a) Co-authored-by: kruskall <[email protected]>
elastic · Jan 15, 2025 · 5a999a7 · 5a999a7
1 parent 85785cb
commit 5a999a7
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/libbeat/publisher/pipeline/controller_test.go b/libbeat/publisher/pipeline/controller_test.go
@@ -231,6 +231,8 @@ func TestQueueProducerBlocksUntilOutputIsSet(t *testing.T) {
 		}()
 	}
 	allStarted := waitUntilTrue(time.Second, func() bool {
+		controller.queueLock.Lock()
+		defer controller.queueLock.Unlock()
 		return len(controller.pendingRequests) == producerCount
 	})
 	assert.True(t, allStarted, "All queueProducer requests should be saved as pending requests by outputController")