From 107483f0a24d31889a7b2706ff64aa206ff196b6 Mon Sep 17 00:00:00 2001 From: Kegan Dougal Date: Tue, 9 May 2017 15:40:48 +0100 Subject: [PATCH] Review comments --- .../dendrite/cmd/syncserver-integration-tests/main.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/github.com/matrix-org/dendrite/cmd/syncserver-integration-tests/main.go b/src/github.com/matrix-org/dendrite/cmd/syncserver-integration-tests/main.go index 6964609f5..7e5ae648f 100644 --- a/src/github.com/matrix-org/dendrite/cmd/syncserver-integration-tests/main.go +++ b/src/github.com/matrix-org/dendrite/cmd/syncserver-integration-tests/main.go @@ -237,9 +237,9 @@ func testSyncServer(input, want []string, since string) { panic(err) } panic("dendrite-sync-api-server timed out") - case err := <-done: + case err, open := <-done: cmd.Process.Kill() // ensure server is dead, only cleaning up so don't care about errors this returns. - if err != nil { + if open { // channel is closed on success fmt.Println("=============================================================================================") fmt.Println("sync server failed to run. If failing with 'pq: password authentication failed for user' try:") fmt.Println(" export PGHOST=/var/run/postgresql\n")