fizzy-cli 0.3.0 → 0.5.0

package/cmd/card_watch.go

@@ -0,0 +1,46 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strconv"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var cardWatchCmd = &cobra.Command{
+	Use:   "watch <card_number>",
+	Short: "Subscribe to card notifications",
+	Long:  `Subscribe to notifications for an existing card`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleWatchCard(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleWatchCard(cmd *cobra.Command, cardNumber string) error {
+	cardNum, err := strconv.Atoi(cardNumber)
+	if err != nil {
+		return fmt.Errorf("invalid card number: %w", err)
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	_, err = a.Client.PostCardWatch(context.Background(), cardNum)
+	if err != nil {
+		return fmt.Errorf("watching card: %w", err)
+	}
+
+	fmt.Printf("✓ Now watching card #%d\n", cardNum)
+	return nil
+}
+
+func init() {
+	cardCmd.AddCommand(cardWatchCmd)
+}

package/cmd/card_watch_test.go

@@ -0,0 +1,92 @@
+package cmd
+
+import (
+	"context"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestCardWatchCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/123/watch" {
+			t.Errorf("expected /cards/123/watch, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodPost {
+			t.Errorf("expected POST, got %s", r.Method)
+		}
+
+		auth := r.Header.Get("Authorization")
+		if auth != "Bearer test-token" {
+			t.Errorf("expected Bearer test-token, got %s", auth)
+		}
+
+		w.WriteHeader(http.StatusNoContent)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := cardWatchCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleWatchCard(cmd, "123"); err != nil {
+		t.Fatalf("handleWatchCard failed: %v", err)
+	}
+}
+
+func TestCardWatchCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := cardWatchCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleWatchCard(cmd, "not-a-number")
+	if err == nil {
+		t.Errorf("expected error for invalid card number")
+	}
+	if err.Error() != "invalid card number: strconv.Atoi: parsing \"not-a-number\": invalid syntax" {
+		t.Errorf("expected invalid card number error, got %v", err)
+	}
+}
+
+func TestCardWatchCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := cardWatchCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleWatchCard(cmd, "123")
+	if err == nil {
+		t.Errorf("expected error when client not available")
+	}
+	if err.Error() != "API client not available" {
+		t.Errorf("expected 'client not available' error, got %v", err)
+	}
+}
+
+func TestCardWatchCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusInternalServerError)
+		w.Write([]byte("Internal Server Error"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := cardWatchCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleWatchCard(cmd, "123")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "watching card: unexpected status code 500: Internal Server Error" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}

package/cmd/column_create.go

@@ -11,11 +11,6 @@ import (
 	"github.com/spf13/cobra"
 )
 
-var (
-	columnName  string
-	columnColor string
-)
-
 func buildColorAliases() map[string]api.Color {
 	aliases := make(map[string]api.Color)
 	for _, colorDef := range colors.All {
@@ -49,15 +44,19 @@ func handleCreateColumn(cmd *cobra.Command) error {
 		return fmt.Errorf("API client not available")
 	}
 
+	// Read flag values directly from command
+	name, _ := cmd.Flags().GetString("name")
+	colorStr, _ := cmd.Flags().GetString("color")
+
 	payload := api.CreateColumnPayload{
-		Name: columnName,
+		Name: name,
 	}
 
-	if columnColor != "" {
+	if colorStr != "" {
 		colorAliases := buildColorAliases()
-		color, ok := colorAliases[columnColor]
+		color, ok := colorAliases[colorStr]
 		if !ok {
-			return fmt.Errorf("invalid color '%s'. Available colors: %s", columnColor, getAvailableColors())
+			return fmt.Errorf("invalid color '%s'. Available colors: %s", colorStr, getAvailableColors())
 		}
 		payload.Color = &color
 	}
@@ -67,14 +66,14 @@ func handleCreateColumn(cmd *cobra.Command) error {
 		return fmt.Errorf("creating column: %w", err)
 	}
 
-	fmt.Printf("✓ Column '%s' created successfully\n", columnName)
+	fmt.Printf("✓ Column '%s' created successfully\n", name)
 	return nil
 }
 
 func init() {
-	columnCreateCmd.Flags().StringVarP(&columnName, "name", "n", "", "Column name (required)")
+	columnCreateCmd.Flags().StringP("name", "n", "", "Column name (required)")
 	columnCreateCmd.MarkFlagRequired("name")
-	columnCreateCmd.Flags().StringVar(&columnColor, "color", "", fmt.Sprintf("Column color (optional). Available: %s", getAvailableColors()))
+	columnCreateCmd.Flags().String("color", "", fmt.Sprintf("Column color (optional). Available: %s", getAvailableColors()))
 
 	columnCmd.AddCommand(columnCreateCmd)
 }

package/cmd/column_create_test.go

@@ -57,9 +57,6 @@ func TestColumnCreateCommandSuccess(t *testing.T) {
 	cmd.SetContext(testApp.ToContext(context.Background()))
 	cmd.ParseFlags([]string{"--name", "Todo"})
 
-	columnName = "Todo"
-	columnColor = ""
-
 	if err := handleCreateColumn(cmd); err != nil {
 		t.Fatalf("handleCreateColumn failed: %v", err)
 	}
@@ -97,9 +94,6 @@ func TestColumnCreateCommandWithColor(t *testing.T) {
 	cmd.SetContext(testApp.ToContext(context.Background()))
 	cmd.ParseFlags([]string{"--name", "In Progress", "--color", "lime"})
 
-	columnName = "In Progress"
-	columnColor = "lime"
-
 	if err := handleCreateColumn(cmd); err != nil {
 		t.Fatalf("handleCreateColumn failed: %v", err)
 	}
@@ -113,9 +107,6 @@ func TestColumnCreateCommandInvalidColor(t *testing.T) {
 	cmd.SetContext(testApp.ToContext(context.Background()))
 	cmd.ParseFlags([]string{"--name", "Todo", "--color", "invalid"})
 
-	columnName = "Todo"
-	columnColor = "invalid"
-
 	err := handleCreateColumn(cmd)
 	if err == nil {
 		t.Errorf("expected error for invalid color")
@@ -136,10 +127,7 @@ func TestColumnCreateCommandNoBoard(t *testing.T) {
 
 	cmd := columnCreateCmd
 	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Todo"})
-
-	columnName = "Todo"
-	columnColor = ""
+	cmd.ParseFlags([]string{"--name", "Todo", "--color", ""})
 
 	err := handleCreateColumn(cmd)
 	if err == nil {
@@ -155,10 +143,7 @@ func TestColumnCreateCommandNoClient(t *testing.T) {
 
 	cmd := columnCreateCmd
 	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Todo"})
-
-	columnName = "Todo"
-	columnColor = ""
+	cmd.ParseFlags([]string{"--name", "Todo", "--color", ""})
 
 	err := handleCreateColumn(cmd)
 	if err == nil {
@@ -181,10 +166,7 @@ func TestColumnCreateCommandAPIError(t *testing.T) {
 
 	cmd := columnCreateCmd
 	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Todo"})
-
-	columnName = "Todo"
-	columnColor = ""
+	cmd.ParseFlags([]string{"--name", "Todo", "--color", ""})
 
 	err := handleCreateColumn(cmd)
 	if err == nil {

package/cmd/comment.go

@@ -0,0 +1,14 @@
+// Package cmd
+package cmd
+
+import "github.com/spf13/cobra"
+
+var commentCmd = &cobra.Command{
+	Use:   "comment",
+	Short: "Manage card comments",
+	Long:  `Manage comments on cards in Fizzy`,
+}
+
+func init() {
+	rootCmd.AddCommand(commentCmd)
+}

package/cmd/comment_create.go

@@ -0,0 +1,51 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strconv"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var commentCreateCmd = &cobra.Command{
+	Use:   "create <card_number>",
+	Short: "Create a new comment",
+	Long:  `Create a new comment on a card`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleCreateComment(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleCreateComment(cmd *cobra.Command, cardNumber string) error {
+	cardNum, err := strconv.Atoi(cardNumber)
+	if err != nil {
+		return fmt.Errorf("invalid card number: %w", err)
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	body, _ := cmd.Flags().GetString("body")
+
+	comment, err := a.Client.PostCardComment(context.Background(), cardNum, body)
+	if err != nil {
+		return fmt.Errorf("creating comment: %w", err)
+	}
+
+	fmt.Printf("✓ Comment created successfully (id: %s)\n", comment.ID)
+	return nil
+}
+
+func init() {
+	commentCreateCmd.Flags().StringP("body", "b", "", "Comment body (required)")
+	commentCreateCmd.MarkFlagRequired("body")
+
+	commentCmd.AddCommand(commentCreateCmd)
+}

package/cmd/comment_create_test.go

@@ -0,0 +1,129 @@
+package cmd
+
+import (
+	"context"
+	"encoding/json"
+	"io"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/api"
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestCommentCreateCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/123/comments" {
+			t.Errorf("expected /cards/123/comments, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodPost {
+			t.Errorf("expected POST, got %s", r.Method)
+		}
+
+		auth := r.Header.Get("Authorization")
+		if auth != "Bearer test-token" {
+			t.Errorf("expected Bearer test-token, got %s", auth)
+		}
+
+		if r.Header.Get("Content-Type") != "application/json" {
+			t.Errorf("expected Content-Type: application/json, got %s", r.Header.Get("Content-Type"))
+		}
+
+		body, _ := io.ReadAll(r.Body)
+		var payload map[string]map[string]string
+		if err := json.Unmarshal(body, &payload); err != nil {
+			t.Fatalf("failed to unmarshal request body: %v", err)
+		}
+
+		commentPayload := payload["comment"]
+		if commentPayload["body"] != "This is a test comment" {
+			t.Errorf("expected body 'This is a test comment', got %s", commentPayload["body"])
+		}
+
+		w.Header().Set("Content-Type", "application/json")
+		w.WriteHeader(http.StatusCreated)
+		response := api.Comment{
+			ID:        "comment-789",
+			CreatedAt: "2025-01-01T00:00:00Z",
+			Creator:   api.User{ID: "user-1", Name: "John Doe"},
+		}
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentCreateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--body", "This is a test comment",
+	})
+
+	if err := handleCreateComment(cmd, "123"); err != nil {
+		t.Fatalf("handleCreateComment failed: %v", err)
+	}
+}
+
+func TestCommentCreateCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentCreateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--body", "Test comment",
+	})
+
+	err := handleCreateComment(cmd, "not-a-number")
+	if err == nil {
+		t.Errorf("expected error for invalid card number")
+	}
+	if err.Error() != "invalid card number: strconv.Atoi: parsing \"not-a-number\": invalid syntax" {
+		t.Errorf("expected invalid card number error, got %v", err)
+	}
+}
+
+func TestCommentCreateCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentCreateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--body", "Test comment",
+	})
+
+	err := handleCreateComment(cmd, "123")
+	if err == nil {
+		t.Errorf("expected error when client not available")
+	}
+	if err.Error() != "API client not available" {
+		t.Errorf("expected 'client not available' error, got %v", err)
+	}
+}
+
+func TestCommentCreateCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusInternalServerError)
+		w.Write([]byte("Internal Server Error"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentCreateCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.ParseFlags([]string{
+		"--body", "Test comment",
+	})
+
+	err := handleCreateComment(cmd, "123")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "creating comment: unexpected status code 500: Internal Server Error" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}

package/cmd/comment_list.go

@@ -0,0 +1,51 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strconv"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/ui"
+	"github.com/spf13/cobra"
+)
+
+var commentListCmd = &cobra.Command{
+	Use:   "list <card_number>",
+	Short: "List comments on a card",
+	Long:  `Retrieve and display all comments on a card`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleListComments(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleListComments(cmd *cobra.Command, cardNumber string) error {
+	cardNum, err := strconv.Atoi(cardNumber)
+	if err != nil {
+		return fmt.Errorf("invalid card number: %w", err)
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	comments, err := a.Client.GetCardComments(context.Background(), cardNum)
+	if err != nil {
+		return fmt.Errorf("fetching comments: %w", err)
+	}
+
+	if len(comments) == 0 {
+		fmt.Println("No comments found")
+		return nil
+	}
+
+	return ui.DisplayComments(comments)
+}
+
+func init() {
+	commentCmd.AddCommand(commentListCmd)
+}

package/cmd/comment_list_test.go

@@ -0,0 +1,132 @@
+package cmd
+
+import (
+	"context"
+	"encoding/json"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/api"
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestCommentListCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/123/comments" {
+			t.Errorf("expected /cards/123/comments, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodGet {
+			t.Errorf("expected GET, got %s", r.Method)
+		}
+
+		auth := r.Header.Get("Authorization")
+		if auth != "Bearer test-token" {
+			t.Errorf("expected Bearer test-token, got %s", auth)
+		}
+
+		w.Header().Set("Content-Type", "application/json")
+		response := []api.Comment{
+			{
+				ID:        "comment-123",
+				CreatedAt: "2025-01-01T00:00:00Z",
+				UpdatedAt: "2025-01-01T00:00:00Z",
+				Creator:   api.User{ID: "user-1", Name: "John Doe"},
+			},
+			{
+				ID:        "comment-456",
+				CreatedAt: "2025-01-02T00:00:00Z",
+				UpdatedAt: "2025-01-02T00:00:00Z",
+				Creator:   api.User{ID: "user-2", Name: "Jane Doe"},
+			},
+		}
+		// Set the body field manually since it's a struct
+		response[0].Body.PlainText = "First comment"
+		response[0].Body.HTML = "<p>First comment</p>"
+		response[1].Body.PlainText = "Second comment"
+		response[1].Body.HTML = "<p>Second comment</p>"
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleListComments(cmd, "123"); err != nil {
+		t.Fatalf("handleListComments failed: %v", err)
+	}
+}
+
+func TestCommentListCommandNoComments(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Type", "application/json")
+		json.NewEncoder(w).Encode([]api.Comment{})
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleListComments(cmd, "123"); err != nil {
+		t.Fatalf("handleListComments failed: %v", err)
+	}
+}
+
+func TestCommentListCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListComments(cmd, "not-a-number")
+	if err == nil {
+		t.Errorf("expected error for invalid card number")
+	}
+	if err.Error() != "invalid card number: strconv.Atoi: parsing \"not-a-number\": invalid syntax" {
+		t.Errorf("expected invalid card number error, got %v", err)
+	}
+}
+
+func TestCommentListCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := commentListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListComments(cmd, "123")
+	if err == nil {
+		t.Errorf("expected error when client not available")
+	}
+	if err.Error() != "API client not available" {
+		t.Errorf("expected 'client not available' error, got %v", err)
+	}
+}
+
+func TestCommentListCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusInternalServerError)
+		w.Write([]byte("Internal Server Error"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := commentListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListComments(cmd, "123")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "fetching comments: unexpected status code 500: Internal Server Error" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}

package/cmd/comment_show.go

@@ -0,0 +1,46 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strconv"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/ui"
+	"github.com/spf13/cobra"
+)
+
+var commentShowCmd = &cobra.Command{
+	Use:   "show <card_number> <comment_id>",
+	Short: "Show a specific comment",
+	Long:  `Display details of a specific comment on a card`,
+	Args:  cobra.ExactArgs(2),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleShowComment(cmd, args[0], args[1]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleShowComment(cmd *cobra.Command, cardNumber, commentID string) error {
+	cardNum, err := strconv.Atoi(cardNumber)
+	if err != nil {
+		return fmt.Errorf("invalid card number: %w", err)
+	}
+
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	comment, err := a.Client.GetCardComment(context.Background(), cardNum, commentID)
+	if err != nil {
+		return fmt.Errorf("fetching comment: %w", err)
+	}
+
+	return ui.DisplayComment(comment)
+}
+
+func init() {
+	commentCmd.AddCommand(commentShowCmd)
+}