Add empty key-ids validation for bulk-delete, assert error messages in tests

Author: lechuckcaptain

cmd/key.go

@@ -145,6 +145,9 @@ var keyBulkDeleteCmd = &cobra.Command{
 	Short: "Delete multiple keys",
 	Long:  "Deletes multiple keys from the project. Requires Manage keys admin right.",
 	RunE: func(*cobra.Command, []string) error {
+		if len(bulkDeleteKeyIds) == 0 {
+			return errors.New("--key-ids must contain at least one key ID")
+		}
 		resp, err := Api.Keys().BulkDelete(projectId, bulkDeleteKeyIds)
 		if err != nil {
 			return err

cmd/key_test.go

@@ -5,6 +5,7 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
+	"strings"
 	"testing"
 
 	"github.com/spf13/cobra"
@@ -243,6 +244,8 @@ func TestKeyBulkUpdate_InvalidJSON(t *testing.T) {
 	err := rootCmd.Execute()
 	if err == nil {
 		t.Error("Expected error for invalid JSON, got nil")
+	} else if !strings.Contains(err.Error(), "invalid character") {
+		t.Errorf("Expected JSON parse error, got: %v", err)
 	}
 }
 
@@ -259,6 +262,8 @@ func TestKeyBulkCreate_InvalidJSON(t *testing.T) {
 	err := rootCmd.Execute()
 	if err == nil {
 		t.Error("Expected error for invalid JSON, got nil")
+	} else if !strings.Contains(err.Error(), "invalid character") {
+		t.Errorf("Expected JSON parse error, got: %v", err)
 	}
 }
 
@@ -275,6 +280,8 @@ func TestKeyBulkUpdate_EmptyKeys(t *testing.T) {
 	err := rootCmd.Execute()
 	if err == nil {
 		t.Error("Expected error for empty keys, got nil")
+	} else if err.Error() != "--keys must contain at least one key object" {
+		t.Errorf("Expected empty keys error, got: %v", err)
 	}
 }
 
@@ -291,5 +298,8 @@ func TestKeyBulkCreate_EmptyKeys(t *testing.T) {
 	err := rootCmd.Execute()
 	if err == nil {
 		t.Error("Expected error for empty keys, got nil")
+	} else if err.Error() != "--keys must contain at least one key object" {
+		t.Errorf("Expected empty keys error, got: %v", err)
 	}
 }
+