status-im/consul
2
0
Fork 0
mirror of https://github.com/status-im/consul.git synced 2025-01-22 03:29:43 +00:00
Code Issues Projects Releases Wiki Activity

remove a race condition in sdk testutil server stop (#10342)

Browse Source 
* remove a race condition in sdk testutil server stop

* ensure the process has actually stopped
This commit is contained in:
David Bariod 2021-06-09 17:30:26 +02:00 committed by GitHub
parent f43993316c
commit 797bfb7b51
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sdk/testutil/server.go
View File

@ -342,7 +342,7 @@ func (s *TestServer) Stop() error {
return err
case <-time.After(10 * time.Second):
s.cmd.Process.Signal(syscall.SIGABRT)
s.cmd.Wait()
<-waitDone
return fmt.Errorf("timeout waiting for server to stop gracefully")
}
}