fizzy-cli 0.7.0 → 0.9.0

package/cmd/column_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 TestColumnDeleteCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards/board-123/columns/col-456" {
-			t.Errorf("expected /boards/board-123/columns/col-456, 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, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleDeleteColumn(cmd, "col-456"); err != nil {
-		t.Fatalf("handleDeleteColumn failed: %v", err)
-	}
-}
-
-func TestColumnDeleteCommandNotFound(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.WriteHeader(http.StatusNotFound)
-		w.Write([]byte("Column not found"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteColumn(cmd, "nonexistent-col")
-	if err == nil {
-		t.Errorf("expected error for column not found")
-	}
-	if err.Error() != "deleting column: unexpected status code 404: Column not found" {
-		t.Errorf("expected column not found error, got %v", err)
-	}
-}
-
-func TestColumnDeleteCommandForbidden(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 column"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteColumn(cmd, "col-456")
-	if err == nil {
-		t.Errorf("expected error for forbidden access")
-	}
-	if err.Error() != "deleting column: unexpected status code 403: You don't have permission to delete this column" {
-		t.Errorf("expected permission error, got %v", err)
-	}
-}
-
-func TestColumnDeleteCommandAPIError(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}
-
-	cmd := columnDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteColumn(cmd, "col-456")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "deleting column: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestColumnDeleteCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := columnDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteColumn(cmd, "col-456")
-	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/column_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 columnListCmd = &cobra.Command{
-	Use:   "list",
-	Short: "List all columns",
-	Long:  `Retrieve and display all columns in the selected board`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleListColumns(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleListColumns(cmd *cobra.Command) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	columns, err := a.Client.GetColumns(context.Background())
-	if err != nil {
-		return fmt.Errorf("fetching columns: %w", err)
-	}
-
-	if len(columns) == 0 {
-		fmt.Println("No columns found")
-		return nil
-	}
-
-	return ui.DisplayColumns(columns)
-}
-
-func init() {
-	columnCmd.AddCommand(columnListCmd)
-}

package/cmd/column_list_test.go

@@ -1,138 +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 TestColumnListCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards/board-123/columns" {
-			t.Errorf("expected /boards/board-123/columns, 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)
-		}
-
-		if r.Header.Get("Accept") != "application/json" {
-			t.Errorf("expected Accept: application/json, got %s", r.Header.Get("Accept"))
-		}
-
-		w.Header().Set("Content-Type", "application/json")
-		response := []api.Column{
-			{
-				ID:   "col-123",
-				Name: "Todo",
-				Color: api.ColorObject{
-					Name:  "Blue",
-					Value: "var(--color-card-default)",
-				},
-				CreatedAt: "2025-01-01T00:00:00Z",
-			},
-			{
-				ID:   "col-456",
-				Name: "In Progress",
-				Color: api.ColorObject{
-					Name:  "Lime",
-					Value: "var(--color-card-4)",
-				},
-				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}
-
-	cmd := columnListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListColumns(cmd); err != nil {
-		t.Fatalf("handleListColumns failed: %v", err)
-	}
-}
-
-func TestColumnListCommandNoColumns(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.Column{})
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListColumns(cmd); err != nil {
-		t.Fatalf("handleListColumns failed: %v", err)
-	}
-}
-
-func TestColumnListCommandAPIError(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}
-
-	cmd := columnListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListColumns(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching columns: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestColumnListCommandNoBoard(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListColumns(cmd)
-	if err == nil {
-		t.Errorf("expected error when board not selected")
-	}
-	if err.Error() != "fetching columns: please select a board first with 'fizzy use --board <board_name>'" {
-		t.Errorf("expected 'board not selected' error, got %v", err)
-	}
-}
-
-func TestColumnListCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := columnListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListColumns(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/column_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 columnShowCmd = &cobra.Command{
-	Use:   "show <column_id>",
-	Short: "Show column details",
-	Long:  `Retrieve and display detailed information about a specific column`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleShowColumnDetails(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleShowColumnDetails(cmd *cobra.Command, columnID string) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	column, err := a.Client.GetColumn(context.Background(), columnID)
-	if err != nil {
-		return fmt.Errorf("fetching column: %w", err)
-	}
-
-	return ui.DisplayColumn(cmd.OutOrStdout(), column)
-}
-
-func init() {
-	columnCmd.AddCommand(columnShowCmd)
-}

package/cmd/column_show_test.go

@@ -1,111 +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 TestColumnShowCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/boards/board-123/columns/col-456" {
-			t.Errorf("expected /boards/board-123/columns/col-456, 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.Column{
-			ID:        "col-456",
-			Name:      "In Progress",
-			CreatedAt: "2025-01-01T00:00:00Z",
-			Color: api.ColorObject{
-				Name:  "Lime",
-				Value: api.Lime,
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleShowColumnDetails(cmd, "col-456"); err != nil {
-		t.Fatalf("handleShowColumnDetails failed: %v", err)
-	}
-}
-
-func TestColumnShowCommandNotFound(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.WriteHeader(http.StatusNotFound)
-		w.Write([]byte("Column not found"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := columnShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowColumnDetails(cmd, "nonexistent-col")
-	if err == nil {
-		t.Errorf("expected error for column not found")
-	}
-	if err.Error() != "fetching column: unexpected status code 404: Column not found" {
-		t.Errorf("expected column not found error, got %v", err)
-	}
-}
-
-func TestColumnShowCommandAPIError(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}
-
-	cmd := columnShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowColumnDetails(cmd, "col-456")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching column: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestColumnShowCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := columnShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowColumnDetails(cmd, "col-456")
-	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/column_update.go

@@ -1,67 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/api"
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var columnUpdateCmd = &cobra.Command{
-	Use:   "update <column_id>",
-	Short: "Update a column",
-	Long:  `Update column settings such as name and color`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleUpdateColumn(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleUpdateColumn(cmd *cobra.Command, columnID string) error {
-	// Check that at least one flag was explicitly set
-	if !cmd.Flags().Changed("name") && !cmd.Flags().Changed("color") {
-		return fmt.Errorf("at least one flag must be provided (--name or --color)")
-	}
-
-	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.UpdateColumnPayload{}
-
-	if cmd.Flags().Changed("name") {
-		name, _ := cmd.Flags().GetString("name")
-		payload.Name = name
-	}
-
-	if cmd.Flags().Changed("color") {
-		colorStr, _ := cmd.Flags().GetString("color")
-		colorAliases := buildColorAliases()
-		color, ok := colorAliases[colorStr]
-		if !ok {
-			return fmt.Errorf("invalid color '%s'. Available colors: %s", colorStr, getAvailableColors())
-		}
-		payload.Color = &color
-	}
-
-	err := a.Client.PutColumn(context.Background(), columnID, payload)
-	if err != nil {
-		return fmt.Errorf("updating column: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Column '%s' updated successfully\n", columnID)
-	return nil
-}
-
-func init() {
-	columnUpdateCmd.Flags().StringP("name", "n", "", "Column name")
-	columnUpdateCmd.Flags().String("color", "", fmt.Sprintf("Column color (optional). Available: %s", getAvailableColors()))
-
-	columnCmd.AddCommand(columnUpdateCmd)
-}