frostfs-node/cmd/internal/common/config/viper_test.go
Aleksey Savchuk 5fac4058e8
All checks were successful
DCO action / DCO (pull_request) Successful in 51s
Tests and linters / Run gofumpt (pull_request) Successful in 1m41s
Vulncheck / Vulncheck (pull_request) Successful in 1m54s
Pre-commit hooks / Pre-commit (pull_request) Successful in 2m13s
Build / Build Components (pull_request) Successful in 2m21s
Tests and linters / gopls check (pull_request) Successful in 2m31s
Tests and linters / Staticcheck (pull_request) Successful in 2m34s
Tests and linters / Lint (pull_request) Successful in 3m30s
Tests and linters / Tests (pull_request) Successful in 3m35s
Tests and linters / Tests with -race (pull_request) Successful in 3m37s
[#1364] cmd/common: Add tests for CreateViper and ReloadViper
Add tests for `CreateViper` and `ReloadViper` to ensure that no extra
files, except *.yaml, *.yml, *.json, are loaded from config directory.

Signed-off-by: Aleksey Savchuk <a.savchuk@yadro.com>
2024-09-11 08:03:05 +00:00

107 lines
3.8 KiB
Go

package config_test
import (
"encoding/json"
"os"
"path"
"testing"
"git.frostfs.info/TrueCloudLab/frostfs-node/cmd/internal/common/config"
configtest "git.frostfs.info/TrueCloudLab/frostfs-node/pkg/util/config/test"
"github.com/spf13/viper"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
"gopkg.in/yaml.v3"
)
func TestCreateReloadViper(t *testing.T) {
type m = map[string]any
dummyFileSize := 1 << 10
configPath := t.TempDir()
configFile := "000_a.yaml"
configDirPath := path.Join(configPath, "conf.d")
require.NoError(t, os.Mkdir(configDirPath, 0o700))
configtest.PrepareConfigFiles(t, configPath, []configtest.ConfigFile{
configtest.NewConfigFile(configFile, m{"a": "000"}, yaml.Marshal),
})
// Not valid configs, dummy files those appear lexicographically first.
configtest.PrepareDummyFiles(t, configDirPath, []configtest.DummyFile{
configtest.NewDummyFile("000_file_1", dummyFileSize),
configtest.NewDummyFile("000_file_2", dummyFileSize),
configtest.NewDummyFile("000_file_3", dummyFileSize),
})
configtest.PrepareConfigFiles(t, configDirPath, []configtest.ConfigFile{
// Valid configs with invalid extensions those appear lexicographically first.
configtest.NewConfigFile("001_a.yaml.un~", m{"a": "101"}, yaml.Marshal),
configtest.NewConfigFile("001_b.yml~", m{"b": m{"a": "102", "b": "103"}}, yaml.Marshal),
configtest.NewConfigFile("001_c.yaml.swp", m{"c": m{"a": "104", "b": "105"}}, yaml.Marshal),
configtest.NewConfigFile("001_d.json.swp", m{"d": m{"a": "106", "b": "107"}}, json.Marshal),
// Valid configs with valid extensions those should be loaded.
configtest.NewConfigFile("010_a.yaml", m{"a": "1"}, yaml.Marshal),
configtest.NewConfigFile("020_b.yml", m{"b": m{"a": "2", "b": "3"}}, yaml.Marshal),
configtest.NewConfigFile("030_c.json", m{"c": m{"a": "4", "b": "5"}}, json.Marshal),
// Valid configs with invalid extensions those appear lexicographically last.
configtest.NewConfigFile("099_a.yaml.un~", m{"a": "201"}, yaml.Marshal),
configtest.NewConfigFile("099_b.yml~", m{"b": m{"a": "202", "b": "203"}}, yaml.Marshal),
configtest.NewConfigFile("099_c.yaml.swp", m{"c": m{"a": "204", "b": "205"}}, yaml.Marshal),
configtest.NewConfigFile("099_c.json.swp", m{"d": m{"a": "206", "b": "207"}}, json.Marshal),
})
// Not valid configs, dummy files those appear lexicographically last.
configtest.PrepareDummyFiles(t, configDirPath, []configtest.DummyFile{
configtest.NewDummyFile("999_file_1", dummyFileSize),
configtest.NewDummyFile("999_file_2", dummyFileSize),
configtest.NewDummyFile("999_file_3", dummyFileSize),
})
finalConfig := m{"a": "1", "b": m{"a": "2", "b": "3"}, "c": m{"a": "4", "b": "5"}}
var (
v *viper.Viper
err error
)
t.Run("create config with config dir only", func(t *testing.T) {
v, err = config.CreateViper(
config.WithConfigDir(configDirPath),
)
require.NoError(t, err)
assert.Equal(t, finalConfig, v.AllSettings())
})
t.Run("reload config with config dir only", func(t *testing.T) {
err = config.ReloadViper(
config.WithViper(v),
config.WithConfigDir(configDirPath),
)
require.NoError(t, err)
assert.Equal(t, finalConfig, v.AllSettings())
})
t.Run("create config with both config and config dir", func(t *testing.T) {
v, err = config.CreateViper(
config.WithConfigFile(path.Join(configPath, configFile)),
config.WithConfigDir(configDirPath),
)
require.NoError(t, err)
assert.Equal(t, finalConfig, v.AllSettings())
})
t.Run("reload config with both config and config dir", func(t *testing.T) {
err = config.ReloadViper(
config.WithViper(v),
config.WithConfigFile(path.Join(configPath, configFile)),
config.WithConfigDir(configDirPath),
)
require.NoError(t, err)
assert.Equal(t, finalConfig, v.AllSettings())
})
}