From 01150ae962980b6b9b86044a58f81fea2adc7dc7 Mon Sep 17 00:00:00 2001 From: Dafydd <72009875+dafydd-t@users.noreply.github.com> Date: Thu, 15 Apr 2021 10:32:49 +0100 Subject: [PATCH] Allow skip migrations in tests via environment variable (#32958) --- pkg/services/sqlstore/sqlstore.go | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/pkg/services/sqlstore/sqlstore.go b/pkg/services/sqlstore/sqlstore.go index f1a8f8a7456..bf36b35a010 100644 --- a/pkg/services/sqlstore/sqlstore.go +++ b/pkg/services/sqlstore/sqlstore.go @@ -447,6 +447,15 @@ func InitTestDB(t ITestDB, opts ...InitTestDBOpt) *SQLStore { } } + // useful if you already have a database that you want to use for tests. + // cannot just set it on testSQLStore as it overrides the config in Init + if _, present := os.LookupEnv("SKIP_MIGRATIONS"); present { + t.Log("Skipping database migrations") + if _, err := sec.NewKey("skip_migrations", "true"); err != nil { + t.Fatalf("Failed to create key: %s", err) + } + } + // need to get engine to clean db before we init t.Logf("Creating database connection: %q", sec.Key("connection_string")) engine, err := xorm.NewEngine(dbType, sec.Key("connection_string").String())