fizzy-cli 0.8.0 → 0.9.0

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

package/cmd/column_update_test.go

@@ -1,198 +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 newTestUpdateColumnCmd() *cobra.Command {
-	cmd := &cobra.Command{
-		Use:  "update <column_id>",
-		Args: cobra.ExactArgs(1),
-	}
-	cmd.Flags().StringP("name", "n", "", "Column name")
-	cmd.Flags().String("color", "", "Column color")
-	return cmd
-}
-
-func TestColumnUpdateCommandSuccess(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.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.UpdateColumnPayload
-		if err := json.Unmarshal(body, &payload); err != nil {
-			t.Fatalf("failed to unmarshal request body: %v", err)
-		}
-
-		columnPayload := payload["column"]
-		if columnPayload.Name != "Updated Column" {
-			t.Errorf("expected name 'Updated Column', got %s", columnPayload.Name)
-		}
-
-		w.WriteHeader(http.StatusNoContent)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := newTestUpdateColumnCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Updated Column"})
-
-	if err := handleUpdateColumn(cmd, "col-456"); err != nil {
-		t.Fatalf("handleUpdateColumn failed: %v", err)
-	}
-}
-
-func TestColumnUpdateCommandWithColor(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.UpdateColumnPayload
-		json.Unmarshal(body, &payload)
-
-		columnPayload := payload["column"]
-		if columnPayload.Name != "Progress" {
-			t.Errorf("expected name 'Progress', got %s", columnPayload.Name)
-		}
-		if columnPayload.Color == nil {
-			t.Error("expected Color to be set")
-		} else if *columnPayload.Color != api.Lime {
-			t.Errorf("expected color Lime, got %s", *columnPayload.Color)
-		}
-
-		w.WriteHeader(http.StatusNoContent)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := newTestUpdateColumnCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Progress", "--color", "lime"})
-
-	if err := handleUpdateColumn(cmd, "col-456"); err != nil {
-		t.Fatalf("handleUpdateColumn failed: %v", err)
-	}
-}
-
-func TestColumnUpdateCommandInvalidColor(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := newTestUpdateColumnCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--color", "invalid-color"})
-
-	err := handleUpdateColumn(cmd, "col-456")
-	if err == nil {
-		t.Errorf("expected error for invalid color")
-	}
-	errMsg := err.Error()
-	if errMsg != "invalid color 'invalid-color'. Available colors: blue, gray, tan, yellow, lime, aqua, violet, purple, pink" {
-		t.Errorf("expected invalid color error, got %v", err)
-	}
-}
-
-func TestColumnUpdateCommandNoFlags(t *testing.T) {
-	client := testutil.NewTestClient("http://localhost", "", "board-123", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := newTestUpdateColumnCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleUpdateColumn(cmd, "col-456")
-	if err == nil {
-		t.Errorf("expected error when no flags provided")
-	}
-	if err.Error() != "at least one flag must be provided (--name or --color)" {
-		t.Errorf("expected flag requirement error, got %v", err)
-	}
-}
-
-func TestColumnUpdateCommandNotFound(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 := newTestUpdateColumnCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Updated Column"})
-
-	err := handleUpdateColumn(cmd, "nonexistent-col")
-	if err == nil {
-		t.Errorf("expected error for column not found")
-	}
-	if err.Error() != "updating column: unexpected status code 404: Column not found" {
-		t.Errorf("expected column not found error, got %v", err)
-	}
-}
-
-func TestColumnUpdateCommandAPIError(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 := newTestUpdateColumnCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Updated Column"})
-
-	err := handleUpdateColumn(cmd, "col-456")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "updating column: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestColumnUpdateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := newTestUpdateColumnCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--name", "Updated Column"})
-
-	err := handleUpdateColumn(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/comment.go

@@ -1,14 +0,0 @@
-// Package cmd
-package cmd
-
-import "github.com/spf13/cobra"
-
-var commentCmd = &cobra.Command{
-	Use:   "comment",
-	Short: "Manage card comments",
-	Long:  `Manage comments on cards in Fizzy`,
-}
-
-func init() {
-	rootCmd.AddCommand(commentCmd)
-}

package/cmd/comment_create.go

@@ -1,51 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var commentCreateCmd = &cobra.Command{
-	Use:   "create <card_number>",
-	Short: "Create a new comment",
-	Long:  `Create a new comment on a card`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleCreateComment(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleCreateComment(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")
-	}
-
-	body, _ := cmd.Flags().GetString("body")
-
-	_, err = a.Client.PostCardComment(context.Background(), cardNum, body)
-	if err != nil {
-		return fmt.Errorf("creating comment: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Comment created successfully\n")
-	return nil
-}
-
-func init() {
-	commentCreateCmd.Flags().StringP("body", "b", "", "Comment body (required)")
-	commentCreateCmd.MarkFlagRequired("body")
-
-	commentCmd.AddCommand(commentCreateCmd)
-}

package/cmd/comment_create_test.go

@@ -1,129 +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 TestCommentCreateCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards/123/comments" {
-			t.Errorf("expected /cards/123/comments, 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]map[string]string
-		if err := json.Unmarshal(body, &payload); err != nil {
-			t.Fatalf("failed to unmarshal request body: %v", err)
-		}
-
-		commentPayload := payload["comment"]
-		if commentPayload["body"] != "This is a test comment" {
-			t.Errorf("expected body 'This is a test comment', got %s", commentPayload["body"])
-		}
-
-		w.Header().Set("Content-Type", "application/json")
-		w.WriteHeader(http.StatusCreated)
-		response := api.Comment{
-			ID:        "comment-789",
-			CreatedAt: "2025-01-01T00:00:00Z",
-			Creator:   api.User{ID: "user-1", Name: "John Doe"},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := commentCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--body", "This is a test comment",
-	})
-
-	if err := handleCreateComment(cmd, "123"); err != nil {
-		t.Fatalf("handleCreateComment failed: %v", err)
-	}
-}
-
-func TestCommentCreateCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--body", "Test comment",
-	})
-
-	err := handleCreateComment(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 TestCommentCreateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--body", "Test comment",
-	})
-
-	err := handleCreateComment(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 TestCommentCreateCommandAPIError(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 := commentCreateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--body", "Test comment",
-	})
-
-	err := handleCreateComment(cmd, "123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "creating comment: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/comment_delete.go

@@ -1,46 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var commentDeleteCmd = &cobra.Command{
-	Use:   "delete <card_number> <comment_id>",
-	Short: "Delete a comment",
-	Long:  `Delete a comment from a card`,
-	Args:  cobra.ExactArgs(2),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleDeleteComment(cmd, args[0], args[1]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleDeleteComment(cmd *cobra.Command, cardNumber, commentID 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.DeleteCardComment(context.Background(), cardNum, commentID)
-	if err != nil {
-		return fmt.Errorf("deleting comment: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Comment deleted successfully\n")
-	return nil
-}
-
-func init() {
-	commentCmd.AddCommand(commentDeleteCmd)
-}

package/cmd/comment_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 TestCommentDeleteCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/cards/123/comments/comment-456" {
-			t.Errorf("expected /cards/123/comments/comment-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, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := commentDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleDeleteComment(cmd, "123", "comment-456"); err != nil {
-		t.Fatalf("handleDeleteComment failed: %v", err)
-	}
-}
-
-func TestCommentDeleteCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteComment(cmd, "not-a-number", "comment-456")
-	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 TestCommentDeleteCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteComment(cmd, "123", "comment-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)
-	}
-}
-
-func TestCommentDeleteCommandAPIError(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.WriteHeader(http.StatusNotFound)
-		w.Write([]byte("Comment not found"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := commentDeleteCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeleteComment(cmd, "123", "comment-456")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "deleting comment: unexpected status code 404: Comment not found" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/comment_list.go

@@ -1,51 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/ui"
-	"github.com/spf13/cobra"
-)
-
-var commentListCmd = &cobra.Command{
-	Use:   "list <card_number>",
-	Short: "List comments on a card",
-	Long:  `Retrieve and display all comments on a card`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleListComments(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleListComments(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")
-	}
-
-	comments, err := a.Client.GetCardComments(context.Background(), cardNum)
-	if err != nil {
-		return fmt.Errorf("fetching comments: %w", err)
-	}
-
-	if len(comments) == 0 {
-		fmt.Println("No comments found")
-		return nil
-	}
-
-	return ui.DisplayComments(comments)
-}
-
-func init() {
-	commentCmd.AddCommand(commentListCmd)
-}