diff options
| author | Pontus Leitzler <leitzler@users.noreply.github.com> | 2017-10-18 07:38:48 +0200 |
|---|---|---|
| committer | Ben Kochie <superq@gmail.com> | 2017-10-18 07:38:48 +0200 |
| commit | 0b6763886ac9a8dba8a9415487943b7984c2a86c (patch) | |
| tree | 655068bbb3ac36a23e907d24026ec395641f6750 /node_exporter_test.go | |
| parent | 1824ac3b9e5b5c810d88a9f0fb5e0aabfa1727ee (diff) | |
| download | prometheus_node_collector-0b6763886ac9a8dba8a9415487943b7984c2a86c.tar.bz2 prometheus_node_collector-0b6763886ac9a8dba8a9415487943b7984c2a86c.tar.xz prometheus_node_collector-0b6763886ac9a8dba8a9415487943b7984c2a86c.zip | |
Remove unnecessary select statement (#692)
* Remove unnecessary select statement
* Remove unnecessary if-statement
Diffstat (limited to 'node_exporter_test.go')
| -rw-r--r-- | node_exporter_test.go | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/node_exporter_test.go b/node_exporter_test.go index a1a214c..5a9599e 100644 --- a/node_exporter_test.go +++ b/node_exporter_test.go | |||
| @@ -129,14 +129,9 @@ func runCommandAndTests(cmd *exec.Cmd, address string, fn func(pid int) error) e | |||
| 129 | errc <- fn(pid) | 129 | errc <- fn(pid) |
| 130 | }(cmd.Process.Pid) | 130 | }(cmd.Process.Pid) |
| 131 | 131 | ||
| 132 | select { | 132 | err := <-errc |
| 133 | case err := <-errc: | 133 | if cmd.Process != nil { |
| 134 | if cmd.Process != nil { | 134 | cmd.Process.Kill() |
| 135 | cmd.Process.Kill() | ||
| 136 | } | ||
| 137 | if err != nil { | ||
| 138 | return err | ||
| 139 | } | ||
| 140 | } | 135 | } |
| 141 | return nil | 136 | return err |
| 142 | } | 137 | } |