tests: Unify test setup

There's been some proliferation in our test's main functions. Naturally
enough, this causes some of them to not work correctly, e.g. because
they use `defer` in the `TestMain` function, which is not getting
executed if `os.Exit` is called.

Unify existing setup functions to follow the same pattern, which is that
`TestMain()` calls out to `testMain()`.

1 files changed, 6 insertions, 1 deletions

diff --git a/internal/git2go/commit_test.go b/internal/git2go/commit_test.go
index 6a1eea7a0..7bba3aa31 100644
--- a/internal/git2go/commit_test.go
+++ b/internal/git2go/commit_test.go
@@ -19,9 +19,14 @@ import (
 )
 
 func TestMain(m *testing.M) {
+	os.Exit(testMain(m))
+}
+
+func testMain(m *testing.M) int {
+	defer testhelper.MustHaveNoChildProcess()
 	testhelper.Configure()
 	testhelper.ConfigureGitalyGit2Go()
-	os.Exit(m.Run())
+	return m.Run()
 }
 
 type commit struct {