chore: Removed unused functions, args

api/v1alpha1/utils.go

@@ -8,7 +8,3 @@ func ptrString(s string) *string {
 func ptrBool(b bool) *bool {
 	return &b
 }
-
-func ptrInt(i int) *int {
-	return &i
-}

cmd/build_id_test.go

@@ -296,18 +296,3 @@ func TestBuildIDCmd(t *testing.T) {
 		}
 	})
 }
-
-// containsBuildID checks if a string contains a substring
-func containsBuildID(s, substr string) bool {
-	return len(s) >= len(substr) && (s == substr || len(substr) == 0 ||
-		(len(s) > len(substr) && (s[:len(substr)] == substr ||
-			s[len(s)-len(substr):] == substr ||
-			func() bool {
-				for i := 0; i <= len(s)-len(substr); i++ {
-					if s[i:i+len(substr)] == substr {
-						return true
-					}
-				}
-				return false
-			}())))
-}

pkg/blueprint/feature_evaluator_test.go

@@ -1691,7 +1691,3 @@ func writeTestFile(path, content string) error {
 	_, err = file.WriteString(content)
 	return err
 }
-
-func mkdirAll(path string) error {
-	return os.MkdirAll(path, 0755)
-}

pkg/config/config_handler_public_test.go

@@ -24,7 +24,7 @@ type SetupOptions struct {
 	ConfigStr string
 }
 
-func setupMocks(t *testing.T, opts ...*SetupOptions) *Mocks {
+func setupMocks(t *testing.T, _ ...*SetupOptions) *Mocks {
 	t.Helper()
 
 	tmpDir := t.TempDir()

pkg/generators/git_generator_test.go

@@ -14,9 +14,8 @@ import (
 // =============================================================================
 
 const (
-	gitGenTestMockGitignorePath = "mock/project/root/.gitignore"
-	gitGenTestExistingContent   = "existing content\n"
-	gitGenTestExpectedContent   = `existing content
+	gitGenTestExistingContent = "existing content\n"
+	gitGenTestExpectedContent = `existing content
 
 # managed by windsor cli
 .windsor/
@@ -29,7 +28,6 @@ contexts/**/.talos/
 contexts/**/.omni/
 contexts/**/.aws/
 contexts/**/.azure/`
-	gitGenTestExpectedPerm = fs.FileMode(0644)
 )
 
 // =============================================================================

pkg/pipelines/shims.go

@@ -2,30 +2,6 @@ package pipelines
 
 import "os"
 
-// osStat retrieves the file info for a given path
-var osStat = os.Stat
-
-// osGetenv retrieves an environment variable
-var osGetenv = os.Getenv
-
-// osSetenv sets an environment variable
-var osSetenv = os.Setenv
-
-// osReadDir reads the directory and returns directory entries
-var osReadDir = os.ReadDir
-
-// osReadFile reads the file and returns its contents
-var osReadFile = os.ReadFile
-
-// osRemoveAll removes a directory and all its contents
-var osRemoveAll = os.RemoveAll
-
-// osMkdirAll creates a directory and all its parents
-var osMkdirAll = os.MkdirAll
-
-// osWriteFile writes data to a file
-var osWriteFile = os.WriteFile
-
 // Shims provides a testable interface for system operations used by pipelines.
 // This struct-based approach allows for better isolation during testing by enabling
 // dependency injection of mock implementations for file system and environment operations.

pkg/workstation/services/shims.go

@@ -59,7 +59,3 @@ func NewShims() *Shims {
 func ptrString(s string) *string {
 	return &s
 }
-
-func ptrBool(b bool) *bool {
-	return &b
-}

pkg/workstation/services/talos_service_test.go

@@ -10,13 +10,6 @@ import (
 	"github.com/windsorcli/cli/pkg/constants"
 )
 
-// Package-level variables for mocking os functions
-var (
-	stat     = os.Stat
-	mkdir    = os.Mkdir
-	mkdirAll = os.MkdirAll
-)
-
 // =============================================================================
 // Test Setup
 // =============================================================================