fizzy-cli 0.7.0 → 0.9.0

package/cmd/card_update_test.go

@@ -1,149 +0,0 @@
-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"
-	"github.com/spf13/cobra"
-)
-
-func TestCardUpdateCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards/1" {
-			t.Errorf("expected /cards/1, got %s", r.URL.Path)
-		}
-		if r.Method != http.MethodPut {
-			t.Errorf("expected PUT, 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.Card{
-			ID:           "card-123",
-			Number:       1,
-			Title:        "Updated card title",
-			Status:       "published",
-			Description:  "Updated description",
-			Tags:         []string{"updated-tag"},
-			Golden:       false,
-			CreatedAt:    "2025-01-01T00:00:00Z",
-			LastActiveAt: "2025-01-15T10:30:00Z",
-			URL:          "https://example.com/card/1",
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := cardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Updated card title",
-		"--description", "Updated description",
-		"--status", "published",
-		"--tag-id", "updated-tag",
-	})
-
-	if err := handleUpdateCard(cmd, "1"); err != nil {
-		t.Fatalf("handleUpdateCard failed: %v", err)
-	}
-}
-
-func TestCardUpdateCommandAPIError(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 := cardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Updated card",
-	})
-
-	err := handleUpdateCard(cmd, "999")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "updating card: unexpected status code 404: Card not found" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestCardUpdateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Updated card",
-	})
-
-	err := handleUpdateCard(cmd, "1")
-	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 TestCardUpdateCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Updated card",
-	})
-
-	err := handleUpdateCard(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 TestCardUpdateCommandNoFlags(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	// Create a fresh command to avoid flag pollution from other tests
-	cmd := &cobra.Command{
-		Use:  "update <card_number>",
-		Args: cobra.ExactArgs(1),
-	}
-	cmd.Flags().String("title", "", "Card title")
-	cmd.Flags().String("description", "", "Card description")
-	cmd.Flags().String("status", "", "Card status")
-	cmd.Flags().StringSlice("tag-id", []string{}, "Tag ID")
-	cmd.Flags().String("last-active-at", "", "Last active timestamp")
-
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleUpdateCard(cmd, "1")
-	if err == nil {
-		t.Errorf("expected error when no flags are provided")
-	}
-	if err.Error() != "must provide at least one flag to update (--title, --description, --status, --tag-id, or --last-active-at)" {
-		t.Errorf("expected 'no flags' error, got %v", err)
-	}
-}

package/cmd/card_watch.go

@@ -1,46 +0,0 @@
-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.Fprintf(cmd.OutOrStdout(), "✓ Now watching card #%d\n", cardNum)
-	return nil
-}
-
-func init() {
-	cardCmd.AddCommand(cardWatchCmd)
-}

package/cmd/card_watch_test.go

@@ -1,92 +0,0 @@
-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.go

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

package/cmd/column_create.go

@@ -1,79 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strings"
-
-	"github.com/rogeriopvl/fizzy/internal/api"
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/colors"
-	"github.com/spf13/cobra"
-)
-
-func buildColorAliases() map[string]api.Color {
-	aliases := make(map[string]api.Color)
-	for _, colorDef := range colors.All {
-		aliases[strings.ToLower(colorDef.Name)] = api.Color(colorDef.CSSValue)
-	}
-	return aliases
-}
-
-func getAvailableColors() string {
-	var names []string
-	for _, colorDef := range colors.All {
-		names = append(names, strings.ToLower(colorDef.Name))
-	}
-	return strings.Join(names, ", ")
-}
-
-var columnCreateCmd = &cobra.Command{
-	Use:   "create",
-	Short: "Create a new column",
-	Long:  `Create a new column in the selected board. Color is optional.`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleCreateColumn(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleCreateColumn(cmd *cobra.Command) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		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: name,
-	}
-
-	if colorStr != "" {
-		colorAliases := buildColorAliases()
-		color, ok := colorAliases[colorStr]
-		if !ok {
-			return fmt.Errorf("invalid color '%s'. Available colors: %s", colorStr, getAvailableColors())
-		}
-		payload.Color = &color
-	}
-
-	_, err := a.Client.PostColumns(context.Background(), payload)
-	if err != nil {
-		return fmt.Errorf("creating column: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Column '%s' created successfully\n", name)
-	return nil
-}
-
-func init() {
-	columnCreateCmd.Flags().StringP("name", "n", "", "Column name (required)")
-	columnCreateCmd.MarkFlagRequired("name")
-	columnCreateCmd.Flags().String("color", "", fmt.Sprintf("Column color (optional). Available: %s", getAvailableColors()))
-
-	columnCmd.AddCommand(columnCreateCmd)
-}

package/cmd/column_create_test.go

@@ -1,178 +0,0 @@
-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 TestColumnCreateCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards/board-123/columns" {
-			t.Errorf("expected /boards/board-123/columns, 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]api.CreateColumnPayload
-		if err := json.Unmarshal(body, &payload); err != nil {
-			t.Fatalf("failed to unmarshal request body: %v", err)
-		}
-
-		columnPayload := payload["column"]
-		if columnPayload.Name != "Todo" {
-			t.Errorf("expected name 'Todo', got %s", columnPayload.Name)
-		}
-		if columnPayload.Color != nil {
-			t.Errorf("expected no color, got %v", columnPayload.Color)
-		}
-
-		w.WriteHeader(http.StatusCreated)
-		w.Header().Set("Location", "/columns/col-123")
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Todo"})
-
-	if err := handleCreateColumn(cmd); err != nil {
-		t.Fatalf("handleCreateColumn failed: %v", err)
-	}
-}
-
-func TestColumnCreateCommandWithColor(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.Method != http.MethodPost {
-			t.Errorf("expected POST, got %s", r.Method)
-		}
-
-		body, _ := io.ReadAll(r.Body)
-		var payload map[string]api.CreateColumnPayload
-		json.Unmarshal(body, &payload)
-
-		columnPayload := payload["column"]
-		if columnPayload.Name != "In Progress" {
-			t.Errorf("expected name 'In Progress', got %s", columnPayload.Name)
-		}
-		if columnPayload.Color == nil {
-			t.Error("expected color to be set")
-		}
-		if *columnPayload.Color != "var(--color-card-4)" {
-			t.Errorf("expected color 'var(--color-card-4)', got %s", *columnPayload.Color)
-		}
-
-		w.WriteHeader(http.StatusCreated)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "In Progress", "--color", "lime"})
-
-	if err := handleCreateColumn(cmd); err != nil {
-		t.Fatalf("handleCreateColumn failed: %v", err)
-	}
-}
-
-func TestColumnCreateCommandInvalidColor(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Todo", "--color", "invalid"})
-
-	err := handleCreateColumn(cmd)
-	if err == nil {
-		t.Errorf("expected error for invalid color")
-	}
-	if err.Error() != "invalid color 'invalid'. Available colors: blue, gray, tan, yellow, lime, aqua, violet, purple, pink" {
-		t.Errorf("expected invalid color error, got %v", err)
-	}
-}
-
-func TestColumnCreateCommandNoBoard(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.WriteHeader(http.StatusInternalServerError)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Todo", "--color", ""})
-
-	err := handleCreateColumn(cmd)
-	if err == nil {
-		t.Errorf("expected error when board not selected")
-	}
-	if err.Error() != "creating column: please select a board first with 'fizzy use --board <board_name>'" {
-		t.Errorf("expected 'board not selected' error, got %v", err)
-	}
-}
-
-func TestColumnCreateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := columnCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Todo", "--color", ""})
-
-	err := handleCreateColumn(cmd)
-	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 TestColumnCreateCommandAPIError(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 := columnCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Todo", "--color", ""})
-
-	err := handleCreateColumn(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "creating column: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/column_delete.go

@@ -1,40 +0,0 @@
-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)
-}