add config command

Author: Mehdi Cheracher

commands/config.go

@@ -0,0 +1,104 @@
+package commands
+
+import (
+	"errors"
+	"fmt"
+	"strconv"
+	"strings"
+
+	"github.com/chermehdi/egor/config"
+	"github.com/fatih/color"
+	"github.com/urfave/cli/v2"
+)
+
+func UpdateConfiguration(config *config.Config, key, value string) error {
+	lowerKey := strings.ToLower(key)
+	if lowerKey == "server.port" {
+		port, err := strconv.Atoi(value)
+		if err != nil {
+			return err
+		}
+		config.Server.Port = port
+	} else if lowerKey == "lang.default" {
+		// suppose that the language is valid (cpp, python, java ...)
+		config.Lang.Default = value
+	} else if lowerKey == "author" {
+		config.Author = value
+	} else {
+		// Unknow key
+		return errors.New(fmt.Sprintf("Unknown configuration property %s", key))
+	}
+	return nil
+}
+
+// Sets the configuration property identified by the first argument
+// To the value identified by the second argument.
+// Note: The configuration keys are not case sensitive, if a configuration key provided
+// is not recognized, an error is thrown
+func SetAction(context *cli.Context) error {
+	argLen := context.Args().Len()
+	if argLen != 2 {
+		color.Red(fmt.Sprintln("Usage egor config set key value"))
+		return errors.New(fmt.Sprintf("Expected 2 parameters, got %d", argLen))
+	}
+	key, value := context.Args().Get(0), context.Args().Get(1)
+
+	configuration, err := config.LoadDefaultConfiguration()
+	if err != nil {
+		return err
+	}
+	if err := UpdateConfiguration(configuration, key, value); err != nil {
+		return err
+	}
+
+	return config.SaveConfiguration(configuration)
+}
+
+// Gets and prints the current configuration associted to the first argument,
+// or Prints all if no argument is specified
+func GetAction(context *cli.Context) error {
+	argLen := context.Args().Len()
+	if argLen > 1 {
+		color.Red(fmt.Sprintln("Usage egor config get <key?>"))
+		return errors.New(fmt.Sprintf("Expected at most 1 parameter, got %d", argLen))
+	}
+	configuration, err := config.LoadDefaultConfiguration()
+	if err != nil {
+		return err
+	}
+	if argLen == 0 {
+		fmt.Println("Current configuration: ")
+		color.Green("server.port\t\t %d\n", configuration.Server.Port)
+		color.Green("lang.default\t\t %s\n", configuration.Lang.Default)
+		color.Green("author\t\t %s\n", configuration.Author)
+		return nil
+	} else {
+		value, err := config.GetConfigurationValue(configuration, context.Args().First())
+		if err != nil {
+			return err
+		}
+		color.Green("%s\t\t %s\n", context.Args().First(), value)
+		return nil
+	}
+}
+
+var ConfigCommand = cli.Command{
+	Name:      "config",
+	Aliases:   []string{"c"},
+	Usage:     "Read/Change global configuration parameters",
+	UsageText: "Read/Change global configuraiton parameters",
+	Subcommands: []*cli.Command{
+		{
+			Name:      "set",
+			Usage:     "set config parameter",
+			UsageText: "Sets a configuration parameter",
+			Action:    SetAction,
+		},
+		{
+			Name:      "get",
+			Usage:     "get config parameter",
+			UsageText: "Gets a configuration parameter",
+			Action:    GetAction,
+		},
+	},
+}

commands/config_test.go

@@ -0,0 +1,39 @@
+package commands
+
+import (
+	"testing"
+
+	"github.com/chermehdi/egor/config"
+	"github.com/stretchr/testify/assert"
+)
+
+// TODO(chermehdi): This probably should be a test fixture.
+func getDefaultConfiguration() *config.Config {
+	return &config.Config{
+		Server: struct {
+			Port int `yaml:"port"`
+		}{Port: 1200},
+		Lang: struct {
+			Default string `yaml:"default"`
+		}{Default: "cpp"},
+	}
+}
+
+func TestSetConfiguration(t *testing.T) {
+	config := getDefaultConfiguration()
+
+	UpdateConfiguration(config, "server.port", "1245")
+
+	assert.Equal(t, config.Server.Port, 1245)
+	assert.Equal(t, config.Lang.Default, "cpp")
+
+	UpdateConfiguration(config, "lang.default", "java")
+	assert.Equal(t, config.Lang.Default, "java")
+}
+
+func TestSetConfigurationUnknownKey(t *testing.T) {
+	config := getDefaultConfiguration()
+	err := UpdateConfiguration(config, "unkown.key", "123")
+
+	assert.Error(t, err, "Error should be returned if the key is unknown")
+}