Add unit tests with dependency injection

  - Refactor CLI commands to support dependency injection via functional options pattern
  - Add DrasiClient interface to enable mocking of Management API
  - Create testutil package with mock implementations for DrasiClient, PlatformClient, and TaskOutput
  - Add comprehensive unit tests covering success/failure paths for all resource management commands
  - Fix error handling bugs: commands now properly return errors instead of printing
  - Fix output routing: use cmd.OutOrStdout() for testability
  - Individual command coverage: list 78.3%, describe 78.6%, apply 84.4%, delete 84.4%

Signed-off-by: Aman Singh <[email protected]>

Author: amansinghoriginal

.github/workflows/build-test.yml

@@ -265,7 +265,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@93397bea11091df50f3d7e59dc26a7711a8bcfbe # v4.1.0
         with:
-          go-version: '1.21'
+          go-version: '1.22'
 
       - name: Cache Go modules
         uses: actions/cache@v4
@@ -287,6 +287,39 @@ jobs:
           name: cli
           path: cli/bin
 
+  test-cli:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@692973e3d937129bcbf40652eb9f2f61becf3332 # v4.1.7
+
+      - name: Set up Go
+        uses: actions/setup-go@93397bea11091df50f3d7e59dc26a7711a8bcfbe # v4.1.0
+        with:
+          go-version: '1.22'
+
+      - name: Cache Go modules
+        uses: actions/cache@v4
+        with:
+          path: |
+            ~/go/pkg/mod
+            ~/.cache/go-build
+          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: |
+            ${{ runner.os }}-go-
+
+      - name: Run unit tests
+        working-directory: cli
+        run: |
+          echo "Running unit tests..."
+          go test -v ./...
+
+      - name: Run tests with race detector
+        working-directory: cli
+        run: |
+          echo "Running tests with race detector..."
+          go test -race ./...
+
   test-source-sdks:
     runs-on: ubuntu-latest
 

cli/.gitignore

@@ -1,2 +1,4 @@
 .idea
-bin
+bin
+*.out
+*.test

cli/cmd/apply.go

@@ -25,7 +25,29 @@ import (
 	"github.com/spf13/cobra"
 )
 
-func NewApplyCommand() *cobra.Command {
+// applyCmdOptions holds dependencies for the apply command
+type applyCmdOptions struct {
+	platformClientFactory func(namespace string) (sdk.PlatformClient, error)
+	outputFactory         func() output.TaskOutput
+}
+
+// NewApplyCommand creates the apply command with optional dependency injection
+func NewApplyCommand(opts ...*applyCmdOptions) *cobra.Command {
+	// Use the real implementation by default
+	opt := &applyCmdOptions{
+		platformClientFactory: func(namespace string) (sdk.PlatformClient, error) {
+			reg, err := registry.LoadCurrentRegistrationWithNamespace(namespace)
+			if err != nil {
+				return nil, err
+			}
+			return sdk.NewPlatformClient(reg)
+		},
+		outputFactory: output.NewTaskOutput,
+	}
+	// If options are provided (from tests), use them instead
+	if len(opts) > 0 && opts[0] != nil {
+		opt = opts[0]
+	}
 	var applyCommand = &cobra.Command{
 		Use:   "apply",
 		Short: "Create or update resources",
@@ -55,12 +77,7 @@ Usage examples:
 				return err
 			}
 
-			reg, err := registry.LoadCurrentRegistrationWithNamespace(namespace)
-			if err != nil {
-				return err
-			}
-
-			platformClient, err := sdk.NewPlatformClient(reg)
+			platformClient, err := opt.platformClientFactory(namespace)
 			if err != nil {
 				return err
 			}
@@ -71,7 +88,7 @@ Usage examples:
 			}
 			defer client.Close()
 
-			output := output.NewTaskOutput()
+			output := opt.outputFactory()
 			defer output.Close()
 
 			err = client.Apply(manifests, output)