test: switch to WAL journal mode in unit tests

Signed-off-by: Jian Zeng <[email protected]>

pkg/server/main.go

@@ -53,10 +53,8 @@ func New(configPath string) (*Server, error) {
 }
 
 func NewWithConfig(cfg Config) (*Server, error) {
-	// TODO: enforce shared cache mode?
 	db, err := gorm.Open(sqlite.Open(cfg.DbURL), &gorm.Config{
-		QueryFields:            true,
-		SkipDefaultTransaction: true,
+		QueryFields: true,
 	})
 	if err != nil {
 		return nil, fmt.Errorf("open db: %w", err)

pkg/server/main_test.go

@@ -56,16 +56,11 @@ func NewTestEnv(t *testing.T) *TestEnv {
 		_ = dbFile.Close()
 		_ = os.Remove(dbFile.Name())
 	})
-	db, err := gorm.Open(sqlite.Open(dbFile.Name()), &gorm.Config{
-		QueryFields:            true,
-		SkipDefaultTransaction: true,
+	// Switch to WAL mode to avoid "database is locked" error.
+	db, err := gorm.Open(sqlite.Open(dbFile.Name()+"?_journal_mode=WAL"), &gorm.Config{
+		QueryFields: true,
 	})
 	require.NoError(t, err)
-	sqlDB, err := db.DB()
-	require.NoError(t, err)
-	// To resolve the "database is locked" error.
-	// See also https://github.com/mattn/go-sqlite3/issues/209
-	sqlDB.SetMaxOpenConns(1)
 	require.NoError(t, model.AutoMigrate(db))
 
 	s := &Server{