diff --git a/pkg/config/config_test.go b/pkg/config/config_test.go index d15a272..84860d8 100644 --- a/pkg/config/config_test.go +++ b/pkg/config/config_test.go @@ -18,6 +18,7 @@ package config import ( "bytes" + "fmt" "io" "net/http" "os" @@ -60,6 +61,7 @@ func TestGetFilePermission(t *testing.T) { fi := mockFileInfo{mode: m} err := getFilePermission(fi) assert.Error(t, err, "Mode %v should be invalid on Windows", m) + assert.EqualError(t, err, fmt.Sprintf("config file has incorrect permission flags: %s, change the file permission either to 0444 or 0666", m.String())) } } else { validModes := []os.FileMode{0600, 0400} @@ -74,6 +76,7 @@ func TestGetFilePermission(t *testing.T) { fi := mockFileInfo{mode: m} err := getFilePermission(fi) assert.Error(t, err, "Mode %v should be invalid on UNIX", m) + assert.EqualError(t, err, fmt.Sprintf("config file has incorrect permission flags: %s, change the file permission either to 0400 or 0600", m.String())) } } } diff --git a/pkg/config/file_permission_unix.go b/pkg/config/file_permission_unix.go index d474c39..9a23077 100644 --- a/pkg/config/file_permission_unix.go +++ b/pkg/config/file_permission_unix.go @@ -27,6 +27,5 @@ func getFilePermission(fi os.FileInfo) error { if fi.Mode().Perm() == 0o600 || fi.Mode().Perm() == 0o400 { return nil } - return fmt.Errorf("config file has incorrect permission flags:%s."+ - "change the file permission either to 0400 or 0600.", fi.Mode().Perm().String()) + return fmt.Errorf("config file has incorrect permission flags: %s, change the file permission either to 0400 or 0600", fi.Mode().Perm().String()) } diff --git a/pkg/config/file_permission_windows.go b/pkg/config/file_permission_windows.go index 42b3c6b..27421ed 100644 --- a/pkg/config/file_permission_windows.go +++ b/pkg/config/file_permission_windows.go @@ -27,6 +27,5 @@ func getFilePermission(fi os.FileInfo) error { if fi.Mode().Perm() == 0666 || fi.Mode().Perm() == 0444 { return nil } - return fmt.Errorf("config file has incorrect permission flags:%s."+ - "change the file permission either to 0444 or 0666.", fi.Mode().Perm().String()) + return fmt.Errorf("config file has incorrect permission flags: %s, change the file permission either to 0444 or 0666", fi.Mode().Perm().String()) }