SQL/Storage: Remove SkipDataMigration flag (#99404)
This commit is contained in:
@@ -62,10 +62,7 @@ func setupBackendTest(t *testing.T) (testBackend, context.Context) {
|
||||
|
||||
ctx := testutil.NewDefaultTestContext(t)
|
||||
dbp := test.NewDBProviderMatchWords(t)
|
||||
b, err := NewBackend(BackendOptions{
|
||||
DBProvider: dbp,
|
||||
SkipDataMigration: true, // Calling migrations makes startup SQL calls (avoid the mock)
|
||||
})
|
||||
b, err := NewBackend(BackendOptions{DBProvider: dbp})
|
||||
require.NoError(t, err)
|
||||
require.NotNil(t, b)
|
||||
|
||||
@@ -112,7 +109,7 @@ func TestBackend_Init(t *testing.T) {
|
||||
|
||||
ctx := testutil.NewDefaultTestContext(t)
|
||||
dbp := test.NewDBProviderWithPing(t)
|
||||
b, err := NewBackend(BackendOptions{DBProvider: dbp, SkipDataMigration: true})
|
||||
b, err := NewBackend(BackendOptions{DBProvider: dbp})
|
||||
require.NoError(t, err)
|
||||
require.NotNil(t, b)
|
||||
|
||||
@@ -169,7 +166,7 @@ func TestBackend_Init(t *testing.T) {
|
||||
|
||||
ctx := testutil.NewDefaultTestContext(t)
|
||||
dbp := test.NewDBProviderWithPing(t)
|
||||
b, err := NewBackend(BackendOptions{DBProvider: dbp, SkipDataMigration: true})
|
||||
b, err := NewBackend(BackendOptions{DBProvider: dbp})
|
||||
require.NoError(t, err)
|
||||
require.NotNil(t, dbp.DB)
|
||||
|
||||
@@ -185,7 +182,7 @@ func TestBackend_IsHealthy(t *testing.T) {
|
||||
|
||||
ctx := testutil.NewDefaultTestContext(t)
|
||||
dbp := test.NewDBProviderWithPing(t)
|
||||
b, err := NewBackend(BackendOptions{DBProvider: dbp, SkipDataMigration: true})
|
||||
b, err := NewBackend(BackendOptions{DBProvider: dbp})
|
||||
require.NoError(t, err)
|
||||
require.NotNil(t, dbp.DB)
|
||||
|
||||
|
||||
Reference in New Issue
Block a user