fizzy-cli 0.8.0 → 0.9.0

package/cmd/card_close_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 TestCardCloseCommandSuccess(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.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 := cardCloseCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleCloseCard(cmd, "123"); err != nil {
-		t.Fatalf("handleCloseCard failed: %v", err)
-	}
-}
-
-func TestCardCloseCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardCloseCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleCloseCard(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 TestCardCloseCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardCloseCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleCloseCard(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 TestCardCloseCommandAPIError(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 := cardCloseCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleCloseCard(cmd, "123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "closing card: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/card_create.go

@@ -1,72 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/api"
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var cardCreateCmd = &cobra.Command{
-	Use:   "create",
-	Short: "Create a new card",
-	Long:  `Create a new card in the selected board`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleCreateCard(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleCreateCard(cmd *cobra.Command) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	if a.Config.SelectedBoard == "" {
-		return fmt.Errorf("no board selected")
-	}
-
-	// Read flag values directly from command
-	title, _ := cmd.Flags().GetString("title")
-	description, _ := cmd.Flags().GetString("description")
-	status, _ := cmd.Flags().GetString("status")
-	imageURL, _ := cmd.Flags().GetString("image-url")
-	tagIDs, _ := cmd.Flags().GetStringSlice("tag-id")
-	createdAt, _ := cmd.Flags().GetString("created-at")
-	lastActiveAt, _ := cmd.Flags().GetString("last-active-at")
-
-	payload := api.CreateCardPayload{
-		Title:        title,
-		Description:  description,
-		Status:       status,
-		ImageURL:     imageURL,
-		TagIDS:       tagIDs,
-		CreatedAt:    createdAt,
-		LastActiveAt: lastActiveAt,
-	}
-
-	_, err := a.Client.PostCards(context.Background(), payload)
-	if err != nil {
-		return fmt.Errorf("creating card: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card '%s' created successfully\n", title)
-	return nil
-}
-
-func init() {
-	cardCreateCmd.Flags().StringP("title", "t", "", "Card title (required)")
-	cardCreateCmd.MarkFlagRequired("title")
-	cardCreateCmd.Flags().StringP("description", "d", "", "Card description")
-	cardCreateCmd.Flags().String("status", "", "Card status")
-	cardCreateCmd.Flags().String("image-url", "", "Card image URL")
-	cardCreateCmd.Flags().StringSlice("tag-id", []string{}, "Tag ID (can be used multiple times)")
-	cardCreateCmd.Flags().String("created-at", "", "Creation timestamp")
-	cardCreateCmd.Flags().String("last-active-at", "", "Last active timestamp")
-
-	cardCmd.AddCommand(cardCreateCmd)
-}

package/cmd/card_create_test.go

@@ -1,186 +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/config"
-	"github.com/rogeriopvl/fizzy/internal/testutil"
-)
-
-func TestCardCreateCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards/board-123/cards" {
-			t.Errorf("expected /boards/board-123/cards, 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.CreateCardPayload
-		if err := json.Unmarshal(body, &payload); err != nil {
-			t.Fatalf("failed to unmarshal request body: %v", err)
-		}
-
-		cardPayload := payload["card"]
-		if cardPayload.Title != "Implement feature" {
-			t.Errorf("expected title 'Implement feature', got %s", cardPayload.Title)
-		}
-		if cardPayload.Description != "A detailed description" {
-			t.Errorf("expected description 'A detailed description', got %s", cardPayload.Description)
-		}
-
-		w.WriteHeader(http.StatusCreated)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := cardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Implement feature",
-		"--description", "A detailed description",
-	})
-
-	if err := handleCreateCard(cmd); err != nil {
-		t.Fatalf("handleCreateCard failed: %v", err)
-	}
-}
-
-func TestCardCreateCommandWithAllFields(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.CreateCardPayload
-		json.Unmarshal(body, &payload)
-
-		cardPayload := payload["card"]
-		if cardPayload.Title != "Fix bug" {
-			t.Errorf("expected title 'Fix bug', got %s", cardPayload.Title)
-		}
-		if cardPayload.Status != "in_progress" {
-			t.Errorf("expected status 'in_progress', got %s", cardPayload.Status)
-		}
-		if cardPayload.ImageURL != "https://example.com/image.jpg" {
-			t.Errorf("expected image URL 'https://example.com/image.jpg', got %s", cardPayload.ImageURL)
-		}
-		if len(cardPayload.TagIDS) != 2 || cardPayload.TagIDS[0] != "tag-1" || cardPayload.TagIDS[1] != "tag-2" {
-			t.Errorf("expected tag IDs [tag-1, tag-2], got %v", cardPayload.TagIDS)
-		}
-
-		w.WriteHeader(http.StatusCreated)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: "board-123"},
-	}
-
-	cmd := cardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Fix bug",
-		"--status", "in_progress",
-		"--image-url", "https://example.com/image.jpg",
-		"--tag-id", "tag-1",
-		"--tag-id", "tag-2",
-	})
-
-	if err := handleCreateCard(cmd); err != nil {
-		t.Fatalf("handleCreateCard failed: %v", err)
-	}
-}
-
-func TestCardCreateCommandNoBoard(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{SelectedBoard: ""},
-	}
-
-	cmd := cardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Test",
-	})
-
-	err := handleCreateCard(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 TestCardCreateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Test",
-	})
-
-	err := handleCreateCard(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 TestCardCreateCommandAPIError(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 := cardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--title", "Test",
-	})
-
-	err := handleCreateCard(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "creating card: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/card_delete.go

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

package/cmd/card_delete_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 TestCardDeleteCommand(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.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 := cardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleDeleteCard(cmd, "1"); err != nil {
-		t.Fatalf("handleDeleteCard failed: %v", err)
-	}
-}
-
-func TestCardDeleteCommandAPIError(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 := cardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteCard(cmd, "999")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "deleting card: unexpected status code 404: Card not found" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestCardDeleteCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteCard(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 TestCardDeleteCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteCard(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)
-	}
-}

package/cmd/card_golden.go

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

package/cmd/card_golden_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 TestCardGoldenCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards/123/goldness" {
-			t.Errorf("expected /cards/123/goldness, 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 := cardGoldenCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleGoldenCard(cmd, "123"); err != nil {
-		t.Fatalf("handleGoldenCard failed: %v", err)
-	}
-}
-
-func TestCardGoldenCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardGoldenCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleGoldenCard(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 TestCardGoldenCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardGoldenCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleGoldenCard(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 TestCardGoldenCommandAPIError(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 := cardGoldenCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleGoldenCard(cmd, "123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "marking card as golden: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}