refactor unit test to fix race condition (#1446)

spencerugbo · Akshay2191 · commit 9e9d10d3e057 · 2025-12-18T22:10:00.000Z
diff --git a/internal/command/command_service_test.go b/internal/command/command_service_test.go
@@ -116,10 +116,11 @@ func TestCommandService_receiveCallback_configApplyRequest(t *testing.T) {
 		10*time.Millisecond,
 	)
 
+	wg.Wait()
+
 	commandService.configApplyRequestQueueMutex.Lock()
 	defer commandService.configApplyRequestQueueMutex.Unlock()
 	assert.Len(t, commandService.configApplyRequestQueue, 1)
-	wg.Wait()
 }
 
 func TestCommandService_UpdateDataPlaneStatus(t *testing.T) {

Original file line number	Diff line number	Diff line change
`@@ -116,10 +116,11 @@ func TestCommandService_receiveCallback_configApplyRequest(t *testing.T) {`
`116`	`116`	`10*time.Millisecond,`
`117`	`117`	`)`
`118`	`118`
	`119`	`+ wg.Wait()`
	`120`	`+`
`119`	`121`	`commandService.configApplyRequestQueueMutex.Lock()`
`120`	`122`	`defer commandService.configApplyRequestQueueMutex.Unlock()`
`121`	`123`	`assert.Len(t, commandService.configApplyRequestQueue, 1)`
`122`		`- wg.Wait()`
`123`	`124`	`}`
`124`	`125`
`125`	`126`	`func TestCommandService_UpdateDataPlaneStatus(t *testing.T) {`