Review comments

This commit is contained in:
Kegan Dougal 2017-05-09 15:40:48 +01:00
parent 2dd056f73e
commit 107483f0a2

View file

@ -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")