fizzy-cli 0.8.0 → 0.9.0

package/cmd/board_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/config"
-	"github.com/rogeriopvl/fizzy/internal/testutil"
-)
-
-func TestBoardCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards/board-123" {
-			t.Errorf("expected /boards/board-123, got %s", r.URL.Path)
-		}
-
-		w.Header().Set("Content-Type", "application/json")
-		response := api.Board{
-			ID:        "board-123",
-			Name:      "Test Board",
-			AllAccess: true,
-			CreatedAt: "2025-12-05T19:36:35.534Z",
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{
-			SelectedBoard: "board-123",
-		},
-	}
-
-	cmd := boardCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleShowBoard(cmd); err != nil {
-		t.Fatalf("handleShowBoard failed: %v", err)
-	}
-}
-
-func TestBoardCommandNoBoard(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{
-			SelectedBoard: "",
-		},
-	}
-
-	cmd := boardCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowBoard(cmd)
-	if err == nil {
-		t.Errorf("expected error when no board selected")
-	}
-	if err.Error() != "no board selected" {
-		t.Errorf("expected 'no board selected' error, got %v", err)
-	}
-}
-
-func TestBoardCommandAPIError(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,
-		Config: &config.Config{
-			SelectedBoard: "board-123",
-		},
-	}
-
-	cmd := boardCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowBoard(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-}

package/cmd/board_update.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 boardUpdateCmd = &cobra.Command{
-	Use:   "update <board_id>",
-	Short: "Update a board",
-	Long:  `Update board settings such as name, access permissions, and auto-postpone period`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleUpdateBoard(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleUpdateBoard(cmd *cobra.Command, boardID string) error {
-	// Check that at least one flag was explicitly set
-	if !cmd.Flags().Changed("name") && !cmd.Flags().Changed("all-access") &&
-		!cmd.Flags().Changed("auto-postpone-period") && !cmd.Flags().Changed("description") {
-		return fmt.Errorf("at least one flag must be provided (--name, --all-access, --auto-postpone-period, or --description)")
-	}
-
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	// Build payload only with flags that were explicitly set
-	payload := api.UpdateBoardPayload{}
-
-	if cmd.Flags().Changed("name") {
-		name, _ := cmd.Flags().GetString("name")
-		payload.Name = name
-	}
-	if cmd.Flags().Changed("all-access") {
-		allAccess, _ := cmd.Flags().GetBool("all-access")
-		payload.AllAccess = &allAccess
-	}
-	if cmd.Flags().Changed("auto-postpone-period") {
-		autoPostponePeriod, _ := cmd.Flags().GetInt("auto-postpone-period")
-		payload.AutoPostponePeriod = &autoPostponePeriod
-	}
-	if cmd.Flags().Changed("description") {
-		publicDescription, _ := cmd.Flags().GetString("description")
-		payload.PublicDescription = publicDescription
-	}
-
-	err := a.Client.PutBoard(context.Background(), boardID, payload)
-	if err != nil {
-		return fmt.Errorf("updating board: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Board '%s' updated successfully\n", boardID)
-	return nil
-}
-
-func init() {
-	boardUpdateCmd.Flags().StringP("name", "n", "", "Board name")
-	boardUpdateCmd.Flags().Bool("all-access", false, "Allow all access to the board")
-	boardUpdateCmd.Flags().Int("auto-postpone-period", 0, "Auto postpone period in days")
-	boardUpdateCmd.Flags().String("description", "", "Public description of the board")
-
-	boardCmd.AddCommand(boardUpdateCmd)
-}

package/cmd/board_update_test.go

@@ -1,233 +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"
-	"github.com/spf13/cobra"
-)
-
-func TestBoardUpdateCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards/board-123" {
-			t.Errorf("expected /boards/board-123, 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)
-		}
-
-		body, _ := io.ReadAll(r.Body)
-		var payload map[string]api.UpdateBoardPayload
-		if err := json.Unmarshal(body, &payload); err != nil {
-			t.Fatalf("failed to unmarshal request body: %v", err)
-		}
-
-		boardPayload := payload["board"]
-		if boardPayload.Name != "Updated Board" {
-			t.Errorf("expected name 'Updated Board', got %s", boardPayload.Name)
-		}
-
-		w.WriteHeader(http.StatusNoContent)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Updated Board"})
-
-	if err := handleUpdateBoard(cmd, "board-123"); err != nil {
-		t.Fatalf("handleUpdateBoard failed: %v", err)
-	}
-}
-
-func TestBoardUpdateCommandWithAllFlags(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.Method != http.MethodPut {
-			t.Errorf("expected PUT, got %s", r.Method)
-		}
-
-		body, _ := io.ReadAll(r.Body)
-		var payload map[string]api.UpdateBoardPayload
-		json.Unmarshal(body, &payload)
-
-		boardPayload := payload["board"]
-		if boardPayload.Name != "Updated Board" {
-			t.Errorf("expected name 'Updated Board', got %s", boardPayload.Name)
-		}
-		if boardPayload.AllAccess == nil || !*boardPayload.AllAccess {
-			t.Error("expected AllAccess to be pointer to true")
-		}
-		if boardPayload.AutoPostponePeriod == nil || *boardPayload.AutoPostponePeriod != 14 {
-			t.Errorf("expected AutoPostponePeriod pointer to 14, got %v", boardPayload.AutoPostponePeriod)
-		}
-		if boardPayload.PublicDescription != "Updated description" {
-			t.Errorf("expected description 'Updated description', got %s", boardPayload.PublicDescription)
-		}
-
-		w.WriteHeader(http.StatusNoContent)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--name", "Updated Board",
-		"--all-access",
-		"--auto-postpone-period", "14",
-		"--description", "Updated description",
-	})
-
-	if err := handleUpdateBoard(cmd, "board-123"); err != nil {
-		t.Fatalf("handleUpdateBoard failed: %v", err)
-	}
-}
-
-func TestBoardUpdateCommandZeroValues(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.Method != http.MethodPut {
-			t.Errorf("expected PUT, got %s", r.Method)
-		}
-
-		body, _ := io.ReadAll(r.Body)
-		var payload map[string]api.UpdateBoardPayload
-		json.Unmarshal(body, &payload)
-
-		boardPayload := payload["board"]
-		// Verify that false value is sent (not omitted)
-		if boardPayload.AllAccess == nil {
-			t.Error("expected AllAccess to be set (pointer), but got nil")
-		}
-		if *boardPayload.AllAccess != false {
-			t.Error("expected AllAccess to be false")
-		}
-		// Verify that 0 value is sent (not omitted)
-		if boardPayload.AutoPostponePeriod == nil {
-			t.Error("expected AutoPostponePeriod to be set (pointer), but got nil")
-		}
-		if *boardPayload.AutoPostponePeriod != 0 {
-			t.Errorf("expected AutoPostponePeriod to be 0, got %d", *boardPayload.AutoPostponePeriod)
-		}
-
-		w.WriteHeader(http.StatusNoContent)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--all-access=false",
-		"--auto-postpone-period", "0",
-	})
-
-	if err := handleUpdateBoard(cmd, "board-123"); err != nil {
-		t.Fatalf("handleUpdateBoard failed: %v", err)
-	}
-}
-
-func TestBoardUpdateCommandNoFlags(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 <board_id>",
-		Args: cobra.ExactArgs(1),
-	}
-	cmd.Flags().String("name", "", "Board name")
-	cmd.Flags().Bool("all-access", false, "Allow all access to the board")
-	cmd.Flags().Int("auto-postpone-period", 0, "Auto postpone period in days")
-	cmd.Flags().String("description", "", "Public description of the board")
-
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleUpdateBoard(cmd, "board-123")
-	if err == nil {
-		t.Errorf("expected error when no flags provided")
-	}
-	if err.Error() != "at least one flag must be provided (--name, --all-access, --auto-postpone-period, or --description)" {
-		t.Errorf("expected flag requirement error, got %v", err)
-	}
-}
-
-func TestBoardUpdateCommandNotFound(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.WriteHeader(http.StatusNotFound)
-		w.Write([]byte("Board not found"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Updated Board"})
-
-	err := handleUpdateBoard(cmd, "nonexistent-board")
-	if err == nil {
-		t.Errorf("expected error for board not found")
-	}
-	if err.Error() != "updating board: unexpected status code 404: Board not found" {
-		t.Errorf("expected board not found error, got %v", err)
-	}
-}
-
-func TestBoardUpdateCommandAPIError(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 := boardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Updated Board"})
-
-	err := handleUpdateBoard(cmd, "board-123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "updating board: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestBoardUpdateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := boardUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Updated Board"})
-
-	err := handleUpdateBoard(cmd, "board-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)
-	}
-}

package/cmd/card.go

@@ -1,24 +0,0 @@
-// Package cmd
-package cmd
-
-import "github.com/spf13/cobra"
-
-var cardCmd = &cobra.Command{
-	Use:   "card",
-	Short: "Manage cards",
-	Long:  `Manage cards in Fizzy`,
-}
-
-func init() {
-	rootCmd.AddCommand(cardCmd)
-
-	// Here you will define your flags and configuration settings.
-
-	// Cobra supports Persistent Flags which will work for this command
-	// and all subcommands, e.g.:
-	// cardsCmd.PersistentFlags().String("foo", "", "A help for foo")
-
-	// Cobra supports local flags which will only run when this command
-	// is called directly, e.g.:
-	// cardsCmd.Flags().BoolP("toggle", "t", false, "Help message for toggle")
-}

package/cmd/card_assign.go

@@ -1,55 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var cardAssignCmd = &cobra.Command{
-	Use:   "assign <card_number> <user_id>",
-	Short: "Assign a user to a card",
-	Long: `Assign or unassign a user to/from a card.
-
-Use "me" as the user_id to assign the card to yourself.`,
-	Args: cobra.ExactArgs(2),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleAssignCard(cmd, args[0], args[1]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleAssignCard(cmd *cobra.Command, cardNumber, userID 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")
-	}
-
-	if userID == "me" {
-		if a.Config.CurrentUserID == "" {
-			return fmt.Errorf("current user ID not available, please run 'fizzy login' first")
-		}
-		userID = a.Config.CurrentUserID
-	}
-
-	_, err = a.Client.PostCardAssignments(context.Background(), cardNum, userID)
-	if err != nil {
-		return fmt.Errorf("assigning card: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Card #%d assignment toggled for user %s\n", cardNum, userID)
-	return nil
-}
-
-func init() {
-	cardCmd.AddCommand(cardAssignCmd)
-}

package/cmd/card_assign_test.go

@@ -1,130 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"net/http"
-	"net/http/httptest"
-	"testing"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/config"
-	"github.com/rogeriopvl/fizzy/internal/testutil"
-)
-
-func TestCardAssignCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards/123/assignments" {
-			t.Errorf("expected /cards/123/assignments, 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 := cardAssignCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleAssignCard(cmd, "123", "user-id-123"); err != nil {
-		t.Fatalf("handleAssignCard failed: %v", err)
-	}
-}
-
-func TestCardAssignCommandWithMe(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards/123/assignments" {
-			t.Errorf("expected /cards/123/assignments, got %s", r.URL.Path)
-		}
-		w.WriteHeader(http.StatusNoContent)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	cfg := &config.Config{CurrentUserID: "my-user-id"}
-	testApp := &app.App{Client: client, Config: cfg}
-
-	cmd := cardAssignCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleAssignCard(cmd, "123", "me"); err != nil {
-		t.Fatalf("handleAssignCard with 'me' failed: %v", err)
-	}
-}
-
-func TestCardAssignCommandAPIError(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 := cardAssignCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleAssignCard(cmd, "999", "user-id-123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "assigning card: unexpected status code 404: Card not found" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestCardAssignCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardAssignCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleAssignCard(cmd, "not-a-number", "user-id-123")
-	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 TestCardAssignCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := cardAssignCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleAssignCard(cmd, "123", "user-id-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 TestCardAssignCommandMeWithoutUserID(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "", "test-token")
-	testApp := &app.App{Client: client, Config: &config.Config{}}
-
-	cmd := cardAssignCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleAssignCard(cmd, "123", "me")
-	if err == nil {
-		t.Errorf("expected error when using 'me' without current user ID")
-	}
-	if err.Error() != "current user ID not available, please run 'fizzy login' first" {
-		t.Errorf("expected 'current user ID not available' error, got %v", err)
-	}
-}

package/cmd/card_close.go

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