packer/rpc: get rid of data races in tests

2013-08-21 10:21:32 -07:00 · 2013-08-21 10:21:32 -07:00 · a66085444c
parent 6f742cb332
commit a66085444c
1 changed files with 4 additions and 1 deletions
--- a/packer/rpc/communicator_test.go
+++ b/packer/rpc/communicator_test.go
@ -99,7 +99,10 @@ func TestCommunicatorRPC(t *testing.T) {
 	c.startCmd.SetExited(42)

 	for i := 0; i < 5; i++ {
-		if cmd.Exited {
+		cmd.Lock()
+		exited := cmd.Exited
+		cmd.Unlock()
+		if exited {
 			assert.Equal(cmd.ExitStatus, 42, "should have proper exit status")
 			break
 		}