diff options
author | wxiaoguang <wxiaoguang@gmail.com> | 2023-05-24 00:30:19 +0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-05-23 16:30:19 +0000 |
commit | 8080ace6fcf73a5fbe4a0dd71881228abd0c68b9 (patch) | |
tree | f9d05eb8d638ecf45c9c00e5234e472542f94c99 /modules/git/git_test.go | |
parent | 910bf3154606107616ed7648004d80d323edf426 (diff) | |
download | gitea-8080ace6fcf73a5fbe4a0dd71881228abd0c68b9.tar.gz gitea-8080ace6fcf73a5fbe4a0dd71881228abd0c68b9.zip |
Support changing git config through `app.ini`, use `diff.algorithm=histogram` by default (#24860)
Close #13454 , Close #23255, Close #14697 (and maybe more related
issues)
Many users have the requirement to customize the git config. This PR
introduces an easy way: put the options in Gitea's app.ini
`[git.config]`, then the config options will be applied to git config.
And it can support more flexible default config values, eg: now
`diff.algorithm=histogram` by default. According to:
https://stackoverflow.com/a/32367597/4754037 , `histogram diff` is
efficient and doesn't like to cause server-side problems.
---------
Co-authored-by: silverwind <me@silverwind.io>
Co-authored-by: KN4CK3R <admin@oldschoolhack.me>
Co-authored-by: Giteabot <teabot@gitea.io>
Diffstat (limited to 'modules/git/git_test.go')
-rw-r--r-- | modules/git/git_test.go | 24 |
1 files changed, 18 insertions, 6 deletions
diff --git a/modules/git/git_test.go b/modules/git/git_test.go index 25eb308531..37ab669ea4 100644 --- a/modules/git/git_test.go +++ b/modules/git/git_test.go @@ -42,14 +42,14 @@ func TestMain(m *testing.M) { } } -func TestGitConfig(t *testing.T) { - gitConfigContains := func(sub string) bool { - if b, err := os.ReadFile(HomeDir() + "/.gitconfig"); err == nil { - return strings.Contains(string(b), sub) - } - return false +func gitConfigContains(sub string) bool { + if b, err := os.ReadFile(HomeDir() + "/.gitconfig"); err == nil { + return strings.Contains(string(b), sub) } + return false +} +func TestGitConfig(t *testing.T) { assert.False(t, gitConfigContains("key-a")) assert.NoError(t, configSetNonExist("test.key-a", "val-a")) @@ -81,3 +81,15 @@ func TestGitConfig(t *testing.T) { assert.NoError(t, configUnsetAll("test.key-x", "*")) assert.False(t, gitConfigContains("key-x = *")) } + +func TestSyncConfig(t *testing.T) { + oldGitConfig := setting.GitConfig + defer func() { + setting.GitConfig = oldGitConfig + }() + + setting.GitConfig.Options["sync-test.cfg-key-a"] = "CfgValA" + assert.NoError(t, syncGitConfig()) + assert.True(t, gitConfigContains("[sync-test]")) + assert.True(t, gitConfigContains("cfg-key-a = CfgValA")) +} |