fizzy-cli 0.8.0 → 1.0.0

package/cmd/account_list_test.go

@@ -1,118 +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 TestAccountListCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/my/identity" {
-			t.Errorf("expected /my/identity, 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 == "" {
-			t.Error("missing Authorization header")
-		}
-		if auth != "Bearer test-token" {
-			t.Errorf("expected Bearer test-token, got %s", auth)
-		}
-
-		w.Header().Set("Content-Type", "application/json")
-		response := api.GetMyIdentityResponse{
-			Accounts: []api.Account{
-				{
-					ID:        "account-123",
-					Name:      "Personal",
-					Slug:      "personal",
-					CreatedAt: "2025-01-01T00:00:00Z",
-				},
-				{
-					ID:        "account-456",
-					Name:      "Work",
-					Slug:      "work",
-					CreatedAt: "2025-01-02T00:00:00Z",
-				},
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := accountListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListAccounts(cmd); err != nil {
-		t.Fatalf("handleListAccounts failed: %v", err)
-	}
-}
-
-func TestAccountListCommandNoAccounts(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.Header().Set("Content-Type", "application/json")
-		response := api.GetMyIdentityResponse{Accounts: []api.Account{}}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := accountListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListAccounts(cmd); err != nil {
-		t.Fatalf("handleListAccounts failed: %v", err)
-	}
-}
-
-func TestAccountListCommandAPIError(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 := accountListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListAccounts(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching accounts: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestAccountListCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := accountListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListAccounts(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/board.go

@@ -1,49 +0,0 @@
-// Package cmd
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/ui"
-	"github.com/spf13/cobra"
-)
-
-var boardCmd = &cobra.Command{
-	Use:   "board",
-	Short: "Show the currently selected board",
-	Long: `Display the name and ID of the currently selected board.
-
-Use subcommands to list, create, or manage boards:
-  fizzy board list      List all boards
-  fizzy board create    Create a new board`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleShowBoard(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleShowBoard(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")
-	}
-
-	board, err := a.Client.GetBoard(context.Background(), a.Config.SelectedBoard)
-	if err != nil {
-		return fmt.Errorf("fetching board: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "%s (%s)\n", board.Name, ui.DisplayID(board.ID))
-	return nil
-}
-
-func init() {
-	rootCmd.AddCommand(boardCmd)
-}

package/cmd/board_create.go

@@ -1,59 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/api"
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var boardCreateCmd = &cobra.Command{
-	Use:   "create",
-	Short: "Create a new board",
-	Long:  `Create a new board in Fizzy`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleCreateBoard(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleCreateBoard(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")
-	allAccess, _ := cmd.Flags().GetBool("all-access")
-	autoPostponePeriod, _ := cmd.Flags().GetInt("auto-postpone-period")
-	publicDescription, _ := cmd.Flags().GetString("description")
-
-	payload := api.CreateBoardPayload{
-		Name:               name,
-		AllAccess:          allAccess,
-		AutoPostponePeriod: autoPostponePeriod,
-		PublicDescription:  publicDescription,
-	}
-
-	_, err := a.Client.PostBoards(context.Background(), payload)
-	if err != nil {
-		return fmt.Errorf("creating board: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Board '%s' created successfully\n", name)
-	return nil
-}
-
-func init() {
-	boardCreateCmd.Flags().StringP("name", "n", "", "Board name (required)")
-	boardCreateCmd.MarkFlagRequired("name")
-	boardCreateCmd.Flags().Bool("all-access", false, "Allow all access to the board")
-	boardCreateCmd.Flags().Int("auto-postpone-period", 0, "Auto postpone period in days")
-	boardCreateCmd.Flags().String("description", "", "Public description of the board")
-
-	boardCmd.AddCommand(boardCreateCmd)
-}

package/cmd/board_create_test.go

@@ -1,141 +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 TestBoardCreateCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards" {
-			t.Errorf("expected /boards, 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.CreateBoardPayload
-		if err := json.Unmarshal(body, &payload); err != nil {
-			t.Fatalf("failed to unmarshal request body: %v", err)
-		}
-
-		boardPayload := payload["board"]
-		if boardPayload.Name != "Test Board" {
-			t.Errorf("expected name 'Test Board', got %s", boardPayload.Name)
-		}
-
-		w.WriteHeader(http.StatusCreated)
-		w.Header().Set("Location", "/boards/board-123")
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Test Board"})
-
-	if err := handleCreateBoard(cmd); err != nil {
-		t.Fatalf("handleCreateBoard failed: %v", err)
-	}
-}
-
-func TestBoardCreateCommandWithAllFlags(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.CreateBoardPayload
-		json.Unmarshal(body, &payload)
-
-		boardPayload := payload["board"]
-		if !boardPayload.AllAccess {
-			t.Error("expected AllAccess to be true")
-		}
-		if boardPayload.AutoPostponePeriod != 7 {
-			t.Errorf("expected AutoPostponePeriod 7, got %d", boardPayload.AutoPostponePeriod)
-		}
-		if boardPayload.PublicDescription != "Team project" {
-			t.Errorf("expected description 'Team project', got %s", boardPayload.PublicDescription)
-		}
-
-		w.WriteHeader(http.StatusCreated)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--name", "Test Board",
-		"--all-access",
-		"--auto-postpone-period", "7",
-		"--description", "Team project",
-	})
-
-	if err := handleCreateBoard(cmd); err != nil {
-		t.Fatalf("handleCreateBoard failed: %v", err)
-	}
-}
-
-func TestBoardCreateCommandAPIError(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 := boardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Test Board"})
-
-	err := handleCreateBoard(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "creating board: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestBoardCreateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := boardCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Test Board"})
-
-	err := handleCreateBoard(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/board_delete.go

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

package/cmd/board_delete_test.go

@@ -1,121 +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 TestBoardDeleteCommandSuccess(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.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 := boardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleDeleteBoard(cmd, "board-123"); err != nil {
-		t.Fatalf("handleDeleteBoard failed: %v", err)
-	}
-}
-
-func TestBoardDeleteCommandNotFound(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 := boardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteBoard(cmd, "nonexistent-board")
-	if err == nil {
-		t.Errorf("expected error for board not found")
-	}
-	if err.Error() != "deleting board: unexpected status code 404: Board not found" {
-		t.Errorf("expected board not found error, got %v", err)
-	}
-}
-
-func TestBoardDeleteCommandForbidden(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 board"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteBoard(cmd, "board-123")
-	if err == nil {
-		t.Errorf("expected error for forbidden access")
-	}
-	if err.Error() != "deleting board: unexpected status code 403: You don't have permission to delete this board" {
-		t.Errorf("expected permission error, got %v", err)
-	}
-}
-
-func TestBoardDeleteCommandAPIError(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 := boardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteBoard(cmd, "board-123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "deleting board: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestBoardDeleteCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := boardDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteBoard(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/board_list.go

@@ -1,44 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/ui"
-	"github.com/spf13/cobra"
-)
-
-var boardListCmd = &cobra.Command{
-	Use:   "list",
-	Short: "List all boards",
-	Long:  `Retrieve and display all boards from Fizzy`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleListBoards(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleListBoards(cmd *cobra.Command) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	boards, err := a.Client.GetBoards(context.Background())
-	if err != nil {
-		return fmt.Errorf("fetching boards: %w", err)
-	}
-
-	if len(boards) == 0 {
-		fmt.Println("No boards found")
-		return nil
-	}
-
-	return ui.DisplayBoards(boards)
-}
-
-func init() {
-	boardCmd.AddCommand(boardListCmd)
-}

package/cmd/board_list_test.go

@@ -1,115 +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 TestBoardListCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards" {
-			t.Errorf("expected /boards, 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 == "" {
-			t.Error("missing Authorization header")
-		}
-		if auth != "Bearer test-token" {
-			t.Errorf("expected Bearer test-token, got %s", auth)
-		}
-
-		w.Header().Set("Content-Type", "application/json")
-		response := []api.Board{
-			{
-				ID:        "board-123",
-				Name:      "Project Alpha",
-				AllAccess: true,
-				CreatedAt: "2025-01-01T00:00:00Z",
-			},
-			{
-				ID:        "board-456",
-				Name:      "Project Beta",
-				AllAccess: false,
-				CreatedAt: "2025-01-02T00:00:00Z",
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListBoards(cmd); err != nil {
-		t.Fatalf("handleListBoards failed: %v", err)
-	}
-}
-
-func TestBoardListCommandNoBoards(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.Board{})
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListBoards(cmd); err != nil {
-		t.Fatalf("handleListBoards failed: %v", err)
-	}
-}
-
-func TestBoardListCommandAPIError(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 := boardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListBoards(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching boards: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestBoardListCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := boardListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListBoards(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)
-	}
-}