fizzy-cli 0.7.0 → 0.9.0

package/cmd/card_list_test.go

@@ -1,148 +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/config"
-	"github.com/rogeriopvl/fizzy/internal/testutil"
-)
-
-func TestCardListCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards" {
-			t.Errorf("expected /cards, got %s", r.URL.Path)
-		}
-		if r.Method != http.MethodGet {
-			t.Errorf("expected GET, got %s", r.Method)
-		}
-
-		boardIDs := r.URL.Query()["board_ids[]"]
-		if len(boardIDs) == 0 || boardIDs[0] != "board-123" {
-			t.Errorf("expected board_ids[]=board-123 in query, got %v", boardIDs)
-		}
-
-		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:     "Implement feature",
-				Status:    "in_progress",
-				CreatedAt: "2025-01-01T00:00:00Z",
-			},
-			{
-				ID:        "card-456",
-				Number:    2,
-				Title:     "Fix bug",
-				Status:    "todo",
-				CreatedAt: "2025-01-02T00:00:00Z",
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := cardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListCards(cmd); err != nil {
-		t.Fatalf("handleListCards failed: %v", err)
-	}
-}
-
-func TestCardListCommandNoCards(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.Card{})
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := cardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListCards(cmd); err != nil {
-		t.Fatalf("handleListCards failed: %v", err)
-	}
-}
-
-func TestCardListCommandAPIError(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,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := cardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListCards(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching cards: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestCardListCommandNoBoard(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: ""},
-	}
-
-	cmd := cardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListCards(cmd)
-	if err == nil {
-		t.Errorf("expected error when board not selected")
-	}
-	if err.Error() != "no board selected" {
-		t.Errorf("expected 'no board selected' error, got %v", err)
-	}
-}
-
-func TestCardListCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListCards(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)
-	}
-}

package/cmd/card_not_now.go

@@ -1,46 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var cardNotNowCmd = &cobra.Command{
-	Use:   "not-now <card_number>",
-	Short: "Move a card to Not Now status",
-	Long:  `Move an existing card to the "Not Now" status`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleNotNowCard(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleNotNowCard(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.PostCardNotNow(context.Background(), cardNum)
-	if err != nil {
-		return fmt.Errorf("moving card to not now: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card #%d moved to Not Now successfully\n", cardNum)
-	return nil
-}
-
-func init() {
-	cardCmd.AddCommand(cardNotNowCmd)
-}

package/cmd/card_not_now_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 TestCardNotNowCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards/123/not_now" {
-			t.Errorf("expected /cards/123/not_now, 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 := cardNotNowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleNotNowCard(cmd, "123"); err != nil {
-		t.Fatalf("handleNotNowCard failed: %v", err)
-	}
-}
-
-func TestCardNotNowCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardNotNowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleNotNowCard(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 TestCardNotNowCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardNotNowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleNotNowCard(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 TestCardNotNowCommandAPIError(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 := cardNotNowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleNotNowCard(cmd, "123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "moving card to not now: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/card_reopen.go

@@ -1,46 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var cardReopenCmd = &cobra.Command{
-	Use:   "reopen <card_number>",
-	Short: "Reopen a card",
-	Long:  `Reopen an existing closed card`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleReopenCard(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleReopenCard(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.DeleteCardsClosure(context.Background(), cardNum)
-	if err != nil {
-		return fmt.Errorf("reopening card: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card #%d reopened successfully\n", cardNum)
-	return nil
-}
-
-func init() {
-	cardCmd.AddCommand(cardReopenCmd)
-}

package/cmd/card_reopen_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 TestCardReopenCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards/123/closure" {
-			t.Errorf("expected /cards/123/closure, 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 := cardReopenCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleReopenCard(cmd, "123"); err != nil {
-		t.Fatalf("handleReopenCard failed: %v", err)
-	}
-}
-
-func TestCardReopenCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardReopenCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleReopenCard(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 TestCardReopenCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardReopenCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleReopenCard(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 TestCardReopenCommandAPIError(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 := cardReopenCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleReopenCard(cmd, "123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "reopening card: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/card_show.go

@@ -1,46 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/ui"
-	"github.com/spf13/cobra"
-)
-
-var cardShowCmd = &cobra.Command{
-	Use:   "show <card_id>",
-	Short: "Show card details",
-	Long:  `Retrieve and display details for a specific card`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleShowCard(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleShowCard(cmd *cobra.Command, cardID string) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	cardNumber, err := strconv.Atoi(cardID)
-	if err != nil {
-		return fmt.Errorf("card ID must be a number: %w", err)
-	}
-
-	card, err := a.Client.GetCard(context.Background(), cardNumber)
-	if err != nil {
-		return fmt.Errorf("fetching card: %w", err)
-	}
-
-	return ui.DisplayCard(card)
-}
-
-func init() {
-	cardCmd.AddCommand(cardShowCmd)
-}

package/cmd/card_show_test.go

@@ -1,92 +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"
-)
-
-func TestCardShowCommand(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.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.Card{
-			ID:           "card-123",
-			Number:       1,
-			Title:        "Implement feature",
-			Status:       "in_progress",
-			Description:  "This is a test card",
-			Tags:         []string{"feature", "backend"},
-			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 := cardShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleShowCard(cmd, "1"); err != nil {
-		t.Fatalf("handleShowCard failed: %v", err)
-	}
-}
-
-func TestCardShowCommandAPIError(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 := cardShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowCard(cmd, "999")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching card: unexpected status code 404: Card not found" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestCardShowCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowCard(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)
-	}
-}

package/cmd/card_tag.go

@@ -1,51 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-	"strings"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var cardTagCmd = &cobra.Command{
-	Use:   "tag <card_number> <tag_title>",
-	Short: "Toggle a tag on or off for a card",
-	Long: `Toggle a tag on or off for a card. If the tag doesn't exist, it will be created.
-
-The tag title can be specified with or without a leading # symbol.`,
-	Args: cobra.ExactArgs(2),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleTagCard(cmd, args[0], args[1]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleTagCard(cmd *cobra.Command, cardNumber, tagTitle 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")
-	}
-
-	tagTitle = strings.TrimPrefix(tagTitle, "#")
-
-	_, err = a.Client.PostCardTagging(context.Background(), cardNum, tagTitle)
-	if err != nil {
-		return fmt.Errorf("toggling tag on card: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Tag '%s' toggled on card #%d\n", tagTitle, cardNum)
-	return nil
-}
-
-func init() {
-	cardCmd.AddCommand(cardTagCmd)
-}