Taskfile.yaml

@@ -24,8 +24,6 @@ tasks:
 
     lint: golangci-lint run
 
-    test: go test ./...
-
     build: go build -ldflags "-s -w" -o "dodo"
 
     build-all:

config/cli_test.go

@@ -8,7 +8,6 @@ import (
 	"time"
 
 	"github.com/aykhans/dodo/types"
-	"github.com/aykhans/dodo/utils"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -43,13 +42,13 @@ func TestReadCLI(t *testing.T) {
 			expectFile:  "/path/to/config.json",
 			expectError: false,
 			expectedConfig: &Config{
-				Method:       utils.ToPtr("POST"),
+				Method:       stringPtr("POST"),
 				URL:          &types.RequestURL{},
-				DodosCount:   utils.ToPtr[uint](10),
-				RequestCount: utils.ToPtr[uint](1000),
+				DodosCount:   uintPtr(10),
+				RequestCount: uintPtr(1000),
 				Duration:     &types.Duration{Duration: 3 * time.Minute},
 				Timeout:      &types.Timeout{Duration: 3 * time.Second},
-				Yes:          utils.ToPtr(true),
+				Yes:          boolPtr(true),
 			},
 		},
 		{
@@ -61,6 +60,10 @@ func TestReadCLI(t *testing.T) {
 		},
 	}
 
+	// Save original command-line arguments
+	origArgs := os.Args
+	origFlagCommandLine := flag.CommandLine
+
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
 			// Reset flag.CommandLine to its original state
@@ -82,6 +85,9 @@ func TestReadCLI(t *testing.T) {
 			// Call the function being tested
 			file, err := config.ReadCLI()
 
+			// Reset os.Args after test
+			os.Args = origArgs
+
 			// Assert expected results
 			assert.Equal(t, tt.expectFile, file)
 			if tt.expectError {
@@ -120,6 +126,9 @@ func TestReadCLI(t *testing.T) {
 			}
 		})
 	}
+
+	// Restore original flag.CommandLine
+	flag.CommandLine = origFlagCommandLine
 }
 
 // Skip the prompt tests as they require interactive input/output handling
@@ -157,3 +166,16 @@ func TestCLIYesOrNoReaderBasic(t *testing.T) {
 	// Default value should be returned
 	assert.True(t, result)
 }
+
+// Helper types and functions for testing
+func stringPtr(s string) *string {
+	return &s
+}
+
+func uintPtr(u uint) *uint {
+	return &u
+}
+
+func boolPtr(b bool) *bool {
+	return &b
+}

config/config_test.go

@@ -3,7 +3,6 @@ package config
 import (
 	"net/url"
 	"os"
-	"slices"
 	"testing"
 	"time"
 
@@ -416,11 +415,11 @@ func TestMergeConfig(t *testing.T) {
 				RequestCount: utils.ToPtr(*baseConfig.RequestCount),
 				Duration:     &types.Duration{Duration: baseConfig.Duration.Duration},
 				Yes:          utils.ToPtr(*baseConfig.Yes),
-				Params:       slices.Clone(baseConfig.Params),
-				Headers:      slices.Clone(baseConfig.Headers),
-				Cookies:      slices.Clone(baseConfig.Cookies),
-				Body:         slices.Clone(baseConfig.Body),
-				Proxies:      slices.Clone(baseConfig.Proxies),
+				Params:       append(types.Params{}, baseConfig.Params...),
+				Headers:      append(types.Headers{}, baseConfig.Headers...),
+				Cookies:      append(types.Cookies{}, baseConfig.Cookies...),
+				Body:         append(types.Body{}, baseConfig.Body...),
+				Proxies:      append(types.Proxies{}, baseConfig.Proxies...),
 			}
 
 			// Call the function being tested