feat: add cmd test

cmd/cosmos-transactions-bot.go

@@ -5,7 +5,6 @@ import (
 	configPkg "main/pkg/config"
 	"main/pkg/fs"
 	loggerPkg "main/pkg/logger"
-	"os"
 
 	"github.com/spf13/cobra"
 )
@@ -14,39 +13,18 @@ var (
 	version = "unknown"
 )
 
-type OsFS struct {
-}
-
-func (fs *OsFS) ReadFile(name string) ([]byte, error) {
-	return os.ReadFile(name)
-}
-
-func (fs *OsFS) Create(path string) (fs.File, error) {
-	return os.Create(path)
-}
-
 func ExecuteMain(configPath string) {
-	filesystem := &OsFS{}
-	config, err := configPkg.GetConfig(configPath, filesystem)
-	if err != nil {
-		loggerPkg.GetDefaultLogger().Fatal().Err(err).Msg("Could not load config")
-	}
-	warnings := config.DisplayWarnings()
-
-	for _, warning := range warnings {
-		warning.Log(loggerPkg.GetDefaultLogger())
-	}
-
-	app := pkg.NewApp(config, version, filesystem)
+	filesystem := &fs.OsFS{}
+	app := pkg.NewApp(filesystem, configPath, version)
 	app.Start()
 }
 
 func ExecuteValidateConfig(configPath string) {
-	filesystem := &OsFS{}
+	filesystem := &fs.OsFS{}
 
 	config, err := configPkg.GetConfig(configPath, filesystem)
 	if err != nil {
-		loggerPkg.GetDefaultLogger().Fatal().Err(err).Msg("Could not load config!")
+		loggerPkg.GetDefaultLogger().Panic().Err(err).Msg("Could not load config!")
 	}
 
 	if warnings := config.DisplayWarnings(); len(warnings) > 0 {
@@ -80,18 +58,14 @@ func main() {
 	}
 
 	rootCmd.PersistentFlags().StringVar(&ConfigPath, "config", "", "Config file path")
-	if err := rootCmd.MarkPersistentFlagRequired("config"); err != nil {
-		loggerPkg.GetDefaultLogger().Fatal().Err(err).Msg("Could not set flag as required")
-	}
+	_ = rootCmd.MarkPersistentFlagRequired("config")
 
 	validateConfigCmd.PersistentFlags().StringVar(&ConfigPath, "config", "", "Config file path")
-	if err := validateConfigCmd.MarkPersistentFlagRequired("config"); err != nil {
-		loggerPkg.GetDefaultLogger().Fatal().Err(err).Msg("Could not set flag as required")
-	}
+	_ = validateConfigCmd.MarkPersistentFlagRequired("config")
 
 	rootCmd.AddCommand(validateConfigCmd)
 
 	if err := rootCmd.Execute(); err != nil {
-		loggerPkg.GetDefaultLogger().Fatal().Err(err).Msg("Could not start application")
+		loggerPkg.GetDefaultLogger().Panic().Err(err).Msg("Could not start application")
 	}
 }

cmd/cosmos-transactions-bot_test.go

@@ -0,0 +1,88 @@
+package main
+
+import (
+	"os"
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+//nolint:paralleltest // disabled
+func TestValidateConfigNoConfigProvided(t *testing.T) {
+	defer func() {
+		if r := recover(); r == nil {
+			require.Fail(t, "Expected to have a panic here!")
+		}
+	}()
+
+	os.Args = []string{"cmd", "validate-config"}
+	main()
+	assert.True(t, true)
+}
+
+//nolint:paralleltest // disabled
+func TestValidateConfigFailedToLoad(t *testing.T) {
+	defer func() {
+		if r := recover(); r == nil {
+			require.Fail(t, "Expected to have a panic here!")
+		}
+	}()
+
+	os.Args = []string{"cmd", "validate-config", "--config", "../assets/config-not-found.toml"}
+	main()
+	assert.True(t, true)
+}
+
+//nolint:paralleltest // disabled
+func TestValidateConfigInvalid(t *testing.T) {
+	defer func() {
+		if r := recover(); r == nil {
+			require.Fail(t, "Expected to have a panic here!")
+		}
+	}()
+
+	os.Args = []string{"cmd", "validate-config", "--config", "../assets/invalid-timezone.toml"}
+	main()
+	assert.True(t, true)
+}
+
+//nolint:paralleltest // disabled
+func TestValidateConfigWithWarnings(t *testing.T) {
+	os.Args = []string{"cmd", "validate-config", "--config", "../assets/valid-unused-chain.toml"}
+	main()
+	assert.True(t, true)
+}
+
+//nolint:paralleltest // disabled
+func TestValidateConfigValid(t *testing.T) {
+	os.Args = []string{"cmd", "validate-config", "--config", "../assets/valid.toml"}
+	main()
+	assert.True(t, true)
+}
+
+//nolint:paralleltest // disabled
+func TestStartNoConfigProvided(t *testing.T) {
+	defer func() {
+		if r := recover(); r == nil {
+			require.Fail(t, "Expected to have a panic here!")
+		}
+	}()
+
+	os.Args = []string{"cmd"}
+	main()
+	assert.True(t, true)
+}
+
+//nolint:paralleltest // disabled
+func TestStartConfigProvided(t *testing.T) {
+	defer func() {
+		if r := recover(); r == nil {
+			require.Fail(t, "Expected to have a panic here!")
+		}
+	}()
+
+	os.Args = []string{"cmd", "--config", "../assets/invalid-timezone.toml"}
+	main()
+	assert.True(t, true)
+}

pkg/app.go

@@ -8,7 +8,7 @@ import (
 	"syscall"
 
 	"main/pkg/alias_manager"
-	"main/pkg/config"
+	configPkg "main/pkg/config"
 	"main/pkg/data_fetcher"
 	filtererPkg "main/pkg/filterer"
 	loggerPkg "main/pkg/logger"
@@ -21,7 +21,7 @@ import (
 
 type App struct {
 	Logger         zerolog.Logger
-	Config         *config.AppConfig
+	Config         *configPkg.AppConfig
 	Chains         []*configTypes.Chain
 	NodesManager   *nodesManagerPkg.NodesManager
 	Reporters      reportersPkg.Reporters
@@ -32,9 +32,19 @@ type App struct {
 	Version string
 }
 
-func NewApp(config *config.AppConfig, version string, fs fsPkg.FS) *App {
+func NewApp(filesystem fsPkg.FS, configPath string, version string) *App {
+	config, err := configPkg.GetConfig(configPath, filesystem)
+	if err != nil {
+		loggerPkg.GetDefaultLogger().Panic().Err(err).Msg("Could not load config")
+	}
+	warnings := config.DisplayWarnings()
+
+	for _, warning := range warnings {
+		warning.Log(loggerPkg.GetDefaultLogger())
+	}
+
 	logger := loggerPkg.GetLogger(config.LogConfig)
-	aliasManager := alias_manager.NewAliasManager(logger, config, fs)
+	aliasManager := alias_manager.NewAliasManager(logger, config, filesystem)
 	aliasManager.Load()
 
 	metricsManager := metricsPkg.NewManager(logger, config.Metrics)

pkg/fs/os_fs.go

@@ -0,0 +1,14 @@
+package fs
+
+import "os"
+
+type OsFS struct {
+}
+
+func (fs *OsFS) ReadFile(name string) ([]byte, error) {
+	return os.ReadFile(name)
+}
+
+func (fs *OsFS) Create(path string) (File, error) {
+	return os.Create(path)
+}

pkg/fs/os_fs_test.go

@@ -0,0 +1,15 @@
+package fs
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/require"
+)
+
+func TestOsFsCreate(t *testing.T) {
+	t.Parallel()
+
+	fs := &OsFS{}
+	_, err := fs.Create("/tmp/file.txt")
+	require.NoError(t, err)
+}