fizzy-cli 0.6.1 → 0.8.0

package/cmd/card_reaction_delete_test.go

@@ -0,0 +1,112 @@
+package cmd
+
+import (
+	"context"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestCardReactionDeleteCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/123/reactions/reaction-456" {
+			t.Errorf("expected /cards/123/reactions/reaction-456, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodDelete {
+			t.Errorf("expected DELETE, 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 := cardReactionDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleDeleteCardReaction(cmd, "123", "reaction-456"); err != nil {
+		t.Fatalf("handleDeleteCardReaction failed: %v", err)
+	}
+}
+
+func TestCardReactionDeleteCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := cardReactionDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteCardReaction(cmd, "not-a-number", "reaction-456")
+	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 TestCardReactionDeleteCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := cardReactionDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteCardReaction(cmd, "123", "reaction-456")
+	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 TestCardReactionDeleteCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("Reaction not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := cardReactionDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteCardReaction(cmd, "123", "reaction-456")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "deleting reaction: unexpected status code 404: Reaction not found" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}
+
+func TestCardReactionDeleteCommandDifferentReactionID(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/456/reactions/reaction-789" {
+			t.Errorf("expected /cards/456/reactions/reaction-789, got %s", r.URL.Path)
+		}
+		w.WriteHeader(http.StatusNoContent)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := cardReactionDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleDeleteCardReaction(cmd, "456", "reaction-789"); err != nil {
+		t.Fatalf("handleDeleteCardReaction failed: %v", err)
+	}
+}

package/cmd/card_reaction_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 cardReactionListCmd = &cobra.Command{
+	Use:   "list <card_number>",
+	Short: "List reactions on a card",
+	Long:  `Retrieve and display all reactions (boosts) on a card`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleListCardReactions(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleListCardReactions(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")
+	}
+
+	reactions, err := a.Client.GetCardReactions(context.Background(), cardNum)
+	if err != nil {
+		return fmt.Errorf("fetching reactions: %w", err)
+	}
+
+	if len(reactions) == 0 {
+		fmt.Println("No reactions found")
+		return nil
+	}
+
+	return ui.DisplayReactions(reactions)
+}
+
+func init() {
+	cardReactionCmd.AddCommand(cardReactionListCmd)
+}

package/cmd/card_reaction_list_test.go

@@ -0,0 +1,127 @@
+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 TestCardReactionListCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/cards/123/reactions" {
+			t.Errorf("expected /cards/123/reactions, 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")
+		w.WriteHeader(http.StatusOK)
+		reactions := []api.Reaction{
+			{
+				ID:      "reaction-1",
+				Content: "👍",
+				Reacter: api.User{ID: "user-1", Name: "Alice"},
+			},
+			{
+				ID:      "reaction-2",
+				Content: "🎉",
+				Reacter: api.User{ID: "user-2", Name: "Bob"},
+			},
+		}
+		json.NewEncoder(w).Encode(reactions)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := cardReactionListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleListCardReactions(cmd, "123"); err != nil {
+		t.Fatalf("handleListCardReactions failed: %v", err)
+	}
+}
+
+func TestCardReactionListCommandNoReactions(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Type", "application/json")
+		w.WriteHeader(http.StatusOK)
+		json.NewEncoder(w).Encode([]api.Reaction{})
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := cardReactionListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleListCardReactions(cmd, "123"); err != nil {
+		t.Fatalf("handleListCardReactions failed: %v", err)
+	}
+}
+
+func TestCardReactionListCommandInvalidCardNumber(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := cardReactionListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListCardReactions(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 TestCardReactionListCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := cardReactionListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListCardReactions(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 TestCardReactionListCommandAPIError(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("Card not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := cardReactionListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListCardReactions(cmd, "123")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "fetching reactions: unexpected status code 404: Card not found" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}

package/cmd/card_reopen.go

@@ -37,7 +37,7 @@ func handleReopenCard(cmd *cobra.Command, cardNumber string) error {
 		return fmt.Errorf("reopening card: %w", err)
 	}
 
-	fmt.Printf("✓ Card #%d reopened successfully\n", cardNum)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card #%d reopened successfully\n", cardNum)
 	return nil
 }
 

package/cmd/card_tag.go

@@ -42,7 +42,7 @@ func handleTagCard(cmd *cobra.Command, cardNumber, tagTitle string) error {
 		return fmt.Errorf("toggling tag on card: %w", err)
 	}
 
-	fmt.Printf("✓ Tag '%s' toggled on card #%d\n", tagTitle, cardNum)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Tag '%s' toggled on card #%d\n", tagTitle, cardNum)
 	return nil
 }
 

package/cmd/card_triage.go

@@ -37,7 +37,7 @@ func handleTriageCard(cmd *cobra.Command, cardNumber string, columnID string) er
 		return fmt.Errorf("triaging card: %w", err)
 	}
 
-	fmt.Printf("✓ Card #%d moved to column successfully\n", cardNum)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card #%d moved to column successfully\n", cardNum)
 	return nil
 }
 

package/cmd/card_ungolden.go

@@ -37,7 +37,7 @@ func handleUngoldenCard(cmd *cobra.Command, cardNumber string) error {
 		return fmt.Errorf("removing golden status: %w", err)
 	}
 
-	fmt.Printf("✓ Card #%d golden status removed\n", cardNum)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card #%d golden status removed\n", cardNum)
 	return nil
 }
 

package/cmd/card_untriage.go

@@ -37,7 +37,7 @@ func handleUntriagedCard(cmd *cobra.Command, cardNumber string) error {
 		return fmt.Errorf("sending card back to triage: %w", err)
 	}
 
-	fmt.Printf("✓ Card #%d sent back to triage successfully\n", cardNum)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card #%d sent back to triage successfully\n", cardNum)
 	return nil
 }
 

package/cmd/card_unwatch.go

@@ -37,7 +37,7 @@ func handleUnwatchCard(cmd *cobra.Command, cardNumber string) error {
 		return fmt.Errorf("unwatching card: %w", err)
 	}
 
-	fmt.Printf("✓ Stopped watching card #%d\n", cardNum)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Stopped watching card #%d\n", cardNum)
 	return nil
 }
 

package/cmd/card_update.go

@@ -67,7 +67,7 @@ func handleUpdateCard(cmd *cobra.Command, cardNumber string) error {
 		return fmt.Errorf("updating card: %w", err)
 	}
 
-	fmt.Printf("✓ Card #%d updated successfully\n", card.Number)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card #%d updated successfully\n", card.Number)
 	return nil
 }
 

package/cmd/card_watch.go

@@ -37,7 +37,7 @@ func handleWatchCard(cmd *cobra.Command, cardNumber string) error {
 		return fmt.Errorf("watching card: %w", err)
 	}
 
-	fmt.Printf("✓ Now watching card #%d\n", cardNum)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Now watching card #%d\n", cardNum)
 	return nil
 }
 

package/cmd/column_create.go

@@ -66,7 +66,7 @@ func handleCreateColumn(cmd *cobra.Command) error {
 		return fmt.Errorf("creating column: %w", err)
 	}
 
-	fmt.Printf("✓ Column '%s' created successfully\n", name)
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Column '%s' created successfully\n", name)
 	return nil
 }
 

package/cmd/column_delete.go

@@ -0,0 +1,40 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var columnDeleteCmd = &cobra.Command{
+	Use:   "delete <column_id>",
+	Short: "Delete a column",
+	Long:  `Delete a column. Only board administrators can delete columns.`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleDeleteColumn(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleDeleteColumn(cmd *cobra.Command, columnID string) error {
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	err := a.Client.DeleteColumn(context.Background(), columnID)
+	if err != nil {
+		return fmt.Errorf("deleting column: %w", err)
+	}
+
+	fmt.Fprintf(cmd.OutOrStdout(), "✓ Column '%s' deleted successfully\n", columnID)
+	return nil
+}
+
+func init() {
+	columnCmd.AddCommand(columnDeleteCmd)
+}

package/cmd/column_delete_test.go

@@ -0,0 +1,121 @@
+package cmd
+
+import (
+	"context"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestColumnDeleteCommandSuccess(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/boards/board-123/columns/col-456" {
+			t.Errorf("expected /boards/board-123/columns/col-456, got %s", r.URL.Path)
+		}
+		if r.Method != http.MethodDelete {
+			t.Errorf("expected DELETE, 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, "", "board-123", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := columnDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleDeleteColumn(cmd, "col-456"); err != nil {
+		t.Fatalf("handleDeleteColumn failed: %v", err)
+	}
+}
+
+func TestColumnDeleteCommandNotFound(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("Column not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := columnDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteColumn(cmd, "nonexistent-col")
+	if err == nil {
+		t.Errorf("expected error for column not found")
+	}
+	if err.Error() != "deleting column: unexpected status code 404: Column not found" {
+		t.Errorf("expected column not found error, got %v", err)
+	}
+}
+
+func TestColumnDeleteCommandForbidden(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusForbidden)
+		w.Write([]byte("You don't have permission to delete this column"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := columnDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteColumn(cmd, "col-456")
+	if err == nil {
+		t.Errorf("expected error for forbidden access")
+	}
+	if err.Error() != "deleting column: unexpected status code 403: You don't have permission to delete this column" {
+		t.Errorf("expected permission error, got %v", err)
+	}
+}
+
+func TestColumnDeleteCommandAPIError(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, "", "board-123", "test-token")
+	testApp := &app.App{Client: client}
+
+	cmd := columnDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteColumn(cmd, "col-456")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "deleting column: unexpected status code 500: Internal Server Error" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}
+
+func TestColumnDeleteCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := columnDeleteCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleDeleteColumn(cmd, "col-456")
+	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)
+	}
+}

package/cmd/column_show.go

@@ -0,0 +1,40 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/ui"
+	"github.com/spf13/cobra"
+)
+
+var columnShowCmd = &cobra.Command{
+	Use:   "show <column_id>",
+	Short: "Show column details",
+	Long:  `Retrieve and display detailed information about a specific column`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleShowColumnDetails(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleShowColumnDetails(cmd *cobra.Command, columnID string) error {
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	column, err := a.Client.GetColumn(context.Background(), columnID)
+	if err != nil {
+		return fmt.Errorf("fetching column: %w", err)
+	}
+
+	return ui.DisplayColumn(cmd.OutOrStdout(), column)
+}
+
+func init() {
+	columnCmd.AddCommand(columnShowCmd)
+}