mirror of
				https://codeberg.org/forgejo/forgejo.git
				synced 2025-10-28 04:51:07 +00:00 
			
		
		
		
	The INI package has many bugs and quirks, and in fact it is unmaintained. This PR is the first step for the INI package refactoring: * Use Gitea's "config_provider" to provide INI access * Deprecate the INI package by golangci.yml rule
		
			
				
	
	
		
			121 lines
		
	
	
	
		
			3.9 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
			
		
		
	
	
			121 lines
		
	
	
	
		
			3.9 KiB
		
	
	
	
		
			Go
		
	
	
	
	
	
| // Copyright 2023 The Gitea Authors. All rights reserved.
 | |
| // SPDX-License-Identifier: MIT
 | |
| 
 | |
| package setting
 | |
| 
 | |
| import (
 | |
| 	"os"
 | |
| 	"testing"
 | |
| 
 | |
| 	"github.com/stretchr/testify/assert"
 | |
| )
 | |
| 
 | |
| func TestConfigProviderBehaviors(t *testing.T) {
 | |
| 	t.Run("BuggyKeyOverwritten", func(t *testing.T) {
 | |
| 		cfg, _ := NewConfigProviderFromData(`
 | |
| [foo]
 | |
| key =
 | |
| `)
 | |
| 		sec := cfg.Section("foo")
 | |
| 		secSub := cfg.Section("foo.bar")
 | |
| 		secSub.Key("key").MustString("1")             // try to read a key from subsection
 | |
| 		assert.Equal(t, "1", sec.Key("key").String()) // TODO: BUGGY! the key in [foo] is overwritten
 | |
| 	})
 | |
| 
 | |
| 	t.Run("SubsectionSeeParentKeys", func(t *testing.T) {
 | |
| 		cfg, _ := NewConfigProviderFromData(`
 | |
| [foo]
 | |
| key = 123
 | |
| `)
 | |
| 		secSub := cfg.Section("foo.bar.xxx")
 | |
| 		assert.Equal(t, "123", secSub.Key("key").String())
 | |
| 	})
 | |
| }
 | |
| 
 | |
| func TestConfigProviderHelper(t *testing.T) {
 | |
| 	cfg, _ := NewConfigProviderFromData(`
 | |
| [foo]
 | |
| empty =
 | |
| key = 123
 | |
| `)
 | |
| 
 | |
| 	sec := cfg.Section("foo")
 | |
| 	secSub := cfg.Section("foo.bar")
 | |
| 
 | |
| 	// test empty key
 | |
| 	assert.Equal(t, "def", ConfigSectionKeyString(sec, "empty", "def"))
 | |
| 	assert.Equal(t, "xyz", ConfigSectionKeyString(secSub, "empty", "xyz"))
 | |
| 
 | |
| 	// test non-inherited key, only see the keys in current section
 | |
| 	assert.NotNil(t, ConfigSectionKey(sec, "key"))
 | |
| 	assert.Nil(t, ConfigSectionKey(secSub, "key"))
 | |
| 
 | |
| 	// test default behavior
 | |
| 	assert.Equal(t, "123", ConfigSectionKeyString(sec, "key"))
 | |
| 	assert.Equal(t, "", ConfigSectionKeyString(secSub, "key"))
 | |
| 	assert.Equal(t, "def", ConfigSectionKeyString(secSub, "key", "def"))
 | |
| 
 | |
| 	assert.Equal(t, "123", ConfigInheritedKeyString(secSub, "key"))
 | |
| 
 | |
| 	// Workaround for ini package's BuggyKeyOverwritten behavior
 | |
| 	assert.Equal(t, "", ConfigSectionKeyString(sec, "empty"))
 | |
| 	assert.Equal(t, "", ConfigSectionKeyString(secSub, "empty"))
 | |
| 	assert.Equal(t, "def", ConfigInheritedKey(secSub, "empty").MustString("def"))
 | |
| 	assert.Equal(t, "def", ConfigInheritedKey(secSub, "empty").MustString("xyz"))
 | |
| 	assert.Equal(t, "", ConfigSectionKeyString(sec, "empty"))
 | |
| 	assert.Equal(t, "def", ConfigSectionKeyString(secSub, "empty"))
 | |
| }
 | |
| 
 | |
| func TestNewConfigProviderFromFile(t *testing.T) {
 | |
| 	_, err := NewConfigProviderFromFile(&Options{CustomConf: "no-such.ini", AllowEmpty: false})
 | |
| 	assert.ErrorContains(t, err, "unable to find configuration file")
 | |
| 
 | |
| 	// load non-existing file and save
 | |
| 	testFile := t.TempDir() + "/test.ini"
 | |
| 	testFile1 := t.TempDir() + "/test1.ini"
 | |
| 	cfg, err := NewConfigProviderFromFile(&Options{CustomConf: testFile, AllowEmpty: true})
 | |
| 	assert.NoError(t, err)
 | |
| 
 | |
| 	sec, _ := cfg.NewSection("foo")
 | |
| 	_, _ = sec.NewKey("k1", "a")
 | |
| 	assert.NoError(t, cfg.Save())
 | |
| 	_, _ = sec.NewKey("k2", "b")
 | |
| 	assert.NoError(t, cfg.SaveTo(testFile1))
 | |
| 
 | |
| 	bs, err := os.ReadFile(testFile)
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "[foo]\nk1=a\n", string(bs))
 | |
| 
 | |
| 	bs, err = os.ReadFile(testFile1)
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "[foo]\nk1=a\nk2=b\n", string(bs))
 | |
| 
 | |
| 	// load existing file and save
 | |
| 	cfg, err = NewConfigProviderFromFile(&Options{CustomConf: testFile, AllowEmpty: true})
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "a", cfg.Section("foo").Key("k1").String())
 | |
| 	sec, _ = cfg.NewSection("bar")
 | |
| 	_, _ = sec.NewKey("k1", "b")
 | |
| 	assert.NoError(t, cfg.Save())
 | |
| 	bs, err = os.ReadFile(testFile)
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "[foo]\nk1=a\n\n[bar]\nk1=b\n", string(bs))
 | |
| }
 | |
| 
 | |
| func TestNewConfigProviderForLocale(t *testing.T) {
 | |
| 	// load locale from file
 | |
| 	localeFile := t.TempDir() + "/locale.ini"
 | |
| 	_ = os.WriteFile(localeFile, []byte(`k1=a`), 0o644)
 | |
| 	cfg, err := NewConfigProviderForLocale(localeFile)
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "a", cfg.Section("").Key("k1").String())
 | |
| 
 | |
| 	// load locale from bytes
 | |
| 	cfg, err = NewConfigProviderForLocale([]byte("k1=foo\nk2=bar"))
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "foo", cfg.Section("").Key("k1").String())
 | |
| 	cfg, err = NewConfigProviderForLocale([]byte("k1=foo\nk2=bar"), []byte("k2=xxx"))
 | |
| 	assert.NoError(t, err)
 | |
| 	assert.Equal(t, "foo", cfg.Section("").Key("k1").String())
 | |
| 	assert.Equal(t, "xxx", cfg.Section("").Key("k2").String())
 | |
| }
 |