diff options
author | Jakob Unterwurzacher | 2018-04-07 15:46:41 +0200 |
---|---|---|
committer | Jakob Unterwurzacher | 2018-04-07 15:46:41 +0200 |
commit | 9d7392a5be7e21aaaf2fb02f19b421c29e304632 (patch) | |
tree | 24d2e83a7fdb3887ac2891c5e6bc2efcf63b0e7e | |
parent | 22031d7e531985e9e94d694e74fb00da99de72a5 (diff) |
tests: cli_test: fix fd leak
One fd leak found in TestMountBackground.
-rw-r--r-- | tests/cli/cli_test.go | 8 | ||||
-rw-r--r-- | tests/matrix/matrix_test.go | 4 |
2 files changed, 9 insertions, 3 deletions
diff --git a/tests/cli/cli_test.go b/tests/cli/cli_test.go index 2f27ec7..24bb029 100644 --- a/tests/cli/cli_test.go +++ b/tests/cli/cli_test.go @@ -3,6 +3,7 @@ package cli // Test CLI operations like "-init", "-password" etc import ( + "fmt" "io/ioutil" "os" "os/exec" @@ -20,7 +21,13 @@ var testPw = []byte("test") func TestMain(m *testing.M) { test_helpers.ResetTmpDir(false) + before := test_helpers.ListFds() r := m.Run() + after := test_helpers.ListFds() + if len(before) != len(after) { + fmt.Printf("fd leak? before, after:\n%v\n%v\n", before, after) + os.Exit(1) + } os.Exit(r) } @@ -404,6 +411,7 @@ func TestMountBackground(t *testing.T) { // We should get io.EOF when the child closes stdout // and stderr. if err != nil { + pr.Close() c1 <- struct{}{} return } diff --git a/tests/matrix/matrix_test.go b/tests/matrix/matrix_test.go index bc964ae..f34c167 100644 --- a/tests/matrix/matrix_test.go +++ b/tests/matrix/matrix_test.go @@ -76,9 +76,7 @@ func TestMain(m *testing.M) { r := m.Run() after := test_helpers.ListFds() if len(before) != len(after) { - fmt.Printf("fd leak? before, after:\n") - fmt.Printf("%v\n", before) - fmt.Printf("%v\n", after) + fmt.Printf("fd leak? before, after:\n%v\n%v\n", before, after) os.Exit(1) } test_helpers.UnmountPanic(test_helpers.DefaultPlainDir) |