test: close test SQL store before deleting its tmp dir in tests (#22421)

pull/22422/head
Daniel Moran 2021-09-08 11:12:21 -04:00 committed by GitHub
parent a7cd53a732
commit eb5f5c3c5b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 11 deletions

View File

@ -5,24 +5,20 @@ import (
"os"
"testing"
"github.com/stretchr/testify/require"
"go.uber.org/zap"
)
func NewTestStore(t *testing.T) (*SqlStore, func(t *testing.T)) {
tempDir, err := ioutil.TempDir("", "")
if err != nil {
t.Fatalf("unable to create temporary test directory %v", err)
}
cleanUpFn := func(t *testing.T) {
if err := os.RemoveAll(tempDir); err != nil {
t.Fatalf("unable to delete temporary test directory %s: %v", tempDir, err)
}
}
require.NoError(t, err, "unable to create temporary test directory")
s, err := NewSqlStore(tempDir+"/"+DefaultFilename, zap.NewNop())
if err != nil {
t.Fatal("unable to open testing database")
require.NoError(t, err, "unable to open testing database")
cleanUpFn := func(t *testing.T) {
require.NoError(t, s.Close(), "failed to close testing database")
require.NoErrorf(t, os.RemoveAll(tempDir), "unable to delete temporary test directory %s", tempDir)
}
return s, cleanUpFn