ci: complete tests

add more tests for cmd
add test for style handlers
add test for main.go
update go-testcoverag config

Author: xenOs76

.testcoverage.yml

@@ -3,10 +3,8 @@ profile: cover.out
 threshold:
   file: 70
   package: 80
-  total: 80
+  total: 85
 
 exclude:
   paths:
-    - internal/style/style_handlers.go
     - main\.go$
-    - cmd/main\.go$

cmd/config_test.go

@@ -12,7 +12,7 @@ import (
 
 var emptyString = ""
 
-func TestNewHTTPSWrenchConfig(t *testing.T) {
+func TestConfig_NewHTTPSWrenchConfig(t *testing.T) {
 	t.Run("new HTTPSWrenchConfig", func(t *testing.T) {
 		var mc requests.RequestsMetaConfig
 

cmd/man_test.go

@@ -0,0 +1,27 @@
+package cmd
+
+import (
+	"bytes"
+	"testing"
+
+	"github.com/stretchr/testify/require"
+)
+
+func TestMan(t *testing.T) {
+	t.Run("Run manCmd", func(t *testing.T) {
+		destDir := t.TempDir()
+		manBuf := new(bytes.Buffer)
+		rootCmd.SetOut(manBuf)
+		rootCmd.SetErr(manBuf)
+		rootCmd.SetArgs([]string{"man", "--dest-dir", destDir})
+
+		err := rootCmd.Execute()
+		require.NoError(t, err)
+
+		rootCmd.SetArgs([]string{"man", "--dest-dir", "fake-dir"})
+		errWrongDir := rootCmd.Execute()
+		require.NoError(t, errWrongDir)
+		// WARN stdout does not get into the buffer
+		// require.Contains(t, manBuf.String(), "no such file or directory--- FAIL")
+	})
+}

cmd/root.go

@@ -80,11 +80,13 @@ https://github.com/xenOs76/https-wrench`,
 	},
 }
 
-func Execute() {
+func Execute() error {
 	err := rootCmd.Execute()
 	if err != nil {
-		return
+		return err
 	}
+
+	return nil
 }
 
 func init() {

cmd/root_test.go

@@ -2,35 +2,105 @@ package cmd
 
 import (
 	"bytes"
+	"crypto/x509"
 	_ "embed"
 	"testing"
 
 	_ "github.com/breml/rootcerts"
+	"github.com/google/go-cmp/cmp"
 	"github.com/stretchr/testify/require"
+	"github.com/xenos76/https-wrench/internal/requests"
 )
 
+func TestRootCmd_LoadConfig(t *testing.T) {
+	t.Run("LoadConfig no config file", func(t *testing.T) {
+		var mc requests.RequestsMetaConfig
+
+		config, err := LoadConfig()
+		require.NoError(t, err)
+		require.False(t, config.Debug)
+		require.False(t, config.Verbose)
+		require.Empty(t, config.CaBundle)
+
+		if diff := cmp.Diff(mc, config.RequestsMetaConfig); diff != "" {
+			t.Errorf(
+				"NewHTTPSWrenchConfig: RequestsMetaConfig mismatch (-want +got):\n%s",
+				diff,
+			)
+		}
+	})
+
+	t.Run("LoadConfig embedded config file", func(t *testing.T) {
+		var expectedCaCertsPool *x509.CertPool
+
+		var expectedRequestsConfigs []requests.RequestConfig
+
+		cfgFile = "./embedded/config-example.yaml"
+
+		initConfig()
+
+		config, err := LoadConfig()
+		require.NoError(t, err)
+		require.False(t, config.Debug)
+		require.True(t, config.Verbose)
+		require.Empty(t, config.CaBundle)
+
+		// testing mapstructure squash/embedding of requests.RequestsMetaConfig
+		// into HTTPSWrenchConfig
+		require.False(t, config.RequestDebug)
+		require.False(t, config.RequestVerbose)
+		require.IsType(t, expectedCaCertsPool, config.CACertsPool)
+		require.IsType(t, expectedRequestsConfigs, config.Requests)
+
+		// testing against the current values of the embedded config
+		require.Equal(t, "httpBunComGet", config.Requests[0].Name)
+		require.Equal(t, "https://cat.httpbun.com:443", config.Requests[0].TransportOverrideURL)
+	})
+}
+
+func TestRootCmd_Execute(t *testing.T) {
+	t.Run("Execute empty config", func(t *testing.T) {
+		cfgFile = ""
+
+		initConfig()
+
+		_, err := LoadConfig()
+
+		require.NoError(t, err)
+		Execute()
+	})
+}
+
 func TestRootCmd(t *testing.T) {
 	tests := []struct {
 		name        string
 		args        []string
 		expectError bool
 		expected    []string
 	}{
+		// {
+		// 	name:        "no args",
+		// 	args:        []string{},
+		// 	expectError: false,
+		// 	expected: []string{
+		// 		"HTTPS Wrench",
+		// 		"Usage:",
+		// 		"Available Commands:",
+		// 		"Flags:",
+		// 		"certinfo",
+		// 		"requests",
+		// 		"--config",
+		// 		"--version",
+		// 		"--help",
+		// 	},
+		// },
+		//
 		{
-			name:        "no args",
-			args:        []string{},
+			name:        "config flag valid arg",
+			args:        []string{"--config", "./embedded/config-example.yaml"},
 			expectError: false,
-			expected: []string{
-				"HTTPS Wrench",
-				"Usage:",
-				"Available Commands:",
-				"Flags:",
-				"certinfo",
-				"requests",
-				"--config",
-				"--version",
-				"--help",
-			},
+			// Unable to intercept the output
+			expected: []string{},
 		},
 
 		{
@@ -39,13 +109,7 @@ func TestRootCmd(t *testing.T) {
 			expectError: true,
 			expected:    []string{"flag needs an argument: --config"},
 		},
-		{
-			name:        "config flag valid arg",
-			args:        []string{"--config", "./embedded/config-example.yaml"},
-			expectError: false,
-			// Unable to intercept the output
-			expected: []string{},
-		},
+
 		{
 			name:        "version",
 			args:        []string{"--version"},