Some checks failed
Server CI / Postgres (shard 0) (push) Has been skipped
Server CI / Postgres (shard 3) (push) Has been skipped
BuildEnv Docker Image / build-image (push) Failing after 4s
BuildEnv Docker Image / build-image-fips (push) Failing after 4s
Migration Review & Release Notes / Generate migration review & release notes (push) Failing after 3s
Server CI / Build Environment (push) Failing after 3s
Server CI / Postgres (shard 1) (push) Has been skipped
Tools CI / check-style (mattermost-govet) (push) Failing after 4s
Server CI / Postgres (shard 2) (push) Has been skipped
Tools CI / check-style (mmgotool) (push) Failing after 3s
Server CI / Elasticsearch v8 Compatibility (push) Has been skipped
Server CI / OpenSearch v2 Compatibility (push) Has been skipped
Server CI / Postgres FIPS (shard 0) (push) Has been skipped
Tools CI / Test (mattermost-govet) (push) Failing after 3s
Server CI / Postgres FIPS (shard 1) (push) Has been skipped
Server CI / Postgres FIPS (shard 2) (push) Has been skipped
Tools CI / Test (mmgotool) (push) Failing after 3s
Server CI / Postgres FIPS (shard 3) (push) Has been skipped
Server CI / Run mmctl tests (push) Has been skipped
Server CI / Run mmctl tests (FIPS) (push) Has been skipped
Server CI / Merge Postgres Test Results (push) Has been skipped
Server CI / Merge Postgres FIPS Test Results (push) Has been skipped
Server CI / Check backport migrations (push) Has been skipped
Server CI / Check generated files (push) Has been skipped
Server CI / check-style (push) Has been skipped
Server CI / Vet API (push) Has been skipped
Server CI / Build mattermost server app (push) Has been skipped
Server CI / Server CI Complete (push) Failing after 2s
Server CI / Server CI Artifacts (push) Has been skipped
Server CI / Sentry Upload (push) Has been skipped
Server CI / Server CI Report (push) Failing after 5m1s
Server CI Nightly Race / Race Detector (push) Has been skipped
CodeQL / Analyze (go) (push) Has been cancelled
CodeQL / Analyze (javascript) (push) Has been cancelled
Webapp CI Cache Warm / Warm Webapp node_modules Cache (push) Has been cancelled
E2E CI Cache Warm / Warm E2E npm Registry Cache (push) Has been cancelled
API / build (push) Has been cancelled
Web App CI / check-types (push) Has been cancelled
Web App CI / check-lint (push) Has been cancelled
Web App CI / check-i18n (push) Has been cancelled
Web App CI / check-external-links (push) Has been cancelled
Web App CI / test (platform) (push) Has been cancelled
Web App CI / test (mattermost-redux) (push) Has been cancelled
Web App CI / test (channels shard 1/4) (push) Has been cancelled
Web App CI / test (channels shard 2/4) (push) Has been cancelled
Web App CI / test (channels shard 3/4) (push) Has been cancelled
Web App CI / test (channels shard 4/4) (push) Has been cancelled
Web App CI / upload-coverage (push) Has been cancelled
Web App CI / build (push) Has been cancelled
YAML Lint / yamllint (push) Has been cancelled
Push mirrored docker images / cd/Push mirrored docker images (push) Has been cancelled
docs-cd / Build and deploy docs (push) Has been cancelled
Docs CI / build (push) Has been cancelled
E2E Cache Warm: Cypress deps / Warm Cypress deps cache (push) Has been cancelled
Scorecards supply-chain security / Scorecard analysis (push) Has been cancelled
Server CI Weekly / Postgres FIPS (push) Has been skipped
Server CI Weekly / Run mmctl tests (FIPS) (push) Has been skipped
Server CI Weekly / Postgres with binary parameters (push) Has been skipped
Server CI Weekly / Build Environment (push) Failing after 1s
Server CI Nightly Race / Build Environment (push) Failing after 1s
Server CI Cache Warm / Warm Go Module and Build Cache (push) Failing after 2s
177 lines
4.6 KiB
Go
177 lines
4.6 KiB
Go
// Copyright (c) 2015-present Mattermost, Inc. All Rights Reserved.
|
|
// See LICENSE.txt for license information.
|
|
|
|
package commands
|
|
|
|
import (
|
|
"os"
|
|
"os/user"
|
|
"path/filepath"
|
|
"strings"
|
|
"testing"
|
|
|
|
"github.com/spf13/viper"
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
)
|
|
|
|
func TestResolveConfigFilePath(t *testing.T) {
|
|
originalUser := *currentUser
|
|
defer func() {
|
|
SetUser(&originalUser)
|
|
}()
|
|
|
|
testUser, err := user.Current()
|
|
require.NoError(t, err)
|
|
|
|
t.Run("should return the default config file location if nothing else is set", func(t *testing.T) {
|
|
tmp, err := os.MkdirTemp("", "mmctl-")
|
|
require.NoError(t, err)
|
|
t.Cleanup(func() {
|
|
err = os.RemoveAll(tmp)
|
|
require.NoError(t, err)
|
|
})
|
|
testUser.HomeDir = tmp
|
|
SetUser(testUser)
|
|
|
|
expected := filepath.Join(getDefaultConfigHomePath(), configParent, configFileName)
|
|
|
|
viper.Set("config", filepath.Join(xdgConfigHomeVar, configParent, configFileName))
|
|
|
|
p := resolveConfigFilePath()
|
|
require.Equal(t, expected, p)
|
|
})
|
|
|
|
t.Run("should return config file location from xdg environment variable", func(t *testing.T) {
|
|
tmp, err := os.MkdirTemp("", "mmctl-")
|
|
require.NoError(t, err)
|
|
t.Cleanup(func() {
|
|
err = os.RemoveAll(tmp)
|
|
require.NoError(t, err)
|
|
})
|
|
testUser.HomeDir = tmp
|
|
SetUser(testUser)
|
|
|
|
expected := filepath.Join(testUser.HomeDir, ".config", configParent, configFileName)
|
|
|
|
t.Setenv("XDG_CONFIG_HOME", filepath.Join(testUser.HomeDir, ".config"))
|
|
viper.Set("config", filepath.Join(xdgConfigHomeVar, configParent, configFileName))
|
|
|
|
p := resolveConfigFilePath()
|
|
require.Equal(t, expected, p)
|
|
})
|
|
|
|
t.Run("should return the user-defined config file path if one is set", func(t *testing.T) {
|
|
tmp, err := os.MkdirTemp("", "mmctl-")
|
|
require.NoError(t, err)
|
|
t.Cleanup(func() {
|
|
err = os.RemoveAll(tmp)
|
|
require.NoError(t, err)
|
|
})
|
|
|
|
testUser.HomeDir = tmp + "/path/should/be/ignored"
|
|
SetUser(testUser)
|
|
|
|
expected := filepath.Join(tmp, configFileName)
|
|
|
|
t.Setenv("XDG_CONFIG_HOME", "path/should/be/ignored")
|
|
viper.Set("config", expected)
|
|
|
|
p := resolveConfigFilePath()
|
|
require.Equal(t, expected, p)
|
|
})
|
|
|
|
t.Run("should resolve config file path if $HOME variable is used", func(t *testing.T) {
|
|
tmp, err := os.MkdirTemp("", "mmctl-")
|
|
require.NoError(t, err)
|
|
t.Cleanup(func() {
|
|
err = os.RemoveAll(tmp)
|
|
require.NoError(t, err)
|
|
})
|
|
|
|
testUser.HomeDir = tmp + "/path/should/be/ignored"
|
|
SetUser(testUser)
|
|
|
|
expected := filepath.Join(testUser.HomeDir, "/.config/mmctl/config")
|
|
|
|
t.Setenv("XDG_CONFIG_HOME", "path/should/be/ignored")
|
|
viper.Set("config", "$HOME/.config/mmctl/config")
|
|
|
|
p := resolveConfigFilePath()
|
|
require.Equal(t, expected, p)
|
|
})
|
|
|
|
t.Run("should create the user-defined config file path if one is set", func(t *testing.T) {
|
|
tmp, err := os.MkdirTemp("", "mmctl-")
|
|
require.NoError(t, err)
|
|
t.Cleanup(func() {
|
|
err = os.RemoveAll(tmp)
|
|
require.NoError(t, err)
|
|
})
|
|
|
|
testUser.HomeDir = tmp + "path/should/be/ignored"
|
|
SetUser(testUser)
|
|
extraDir := "extra"
|
|
|
|
expected := filepath.Join(tmp, extraDir, "config.json")
|
|
|
|
t.Setenv("XDG_CONFIG_HOME", "path/should/be/ignored")
|
|
viper.Set("config", expected)
|
|
|
|
err = SaveCredentials(Credentials{})
|
|
require.NoError(t, err)
|
|
info, err := os.Stat(expected)
|
|
require.NoError(t, err)
|
|
|
|
assert.False(t, info.IsDir())
|
|
assert.True(t, info.Name() == "config.json")
|
|
})
|
|
|
|
t.Run("should return error if the config flag is set to a directory", func(t *testing.T) {
|
|
tmp, err := os.MkdirTemp("", "mmctl-")
|
|
require.NoError(t, err)
|
|
t.Cleanup(func() {
|
|
err = os.RemoveAll(tmp)
|
|
require.NoError(t, err)
|
|
})
|
|
testUser.HomeDir = tmp + "path/should/be/ignored"
|
|
SetUser(testUser)
|
|
|
|
t.Setenv("XDG_CONFIG_HOME", "path/should/be/ignored")
|
|
viper.Set("config", tmp)
|
|
|
|
err = SaveCredentials(Credentials{})
|
|
require.Error(t, err)
|
|
require.True(t, strings.HasSuffix(err.Error(), "is a directory"))
|
|
})
|
|
}
|
|
|
|
func TestReadSecretFromFile(t *testing.T) {
|
|
f, err := os.CreateTemp(t.TempDir(), "mmctl")
|
|
require.NoError(t, err)
|
|
|
|
_, err = f.WriteString("test-pass")
|
|
require.NoError(t, err)
|
|
|
|
t.Run("password from file", func(t *testing.T) {
|
|
var pass string
|
|
err := readSecretFromFile(f.Name(), &pass)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "test-pass", pass)
|
|
})
|
|
|
|
t.Run("no file path is provided", func(t *testing.T) {
|
|
pass := "test-pass-2"
|
|
err := readSecretFromFile("", &pass)
|
|
require.NoError(t, err)
|
|
require.Equal(t, "test-pass-2", pass)
|
|
})
|
|
|
|
t.Run("nonexistent file provided", func(t *testing.T) {
|
|
var pass string
|
|
err := readSecretFromFile(filepath.Join(t.TempDir(), "bla"), &pass)
|
|
require.Error(t, err)
|
|
require.Empty(t, pass)
|
|
})
|
|
}
|