Merge branch 'master' into jonathan/4875-staking-test-use-simapp

.github/CODEOWNERS

@@ -1,4 +1,4 @@
 # CODEOWNERS: https://help.github.com/articles/about-codeowners/
 
 # Primary repo maintainers
-*       @alexanderbez @jackzampolin @alessio @fedekunze 
+*       @alexanderbez @alessio @fedekunze @nylira @hschoenburg

version/command.go

@@ -2,7 +2,6 @@ package version
 
 import (
 	"encoding/json"
-	"fmt"
 
 	"github.com/spf13/cobra"
 	"github.com/spf13/viper"
@@ -21,29 +20,31 @@ func init() {
 var Cmd = &cobra.Command{
 	Use:   "version",
 	Short: "Print the app version",
-	RunE: func(_ *cobra.Command, _ []string) error {
-		verInfo := NewInfo()
+	RunE:  runVersionCmd,
+}
 
-		if !viper.GetBool(flagLong) {
-			fmt.Println(verInfo.Version)
-			return nil
-		}
+func runVersionCmd(cmd *cobra.Command, args []string) error {
+	verInfo := NewInfo()
 
-		var bz []byte
-		var err error
+	if !viper.GetBool(flagLong) {
+		cmd.Println(verInfo.Version)
+		return nil
+	}
 
-		switch viper.GetString(cli.OutputFlag) {
-		case "json":
-			bz, err = json.Marshal(verInfo)
-		default:
-			bz, err = yaml.Marshal(&verInfo)
-		}
+	var bz []byte
+	var err error
 
-		if err != nil {
-			return err
-		}
+	switch viper.GetString(cli.OutputFlag) {
+	case "json":
+		bz, err = json.Marshal(verInfo)
+	default:
+		bz, err = yaml.Marshal(&verInfo)
+	}
 
-		_, err = fmt.Println(string(bz))
+	if err != nil {
 		return err
-	},
+	}
+
+	cmd.Println(string(bz))
+	return nil
 }

version/version_test.go

@@ -0,0 +1,60 @@
+package version
+
+import (
+	"encoding/json"
+	"fmt"
+	"runtime"
+	"testing"
+
+	"github.com/spf13/viper"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+	"github.com/tendermint/tendermint/libs/cli"
+
+	"github.com/cosmos/cosmos-sdk/tests"
+)
+
+func TestNewInfo(t *testing.T) {
+	info := NewInfo()
+	want := fmt.Sprintf(`: 
+git commit: 
+build tags: 
+%s`, fmt.Sprintf("go version %s %s/%s", runtime.Version(), runtime.GOOS, runtime.GOARCH))
+	require.Equal(t, want, info.String())
+}
+
+func TestInfo_String(t *testing.T) {
+	info := Info{
+		Name:       "testapp",
+		ServerName: "testappd",
+		ClientName: "testappcli",
+		Version:    "1.0.0",
+		GitCommit:  "1b78457135a4104bc3af97f20654d49e2ea87454",
+		BuildTags:  "netgo,ledger",
+		GoVersion:  "go version go1.14 linux/amd64",
+	}
+	want := fmt.Sprintf(`testapp: 1.0.0
+git commit: 1b78457135a4104bc3af97f20654d49e2ea87454
+build tags: netgo,ledger
+go version go1.14 linux/amd64`)
+	require.Equal(t, want, info.String())
+}
+
+func Test_runVersionCmd(t *testing.T) {
+	require.NotNil(t, Cmd)
+	_, mockOut, _ := tests.ApplyMockIO(Cmd)
+
+	viper.Set(cli.OutputFlag, "")
+	viper.Set(flagLong, false)
+	require.NoError(t, runVersionCmd(Cmd, nil))
+	assert.Equal(t, "\n", mockOut.String())
+	mockOut.Reset()
+
+	viper.Set(cli.OutputFlag, "json")
+	viper.Set(flagLong, true)
+	info := NewInfo()
+	stringInfo, err := json.Marshal(info)
+	require.NoError(t, err)
+	require.NoError(t, runVersionCmd(Cmd, nil))
+	assert.Equal(t, string(stringInfo)+"\n", mockOut.String())
+}