Unverified Commit 5bcbbd4c authored by Lincoln Nogueira's avatar Lincoln Nogueira Committed by GitHub

chore: fix store tests on Windows (#2769)

It's just a matter of explicitly closing the database, so that TempDir.removeAll doesn't fail.
parent ff13d977
...@@ -30,4 +30,5 @@ func TestActivityStore(t *testing.T) { ...@@ -30,4 +30,5 @@ func TestActivityStore(t *testing.T) {
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 1, len(activities)) require.Equal(t, 1, len(activities))
require.Equal(t, activity, activities[0]) require.Equal(t, activity, activities[0])
ts.Close()
} }
...@@ -53,4 +53,5 @@ func TestIdentityProviderStore(t *testing.T) { ...@@ -53,4 +53,5 @@ func TestIdentityProviderStore(t *testing.T) {
idpList, err := ts.ListIdentityProviders(ctx, &store.FindIdentityProvider{}) idpList, err := ts.ListIdentityProviders(ctx, &store.FindIdentityProvider{})
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 0, len(idpList)) require.Equal(t, 0, len(idpList))
ts.Close()
} }
...@@ -50,4 +50,5 @@ func TestInboxStore(t *testing.T) { ...@@ -50,4 +50,5 @@ func TestInboxStore(t *testing.T) {
}) })
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 0, len(inboxes)) require.Equal(t, 0, len(inboxes))
ts.Close()
} }
...@@ -59,4 +59,5 @@ func TestMemoOrganizerStore(t *testing.T) { ...@@ -59,4 +59,5 @@ func TestMemoOrganizerStore(t *testing.T) {
}) })
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 0, len(memoOrganizers)) require.Equal(t, 0, len(memoOrganizers))
ts.Close()
} }
...@@ -55,4 +55,5 @@ func TestMemoRelationStore(t *testing.T) { ...@@ -55,4 +55,5 @@ func TestMemoRelationStore(t *testing.T) {
} }
_, err = ts.UpsertMemoRelation(ctx, commentRelation) _, err = ts.UpsertMemoRelation(ctx, commentRelation)
require.NoError(t, err) require.NoError(t, err)
ts.Close()
} }
...@@ -58,6 +58,7 @@ func TestMemoStore(t *testing.T) { ...@@ -58,6 +58,7 @@ func TestMemoStore(t *testing.T) {
}) })
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 0, len(memoList)) require.Equal(t, 0, len(memoList))
ts.Close()
} }
func TestDeleteMemoStore(t *testing.T) { func TestDeleteMemoStore(t *testing.T) {
...@@ -77,4 +78,5 @@ func TestDeleteMemoStore(t *testing.T) { ...@@ -77,4 +78,5 @@ func TestDeleteMemoStore(t *testing.T) {
ID: memo.ID, ID: memo.ID,
}) })
require.NoError(t, err) require.NoError(t, err)
ts.Close()
} }
...@@ -59,4 +59,5 @@ func TestResourceStore(t *testing.T) { ...@@ -59,4 +59,5 @@ func TestResourceStore(t *testing.T) {
ID: 2, ID: 2,
}) })
require.NoError(t, err) require.NoError(t, err)
ts.Close()
} }
...@@ -36,4 +36,5 @@ func TestStorageStore(t *testing.T) { ...@@ -36,4 +36,5 @@ func TestStorageStore(t *testing.T) {
storageList, err = ts.ListStorages(ctx, &store.FindStorage{}) storageList, err = ts.ListStorages(ctx, &store.FindStorage{})
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 0, len(storageList)) require.Equal(t, 0, len(storageList))
ts.Close()
} }
...@@ -36,4 +36,5 @@ func TestSystemSettingStore(t *testing.T) { ...@@ -36,4 +36,5 @@ func TestSystemSettingStore(t *testing.T) {
list, err := ts.ListSystemSettings(ctx, &store.FindSystemSetting{}) list, err := ts.ListSystemSettings(ctx, &store.FindSystemSetting{})
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 4, len(list)) require.Equal(t, 4, len(list))
ts.Close()
} }
...@@ -35,4 +35,5 @@ func TestTagStore(t *testing.T) { ...@@ -35,4 +35,5 @@ func TestTagStore(t *testing.T) {
tags, err = ts.ListTags(ctx, &store.FindTag{}) tags, err = ts.ListTags(ctx, &store.FindTag{})
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 0, len(tags)) require.Equal(t, 0, len(tags))
ts.Close()
} }
...@@ -24,4 +24,5 @@ func TestUserSettingStore(t *testing.T) { ...@@ -24,4 +24,5 @@ func TestUserSettingStore(t *testing.T) {
list, err := ts.ListUserSettings(ctx, &store.FindUserSetting{}) list, err := ts.ListUserSettings(ctx, &store.FindUserSetting{})
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 1, len(list)) require.Equal(t, 1, len(list))
ts.Close()
} }
...@@ -35,6 +35,7 @@ func TestUserStore(t *testing.T) { ...@@ -35,6 +35,7 @@ func TestUserStore(t *testing.T) {
users, err = ts.ListUsers(ctx, &store.FindUser{}) users, err = ts.ListUsers(ctx, &store.FindUser{})
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 0, len(users)) require.Equal(t, 0, len(users))
ts.Close()
} }
func createTestingHostUser(ctx context.Context, ts *store.Store) (*store.User, error) { func createTestingHostUser(ctx context.Context, ts *store.Store) (*store.User, error) {
......
...@@ -47,4 +47,5 @@ func TestWebhookStore(t *testing.T) { ...@@ -47,4 +47,5 @@ func TestWebhookStore(t *testing.T) {
}) })
require.NoError(t, err) require.NoError(t, err)
require.Equal(t, 0, len(webhooks)) require.Equal(t, 0, len(webhooks))
ts.Close()
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment