From 065d8e97313b9c8ffad37862665186668c88499f Mon Sep 17 00:00:00 2001
From: Chris <ccbrown112@gmail.com>
Date: Fri, 17 Nov 2017 08:27:26 -0600
Subject: make `config validate` exit with non-zero code on failure (#7855)

---
 cmd/platform/config.go          | 21 +++++++++------------
 cmd/platform/config_test.go     |  3 ++-
 cmd/platform/mattermost_test.go | 11 +----------
 3 files changed, 12 insertions(+), 23 deletions(-)

(limited to 'cmd/platform')

diff --git a/cmd/platform/config.go b/cmd/platform/config.go
index 6a3df0ade..0b3f26ee0 100644
--- a/cmd/platform/config.go
+++ b/cmd/platform/config.go
@@ -20,7 +20,8 @@ var configCmd = &cobra.Command{
 var validateConfigCmd = &cobra.Command{
 	Use:   "validate",
 	Short: "Validate config file",
-	Run:   configValidateCmdF,
+	Long:  "If the config file is valid, this command will output a success message and have a zero exit code. If it is invalid, this command will output an error and have a non-zero exit code.",
+	RunE:  configValidateCmdF,
 }
 
 func init() {
@@ -29,39 +30,35 @@ func init() {
 	)
 }
 
-func configValidateCmdF(cmd *cobra.Command, args []string) {
+func configValidateCmdF(cmd *cobra.Command, args []string) error {
 	utils.TranslationsPreInit()
 	filePath, err := cmd.Flags().GetString("config")
 	if err != nil {
-		CommandPrintErrorln(err)
-		return
+		return err
 	}
 
 	filePath = utils.FindConfigFile(filePath)
 
 	file, err := os.Open(filePath)
 	if err != nil {
-		CommandPrintErrorln(err)
-		return
+		return err
 	}
 
 	decoder := json.NewDecoder(file)
 	config := model.Config{}
 	err = decoder.Decode(&config)
 	if err != nil {
-		CommandPrintErrorln(err)
-		return
+		return err
 	}
 
 	if _, err := file.Stat(); err != nil {
-		CommandPrintErrorln(err)
-		return
+		return err
 	}
 
 	if err := config.IsValid(); err != nil {
-		CommandPrintErrorln(errors.New(utils.T(err.Id)))
-		return
+		return errors.New(utils.T(err.Id))
 	}
 
 	CommandPrettyPrintln("The document is valid")
+	return nil
 }
diff --git a/cmd/platform/config_test.go b/cmd/platform/config_test.go
index 781e40de8..f1c09c6f5 100644
--- a/cmd/platform/config_test.go
+++ b/cmd/platform/config_test.go
@@ -25,5 +25,6 @@ func TestConfigValidate(t *testing.T) {
 	config.SetDefaults()
 	require.NoError(t, ioutil.WriteFile(path, []byte(config.ToJson()), 0600))
 
-	assert.Contains(t, checkCommand(t, "--config", path, "config", "validate"), "The document is valid")
+	assert.Error(t, runCommand(t, "--config", "foo.json", "config", "validate"))
+	assert.NoError(t, runCommand(t, "--config", path, "config", "validate"))
 }
diff --git a/cmd/platform/mattermost_test.go b/cmd/platform/mattermost_test.go
index c7e1f6f2b..eba45801a 100644
--- a/cmd/platform/mattermost_test.go
+++ b/cmd/platform/mattermost_test.go
@@ -4,7 +4,6 @@
 package main
 
 import (
-	"io"
 	"io/ioutil"
 	"os"
 	"path/filepath"
@@ -25,17 +24,9 @@ func TestConfigFlag(t *testing.T) {
 	configPath := filepath.Join(dir, "foo.json")
 	require.NoError(t, ioutil.WriteFile(configPath, []byte(config.ToJson()), 0600))
 
-	os.Mkdir(filepath.Join(dir, "i18n"), 0700)
 	i18n, ok := utils.FindDir("i18n")
 	require.True(t, ok)
-	en, err := os.Open(filepath.Join(i18n, "en.json"))
-	require.NoError(t, err)
-	defer en.Close()
-	dest, err := os.OpenFile(filepath.Join(dir, "i18n", "en.json"), os.O_RDWR|os.O_CREATE|os.O_TRUNC, 0600)
-	require.NoError(t, err)
-	defer dest.Close()
-	_, err = io.Copy(dest, en)
-	require.NoError(t, err)
+	require.NoError(t, utils.CopyDir(i18n, filepath.Join(dir, "i18n")))
 
 	prevDir, err := os.Getwd()
 	require.NoError(t, err)
-- 
cgit v1.2.3-1-g7c22