veyron/services/mgmt/node/impl: Reduce test flakiness under -race.
With -race, some operations take more than 1 second, so remove this
timeout from the test.
Change-Id: Ia22d7ff8e08a030ea473a7e3b9d23033c5329cf0
diff --git a/services/mgmt/node/impl/impl_test.go b/services/mgmt/node/impl/impl_test.go
index 6b473f6..2ca465c 100644
--- a/services/mgmt/node/impl/impl_test.go
+++ b/services/mgmt/node/impl/impl_test.go
@@ -5,7 +5,6 @@
"os"
"syscall"
"testing"
- "time"
"veyron/lib/exec"
"veyron/lib/signals"
@@ -196,9 +195,7 @@
defer os.Remove(idFile)
child := spawnNodeManager(t, mtName, idFile)
defer child.Cleanup()
- if err := child.WaitForLine("ready", time.Second); err != nil {
- t.Fatal("WaitForLine() failed: %v", err)
- }
+ child.Expect("ready")
name := naming.Join(mtName, "nm")
results, err := mt.Resolve(ctx, name)
if err != nil {
@@ -213,7 +210,5 @@
// Invoke the Update method and check that another instance of the
// node manager binary has been started.
invokeUpdate(t, nmAddress)
- if err := child.WaitForLine("ready", time.Second); err != nil {
- t.Fatal("WaitForLine() failed: %v", err)
- }
+ child.Expect("ready")
}
