]> source.dussan.org Git - gitea.git/commitdiff
Merge setting.InitXXX into one function with options (#24389)
authorLunny Xiao <xiaolunwen@gmail.com>
Thu, 4 May 2023 03:55:35 +0000 (11:55 +0800)
committerGitHub <noreply@github.com>
Thu, 4 May 2023 03:55:35 +0000 (11:55 +0800)
This PR will merge 3 Init functions on setting packages as 1 and
introduce an options struct.

28 files changed:
cmd/actions.go
cmd/cmd.go
cmd/doctor.go
cmd/dump.go
cmd/embedded.go
cmd/mailer.go
cmd/main_test.go
cmd/restore_repo.go
cmd/serv.go
cmd/web.go
models/asymkey/main_test.go
models/dbfs/main_test.go
models/issues/main_test.go
models/main_test.go
models/migrations/base/tests.go
models/unittest/testdb.go
modules/doctor/doctor.go
modules/doctor/paths.go
modules/markup/html_test.go
modules/markup/markdown/markdown_test.go
modules/setting/config_provider.go
modules/setting/setting.go
routers/api/v1/repo/main_test.go
routers/install/setting.go
routers/web/user/setting/account_test.go
services/webhook/main_test.go
tests/integration/migration-test/migration_test.go
tests/test_utils.go

index 66ad336da508eab502a3b89f884b720dca5825fc..346de5b21a6fc46022b8bcccaeef65f8c737483d 100644 (file)
@@ -42,8 +42,7 @@ func runGenerateActionsRunnerToken(c *cli.Context) error {
        ctx, cancel := installSignals()
        defer cancel()
 
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
 
        scope := c.String("scope")
 
index 18d5db3987bdd61299c041f4762b70d5243ee4ec..cf2d9ef89e831f9e717ebf01aac4e422632685a9 100644 (file)
@@ -57,8 +57,7 @@ func confirm() (bool, error) {
 }
 
 func initDB(ctx context.Context) error {
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
        setting.LoadDBSetting()
        setting.InitSQLLog(false)
 
index e7baad60c1f9491dedd3269efaa1b409768cb06e..65c028c5ed19d7bb97dfa7861b0e1b195d482c0d 100644 (file)
@@ -87,8 +87,7 @@ func runRecreateTable(ctx *cli.Context) error {
        golog.SetPrefix("")
        golog.SetOutput(log.NewLoggerAsWriter("INFO", log.GetLogger(log.DEFAULT)))
 
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
        setting.LoadDBSetting()
 
        setting.Log.EnableXORMLog = ctx.Bool("debug")
index 309bd01f6645f99f5e82c1a53e5a47536ebd7808..32ccc5566c8a9381248dd4be59ebe21bf7aa3342 100644 (file)
@@ -185,8 +185,7 @@ func runDump(ctx *cli.Context) error {
                }
                fileName += "." + outType
        }
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
 
        // make sure we are logging to the console no matter what the configuration tells us do to
        // FIXME: don't use CfgProvider directly
index cee8928ce08db262584a95fa93ff15853d2ad47c..3f849bea0a2642f95f071478d686741ae45e1eaa 100644 (file)
@@ -106,8 +106,9 @@ func initEmbeddedExtractor(c *cli.Context) error {
        log.DelNamedLogger(log.DEFAULT)
 
        // Read configuration file
-       setting.InitProviderAllowEmpty()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{
+               AllowEmpty: true,
+       })
 
        patterns, err := compileCollectPatterns(c.Args())
        if err != nil {
index 50ba4b474110b53112365f65355e12ae2b396155..74bae1ab68c76273313ecc9ec38975f1c7452cbf 100644 (file)
@@ -16,8 +16,7 @@ func runSendMail(c *cli.Context) error {
        ctx, cancel := installSignals()
        defer cancel()
 
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
 
        if err := argsSet(c, "title"); err != nil {
                return err
index ba323af47217874132d235147653ef980b36797f..6e20be69451c3956ce9e1894a832498eb11c2ed8 100644 (file)
@@ -7,14 +7,8 @@ import (
        "testing"
 
        "code.gitea.io/gitea/models/unittest"
-       "code.gitea.io/gitea/modules/setting"
 )
 
-func init() {
-       setting.SetCustomPathAndConf("", "", "")
-       setting.InitProviderAndLoadCommonSettingsForTest()
-}
-
 func TestMain(m *testing.M) {
        unittest.MainTest(m, &unittest.TestOptions{
                GiteaRootPath: "..",
index 887b59bba9e14f0cec60ec8318c4c2bce67405f6..5a7ede49397568798cdfedf37a31e75bda8aa761 100644 (file)
@@ -51,8 +51,7 @@ func runRestoreRepository(c *cli.Context) error {
        ctx, cancel := installSignals()
        defer cancel()
 
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
        var units []string
        if s := c.String("units"); s != "" {
                units = strings.Split(s, ",")
index 72eb6370711e420061e2a7565bf8d6332411b425..a79f314d00b691887921ff3797f117e9fd2909e5 100644 (file)
@@ -62,8 +62,7 @@ func setup(ctx context.Context, debug bool) {
        } else {
                _ = log.NewLogger(1000, "console", "console", `{"level":"fatal","stacktracelevel":"NONE","stderr":true}`)
        }
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
        if debug {
                setting.RunMode = "dev"
        }
index e451cf7dfa4fe8b259cc2ea9943dd019c7f71350..3a01d07b05f591ac3f08fcb1517274635c77090b 100644 (file)
@@ -177,8 +177,7 @@ func runWeb(ctx *cli.Context) error {
 
        log.Info("Global init")
        // Perform global initialization
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
        routers.GlobalInitInstalled(graceful.GetManager().HammerContext())
 
        // We check that AppDataPath exists here (it should have been created during installation)
index 7f8657189fd182e0acaccae8d5b752768ad5f9a8..701722be12ec8222ae597574aa27d46732b4698d 100644 (file)
@@ -8,14 +8,8 @@ import (
        "testing"
 
        "code.gitea.io/gitea/models/unittest"
-       "code.gitea.io/gitea/modules/setting"
 )
 
-func init() {
-       setting.SetCustomPathAndConf("", "", "")
-       setting.InitProviderAndLoadCommonSettingsForTest()
-}
-
 func TestMain(m *testing.M) {
        unittest.MainTest(m, &unittest.TestOptions{
                GiteaRootPath: filepath.Join("..", ".."),
index 9dce663eeab99eb45c444d2a7e532721147d12f6..62db3592bed2a5725def938e295694c329771617 100644 (file)
@@ -8,14 +8,8 @@ import (
        "testing"
 
        "code.gitea.io/gitea/models/unittest"
-       "code.gitea.io/gitea/modules/setting"
 )
 
-func init() {
-       setting.SetCustomPathAndConf("", "", "")
-       setting.InitProviderAndLoadCommonSettingsForTest()
-}
-
 func TestMain(m *testing.M) {
        unittest.MainTest(m, &unittest.TestOptions{
                GiteaRootPath: filepath.Join("..", ".."),
index de84da30ecc0a62f0e1d871963e63f0323a7ea20..9fbe294f7067da52225013df91be24649f62477b 100644 (file)
@@ -9,7 +9,6 @@ import (
 
        issues_model "code.gitea.io/gitea/models/issues"
        "code.gitea.io/gitea/models/unittest"
-       "code.gitea.io/gitea/modules/setting"
 
        _ "code.gitea.io/gitea/models"
        _ "code.gitea.io/gitea/models/repo"
@@ -18,11 +17,6 @@ import (
        "github.com/stretchr/testify/assert"
 )
 
-func init() {
-       setting.SetCustomPathAndConf("", "", "")
-       setting.InitProviderAndLoadCommonSettingsForTest()
-}
-
 func TestFixturesAreConsistent(t *testing.T) {
        assert.NoError(t, unittest.PrepareTestDatabase())
        unittest.CheckConsistencyFor(t,
index b5919bb28615f0f4a2b2d43deea6171f8de12273..d490507649a33b407aafb707ed7038b40892a344 100644 (file)
@@ -11,18 +11,12 @@ import (
        repo_model "code.gitea.io/gitea/models/repo"
        "code.gitea.io/gitea/models/unittest"
        user_model "code.gitea.io/gitea/models/user"
-       "code.gitea.io/gitea/modules/setting"
 
        _ "code.gitea.io/gitea/models/system"
 
        "github.com/stretchr/testify/assert"
 )
 
-func init() {
-       setting.SetCustomPathAndConf("", "", "")
-       setting.InitProviderAndLoadCommonSettingsForTest()
-}
-
 // TestFixturesAreConsistent assert that test fixtures are consistent
 func TestFixturesAreConsistent(t *testing.T) {
        assert.NoError(t, unittest.PrepareTestDatabase())
index 14f374f1a7c3dc1c2383f708440b472fe8f6cbee..124111f51fb2f35d5817e1e327485dda7cc1dd7d 100644 (file)
@@ -150,7 +150,7 @@ func MainTest(m *testing.M) {
        setting.AppDataPath = tmpDataPath
 
        setting.SetCustomPathAndConf("", "", "")
-       setting.InitProviderAndLoadCommonSettingsForTest()
+       unittest.InitSettings()
        if err = git.InitFull(context.Background()); err != nil {
                fmt.Printf("Unable to InitFull: %v\n", err)
                os.Exit(1)
index cff1489a7c7bcac6003d4f0243c79f9b752fdb81..a5b126350d8e153b9bc6cbaa7bba16c7cc363bfe 100644 (file)
@@ -6,12 +6,15 @@ package unittest
 import (
        "context"
        "fmt"
+       "log"
        "os"
        "path/filepath"
+       "strings"
        "testing"
 
        "code.gitea.io/gitea/models/db"
        system_model "code.gitea.io/gitea/models/system"
+       "code.gitea.io/gitea/modules/auth/password/hash"
        "code.gitea.io/gitea/modules/base"
        "code.gitea.io/gitea/modules/git"
        "code.gitea.io/gitea/modules/setting"
@@ -39,6 +42,22 @@ func fatalTestError(fmtStr string, args ...interface{}) {
        os.Exit(1)
 }
 
+// InitSettings initializes config provider and load common setttings for tests
+func InitSettings(extraConfigs ...string) {
+       setting.Init(&setting.Options{
+               AllowEmpty:  true,
+               ExtraConfig: strings.Join(extraConfigs, "\n"),
+       })
+
+       if err := setting.PrepareAppDataPath(); err != nil {
+               log.Fatalf("Can not prepare APP_DATA_PATH: %v", err)
+       }
+       // register the dummy hash algorithm function used in the test fixtures
+       _ = hash.Register("dummy", hash.NewDummyHasher)
+
+       setting.PasswordHashAlgo, _ = hash.SetDefaultPasswordHashAlgorithm("dummy")
+}
+
 // TestOptions represents test options
 type TestOptions struct {
        GiteaRootPath string
@@ -50,6 +69,9 @@ type TestOptions struct {
 // MainTest a reusable TestMain(..) function for unit tests that need to use a
 // test database. Creates the test database, and sets necessary settings.
 func MainTest(m *testing.M, testOpts *TestOptions) {
+       setting.SetCustomPathAndConf("", "", "")
+       InitSettings()
+
        var err error
 
        giteaRoot = testOpts.GiteaRootPath
index b23805bc4c963c4ea77f300a958ac060684c4c2c..32eb5938c307ca9910f7ec8494d09f374b8b47d8 100644 (file)
@@ -44,8 +44,7 @@ func (w *wrappedLevelLogger) Log(skip int, level log.Level, format string, v ...
 }
 
 func initDBDisableConsole(ctx context.Context, disableConsole bool) error {
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
        setting.LoadDBSetting()
        setting.InitSQLLog(disableConsole)
        if err := db.InitEngine(ctx); err != nil {
index 1558efc25b90c19a3cafdafc58cb6ee494fe0158..957152349c25fcc9f2ecfab5199c904d9be9b479 100644 (file)
@@ -66,8 +66,7 @@ func checkConfigurationFiles(ctx context.Context, logger log.Logger, autofix boo
                return err
        }
 
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
 
        configurationFiles := []configurationFile{
                {"Configuration File Path", setting.CustomConf, false, true, false},
index cb1216ec946eef915fe99560bac5f4d8275d5166..5e5e4fecbbb7fb9e9b35c0f0bcf70973c56404ce 100644 (file)
@@ -28,8 +28,9 @@ var localMetas = map[string]string{
 }
 
 func TestMain(m *testing.M) {
-       setting.InitProviderAllowEmpty()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{
+               AllowEmpty: true,
+       })
        if err := git.InitSimple(context.Background()); err != nil {
                log.Fatal("git init failed, err: %v", err)
        }
index 0c7650a5ffabc8cb3aef299c3e40918ef9935c55..e81869d7a44366bec1d7d41f8e6561e286cbf95b 100644 (file)
@@ -33,8 +33,9 @@ var localMetas = map[string]string{
 }
 
 func TestMain(m *testing.M) {
-       setting.InitProviderAllowEmpty()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{
+               AllowEmpty: true,
+       })
        if err := git.InitSimple(context.Background()); err != nil {
                log.Fatal("git init failed, err: %v", err)
        }
index 92c8c97fe95289f98abe22debb3eecc278688d4b..16859582986352b39ba3956248f8f0dada4456da 100644 (file)
@@ -35,10 +35,9 @@ type ConfigProvider interface {
 }
 
 type iniFileConfigProvider struct {
+       opts *Options
        *ini.File
-       filepath   string // the ini file path
-       newFile    bool   // whether the file has not existed previously
-       allowEmpty bool   // whether not finding configuration files is allowed (only true for the tests)
+       newFile bool // whether the file has not existed previously
 }
 
 // NewEmptyConfigProvider create a new empty config provider
@@ -66,41 +65,47 @@ func newConfigProviderFromData(configContent string) (ConfigProvider, error) {
        }, nil
 }
 
+type Options struct {
+       CustomConf                string // the ini file path
+       AllowEmpty                bool   // whether not finding configuration files is allowed (only true for the tests)
+       ExtraConfig               string
+       DisableLoadCommonSettings bool
+}
+
 // newConfigProviderFromFile load configuration from file.
 // NOTE: do not print any log except error.
-func newConfigProviderFromFile(customConf string, allowEmpty bool, extraConfig string) (*iniFileConfigProvider, error) {
+func newConfigProviderFromFile(opts *Options) (*iniFileConfigProvider, error) {
        cfg := ini.Empty()
        newFile := true
 
-       if customConf != "" {
-               isFile, err := util.IsFile(customConf)
+       if opts.CustomConf != "" {
+               isFile, err := util.IsFile(opts.CustomConf)
                if err != nil {
-                       return nil, fmt.Errorf("unable to check if %s is a file. Error: %v", customConf, err)
+                       return nil, fmt.Errorf("unable to check if %s is a file. Error: %v", opts.CustomConf, err)
                }
                if isFile {
-                       if err := cfg.Append(customConf); err != nil {
-                               return nil, fmt.Errorf("failed to load custom conf '%s': %v", customConf, err)
+                       if err := cfg.Append(opts.CustomConf); err != nil {
+                               return nil, fmt.Errorf("failed to load custom conf '%s': %v", opts.CustomConf, err)
                        }
                        newFile = false
                }
        }
 
-       if newFile && !allowEmpty {
+       if newFile && !opts.AllowEmpty {
                return nil, fmt.Errorf("unable to find configuration file: %q, please ensure you are running in the correct environment or set the correct configuration file with -c", CustomConf)
        }
 
-       if extraConfig != "" {
-               if err := cfg.Append([]byte(extraConfig)); err != nil {
+       if opts.ExtraConfig != "" {
+               if err := cfg.Append([]byte(opts.ExtraConfig)); err != nil {
                        return nil, fmt.Errorf("unable to append more config: %v", err)
                }
        }
 
        cfg.NameMapper = ini.SnackCase
        return &iniFileConfigProvider{
-               File:       cfg,
-               filepath:   customConf,
-               newFile:    newFile,
-               allowEmpty: allowEmpty,
+               opts:    opts,
+               File:    cfg,
+               newFile: newFile,
        }, nil
 }
 
@@ -123,8 +128,8 @@ func (p *iniFileConfigProvider) DeleteSection(name string) error {
 
 // Save save the content into file
 func (p *iniFileConfigProvider) Save() error {
-       if p.filepath == "" {
-               if !p.allowEmpty {
+       if p.opts.CustomConf == "" {
+               if !p.opts.AllowEmpty {
                        return fmt.Errorf("custom config path must not be empty")
                }
                return nil
@@ -135,8 +140,8 @@ func (p *iniFileConfigProvider) Save() error {
                        return fmt.Errorf("failed to create '%s': %v", CustomConf, err)
                }
        }
-       if err := p.SaveTo(p.filepath); err != nil {
-               return fmt.Errorf("failed to save '%s': %v", p.filepath, err)
+       if err := p.SaveTo(p.opts.CustomConf); err != nil {
+               return fmt.Errorf("failed to save '%s': %v", p.opts.CustomConf, err)
        }
 
        // Change permissions to be more restrictive
index 9ab55e91c5316e4bb383472608d167f077f343df..b085a7b321495843f2b57917b43986c378b6f2ae 100644 (file)
@@ -15,7 +15,6 @@ import (
        "strings"
        "time"
 
-       "code.gitea.io/gitea/modules/auth/password/hash"
        "code.gitea.io/gitea/modules/log"
        "code.gitea.io/gitea/modules/user"
 )
@@ -203,44 +202,18 @@ func PrepareAppDataPath() error {
        return nil
 }
 
-// InitProviderFromExistingFile initializes config provider from an existing config file (app.ini)
-func InitProviderFromExistingFile() {
-       var err error
-       CfgProvider, err = newConfigProviderFromFile(CustomConf, false, "")
-       if err != nil {
-               log.Fatal("InitProviderFromExistingFile: %v", err)
-       }
-}
-
-// InitProviderAllowEmpty initializes config provider from file, it's also fine that if the config file (app.ini) doesn't exist
-func InitProviderAllowEmpty() {
-       var err error
-       CfgProvider, err = newConfigProviderFromFile(CustomConf, true, "")
-       if err != nil {
-               log.Fatal("InitProviderAllowEmpty: %v", err)
+func Init(opts *Options) {
+       if opts.CustomConf == "" {
+               opts.CustomConf = CustomConf
        }
-}
-
-// InitProviderAndLoadCommonSettingsForTest initializes config provider and load common setttings for tests
-func InitProviderAndLoadCommonSettingsForTest(extraConfigs ...string) {
        var err error
-       CfgProvider, err = newConfigProviderFromFile(CustomConf, true, strings.Join(extraConfigs, "\n"))
+       CfgProvider, err = newConfigProviderFromFile(opts)
        if err != nil {
-               log.Fatal("InitProviderAndLoadCommonSettingsForTest: %v", err)
+               log.Fatal("Init[%v]: %v", opts, err)
        }
-       loadCommonSettingsFrom(CfgProvider)
-       if err := PrepareAppDataPath(); err != nil {
-               log.Fatal("Can not prepare APP_DATA_PATH: %v", err)
+       if !opts.DisableLoadCommonSettings {
+               loadCommonSettingsFrom(CfgProvider)
        }
-       // register the dummy hash algorithm function used in the test fixtures
-       _ = hash.Register("dummy", hash.NewDummyHasher)
-
-       PasswordHashAlgo, _ = hash.SetDefaultPasswordHashAlgorithm("dummy")
-}
-
-// LoadCommonSettings loads common configurations from a configuration provider.
-func LoadCommonSettings() {
-       loadCommonSettingsFrom(CfgProvider)
 }
 
 // loadCommonSettingsFrom loads common configurations from a configuration provider.
index c7466c493f015eb6120ffc53452f80b3d0792f7a..bc048505f472f32c3d18aa8765853d548195fe12 100644 (file)
@@ -13,10 +13,11 @@ import (
 )
 
 func TestMain(m *testing.M) {
-       setting.InitProviderAndLoadCommonSettingsForTest()
-       setting.LoadQueueSettings()
        unittest.MainTest(m, &unittest.TestOptions{
                GiteaRootPath: filepath.Join("..", "..", "..", ".."),
-               SetUp:         webhook_service.Init,
+               SetUp: func() error {
+                       setting.LoadQueueSettings()
+                       return webhook_service.Init()
+               },
        })
 }
index dadefa26a2930524faa5d6bbb5831ae5f3ed3418..c14843d8ee4d32c2ee44153991ffba9941199752 100644 (file)
@@ -15,8 +15,9 @@ import (
 
 // PreloadSettings preloads the configuration to check if we need to run install
 func PreloadSettings(ctx context.Context) bool {
-       setting.InitProviderAllowEmpty()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{
+               AllowEmpty: true,
+       })
        if !setting.InstallLock {
                log.Info("AppPath: %s", setting.AppPath)
                log.Info("AppWorkPath: %s", setting.AppWorkPath)
@@ -38,8 +39,7 @@ func PreloadSettings(ctx context.Context) bool {
 
 // reloadSettings reloads the existing settings and starts up the database
 func reloadSettings(ctx context.Context) {
-       setting.InitProviderFromExistingFile()
-       setting.LoadCommonSettings()
+       setting.Init(&setting.Options{})
        setting.LoadDBSetting()
        if setting.InstallLock {
                if err := common.InitDBEngine(ctx); err == nil {
index 5fce41f065ba464ded877b0457bc9d0ad668bdec..569d597722ee3626759b358df616d94a02ed9231 100644 (file)
@@ -80,19 +80,22 @@ func TestChangePassword(t *testing.T) {
                        PasswordComplexity: pcLU,
                },
        } {
-               unittest.PrepareTestEnv(t)
-               ctx := test.MockContext(t, "user/settings/security")
-               test.LoadUser(t, ctx, 2)
-               test.LoadRepo(t, ctx, 1)
+               t.Run(req.OldPassword+"__"+req.NewPassword, func(t *testing.T) {
+                       unittest.PrepareTestEnv(t)
+                       setting.PasswordComplexity = req.PasswordComplexity
+                       ctx := test.MockContext(t, "user/settings/security")
+                       test.LoadUser(t, ctx, 2)
+                       test.LoadRepo(t, ctx, 1)
 
-               web.SetForm(ctx, &forms.ChangePasswordForm{
-                       OldPassword: req.OldPassword,
-                       Password:    req.NewPassword,
-                       Retype:      req.Retype,
-               })
-               AccountPost(ctx)
+                       web.SetForm(ctx, &forms.ChangePasswordForm{
+                               OldPassword: req.OldPassword,
+                               Password:    req.NewPassword,
+                               Retype:      req.Retype,
+                       })
+                       AccountPost(ctx)
 
-               assert.Contains(t, ctx.Flash.ErrorMsg, req.Message)
-               assert.EqualValues(t, http.StatusSeeOther, ctx.Resp.Status())
+                       assert.Contains(t, ctx.Flash.ErrorMsg, req.Message)
+                       assert.EqualValues(t, http.StatusSeeOther, ctx.Resp.Status())
+               })
        }
 }
index 210221b12041369ce5af08bc5bc03b9782a57ba7..0189e178401650651e8c5db7751c337b984e8384 100644 (file)
@@ -15,13 +15,13 @@ import (
 )
 
 func TestMain(m *testing.M) {
-       setting.InitProviderAndLoadCommonSettingsForTest()
-       setting.LoadQueueSettings()
-
        // for tests, allow only loopback IPs
        setting.Webhook.AllowedHostList = hostmatcher.MatchBuiltinLoopback
        unittest.MainTest(m, &unittest.TestOptions{
                GiteaRootPath: filepath.Join("..", ".."),
-               SetUp:         Init,
+               SetUp: func() error {
+                       setting.LoadQueueSettings()
+                       return Init()
+               },
        })
 }
index 4152379a9b80a53ad940554201c7363891a7be81..a68d458a0b1ea58481f1df40796897727ac5f6df 100644 (file)
@@ -57,7 +57,7 @@ func initMigrationTest(t *testing.T) func() {
                setting.CustomConf = giteaConf
        }
 
-       setting.InitProviderAndLoadCommonSettingsForTest()
+       unittest.InitSettings()
 
        assert.True(t, len(setting.RepoRootPath) != 0)
        assert.NoError(t, util.RemoveAll(setting.RepoRootPath))
index 0f4aa26a60742b6fd4444222fd05614b48faaaf2..c22b2c356c65620919d997115afedd474c653e55 100644 (file)
@@ -74,7 +74,7 @@ func InitTest(requireGitea bool) {
        }
 
        setting.SetCustomPathAndConf("", "", "")
-       setting.InitProviderAndLoadCommonSettingsForTest()
+       unittest.InitSettings()
        setting.Repository.DefaultBranch = "master" // many test code still assume that default branch is called "master"
        _ = util.RemoveAll(repo_module.LocalCopyPath())