fizzy-cli 0.8.0 → 0.9.0

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)
-	}
-}

package/cmd/board_show.go

@@ -1,40 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/ui"
-	"github.com/spf13/cobra"
-)
-
-var boardShowCmd = &cobra.Command{
-	Use:   "show <board_id>",
-	Short: "Show board details",
-	Long:  `Retrieve and display detailed information about a specific board`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleShowBoardDetails(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleShowBoardDetails(cmd *cobra.Command, boardID string) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	board, err := a.Client.GetBoard(context.Background(), boardID)
-	if err != nil {
-		return fmt.Errorf("fetching board: %w", err)
-	}
-
-	return ui.DisplayBoard(cmd.OutOrStdout(), board)
-}
-
-func init() {
-	boardCmd.AddCommand(boardShowCmd)
-}

package/cmd/board_show_test.go

@@ -1,113 +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 TestBoardShowCommandSuccess(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.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.Board{
-			ID:        "board-123",
-			Name:      "Project Alpha",
-			AllAccess: true,
-			CreatedAt: "2025-01-01T00:00:00Z",
-			Creator: api.User{
-				ID:   "user-123",
-				Name: "John Doe",
-				Role: "owner",
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := boardShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleShowBoardDetails(cmd, "board-123"); err != nil {
-		t.Fatalf("handleShowBoardDetails failed: %v", err)
-	}
-}
-
-func TestBoardShowCommandNotFound(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 := boardShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowBoardDetails(cmd, "nonexistent-board")
-	if err == nil {
-		t.Errorf("expected error for board not found")
-	}
-	if err.Error() != "fetching board: unexpected status code 404: Board not found" {
-		t.Errorf("expected board not found error, got %v", err)
-	}
-}
-
-func TestBoardShowCommandAPIError(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 := boardShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowBoardDetails(cmd, "board-123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching board: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestBoardShowCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := boardShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowBoardDetails(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)
-	}
-}