fizzy-cli 0.8.0 → 0.9.0

package/cmd/comment_list_test.go

@@ -1,132 +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 TestCommentListCommandSuccess(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.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.Comment{
-			{
-				ID:        "comment-123",
-				CreatedAt: "2025-01-01T00:00:00Z",
-				UpdatedAt: "2025-01-01T00:00:00Z",
-				Creator:   api.User{ID: "user-1", Name: "John Doe"},
-			},
-			{
-				ID:        "comment-456",
-				CreatedAt: "2025-01-02T00:00:00Z",
-				UpdatedAt: "2025-01-02T00:00:00Z",
-				Creator:   api.User{ID: "user-2", Name: "Jane Doe"},
-			},
-		}
-		// Set the body field manually since it's a struct
-		response[0].Body.PlainText = "First comment"
-		response[0].Body.HTML = "<p>First comment</p>"
-		response[1].Body.PlainText = "Second comment"
-		response[1].Body.HTML = "<p>Second comment</p>"
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := commentListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListComments(cmd, "123"); err != nil {
-		t.Fatalf("handleListComments failed: %v", err)
-	}
-}
-
-func TestCommentListCommandNoComments(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.Comment{})
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := commentListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListComments(cmd, "123"); err != nil {
-		t.Fatalf("handleListComments failed: %v", err)
-	}
-}
-
-func TestCommentListCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListComments(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 TestCommentListCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListComments(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 TestCommentListCommandAPIError(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 := commentListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListComments(cmd, "123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching comments: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/comment_show.go

@@ -1,46 +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 commentShowCmd = &cobra.Command{
-	Use:   "show <card_number> <comment_id>",
-	Short: "Show a specific comment",
-	Long:  `Display details of a specific comment on a card`,
-	Args:  cobra.ExactArgs(2),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleShowComment(cmd, args[0], args[1]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleShowComment(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")
-	}
-
-	comment, err := a.Client.GetCardComment(context.Background(), cardNum, commentID)
-	if err != nil {
-		return fmt.Errorf("fetching comment: %w", err)
-	}
-
-	return ui.DisplayComment(comment)
-}
-
-func init() {
-	commentCmd.AddCommand(commentShowCmd)
-}

package/cmd/comment_show_test.go

@@ -1,104 +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 TestCommentShowCommandSuccess(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.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.Comment{
-			ID:        "comment-456",
-			CreatedAt: "2025-01-01T00:00:00Z",
-			UpdatedAt: "2025-01-01T00:00:00Z",
-			Creator:   api.User{ID: "user-1", Name: "John Doe"},
-			Card:      api.CardReference{ID: "card-123", Title: "Test Card"},
-		}
-		response.Body.PlainText = "This is a test comment"
-		response.Body.HTML = "<p>This is a test comment</p>"
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := commentShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleShowComment(cmd, "123", "comment-456"); err != nil {
-		t.Fatalf("handleShowComment failed: %v", err)
-	}
-}
-
-func TestCommentShowCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowComment(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 TestCommentShowCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowComment(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 TestCommentShowCommandAPIError(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 := commentShowCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleShowComment(cmd, "123", "comment-456")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "fetching comment: unexpected status code 404: Comment not found" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/comment_update.go

@@ -1,51 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"strconv"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var commentUpdateCmd = &cobra.Command{
-	Use:   "update <card_number> <comment_id>",
-	Short: "Update an existing comment",
-	Long:  `Update the body of an existing comment on a card`,
-	Args:  cobra.ExactArgs(2),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleUpdateComment(cmd, args[0], args[1]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleUpdateComment(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")
-	}
-
-	body, _ := cmd.Flags().GetString("body")
-
-	comment, err := a.Client.PutCardComment(context.Background(), cardNum, commentID, body)
-	if err != nil {
-		return fmt.Errorf("updating comment: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Comment updated successfully (id: %s)\n", comment.ID)
-	return nil
-}
-
-func init() {
-	commentUpdateCmd.Flags().StringP("body", "b", "", "New comment body (required)")
-	commentUpdateCmd.MarkFlagRequired("body")
-
-	commentCmd.AddCommand(commentUpdateCmd)
-}

package/cmd/comment_update_test.go

@@ -1,130 +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 TestCommentUpdateCommandSuccess(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.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)
-		}
-
-		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"] != "Updated comment text" {
-			t.Errorf("expected body 'Updated comment text', got %s", commentPayload["body"])
-		}
-
-		w.Header().Set("Content-Type", "application/json")
-		response := api.Comment{
-			ID:        "comment-456",
-			CreatedAt: "2025-01-01T00:00:00Z",
-			UpdatedAt: "2025-01-02T00:00:00Z",
-			Creator:   api.User{ID: "user-1", Name: "John Doe"},
-		}
-		response.Body.PlainText = "Updated comment text"
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := commentUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--body", "Updated comment text",
-	})
-
-	if err := handleUpdateComment(cmd, "123", "comment-456"); err != nil {
-		t.Fatalf("handleUpdateComment failed: %v", err)
-	}
-}
-
-func TestCommentUpdateCommandInvalidCardNumber(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--body", "Updated text",
-	})
-
-	err := handleUpdateComment(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 TestCommentUpdateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := commentUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--body", "Updated text",
-	})
-
-	err := handleUpdateComment(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 TestCommentUpdateCommandAPIError(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 := commentUpdateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{
-		"--body", "Updated text",
-	})
-
-	err := handleUpdateComment(cmd, "123", "comment-456")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "updating comment: unexpected status code 404: Comment not found" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}

package/cmd/login.go

@@ -1,81 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-	"os"
-
-	"github.com/rogeriopvl/fizzy/internal/api"
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/ui"
-	"github.com/spf13/cobra"
-)
-
-var loginCmd = &cobra.Command{
-	Use:   "login",
-	Short: "Prints instructions on how to authenticate with the Fizzy API",
-	Long:  `Prints intructions on how to authenticate with the Fizzy API`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleLogin(cmd); err != nil {
-			fmt.Fprintf(os.Stderr, "Error: %v\n", err)
-		}
-	},
-}
-
-func handleLogin(cmd *cobra.Command) error {
-	token, isSet := os.LookupEnv("FIZZY_ACCESS_TOKEN")
-	if !isSet || token == "" {
-		return printAuthInstructions(cmd)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ Authenticated with access token: %s\n", token[:6]+"...")
-
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	identity, err := a.Client.GetMyIdentity(context.Background())
-	if err != nil {
-		return fmt.Errorf("fetching identity: %w", err)
-	}
-
-	selected, err := chooseAccount(cmd, identity.Accounts)
-	if err != nil {
-		return err
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "\nSelected account: %s (%s)\n", selected.Name, selected.Slug)
-
-	// Save the selected account and current user ID to config
-	a.Config.SelectedAccount = selected.Slug
-	a.Config.CurrentUserID = selected.User.ID
-	if err := a.Config.Save(); err != nil {
-		return fmt.Errorf("saving config: %w", err)
-	}
-
-	return nil
-}
-
-func chooseAccount(cmd *cobra.Command, accounts []api.Account) (api.Account, error) {
-	if len(accounts) == 1 {
-		selected := accounts[0]
-		return selected, nil
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "\nAvailable accounts:\n")
-	return ui.SelectAccount(accounts)
-}
-
-func printAuthInstructions(cmd *cobra.Command) error {
-	fmt.Fprintf(cmd.OutOrStdout(), "To authenticate with Fizzy's API you need an access token.\n")
-	fmt.Fprintf(cmd.OutOrStdout(), "\nGo to https://app.fizzy.do/<account_slug>/my/access_tokens and follow the instructions...\n")
-	fmt.Fprintf(cmd.OutOrStdout(), "(Replace <account_slug> with your account slug)\n")
-	fmt.Fprintf(cmd.OutOrStdout(), "\nThen export it as an environment variable in your shell, with the name FIZZY_ACCESS_TOKEN\n")
-	fmt.Fprintf(cmd.OutOrStdout(), "And re-run this command.\n")
-	return nil
-}
-
-func init() {
-	rootCmd.AddCommand(loginCmd)
-}

package/cmd/login_test.go

@@ -1,98 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"encoding/json"
-	"net/http"
-	"net/http/httptest"
-	"os"
-	"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 TestLoginCommand(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")
-			w.WriteHeader(http.StatusUnauthorized)
-			return
-		}
-		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.GetMyIdentityResponse{
-			Accounts: []api.Account{
-				{
-					ID:   "acc-123",
-					Name: "Test Account",
-					Slug: "test-account",
-					User: api.User{
-						ID:    "user-123",
-						Email: "test@example.com",
-						Name:  "Test User",
-					},
-				},
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	tmpDir := t.TempDir()
-	configPath := tmpDir + "/.config/fizzy-cli/config.json"
-
-	t.Setenv("FIZZY_ACCESS_TOKEN", "test-token")
-	t.Setenv("HOME", tmpDir)
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-
-	cfg := &config.Config{}
-	testApp := &app.App{
-		Client: client,
-		Config: cfg,
-	}
-
-	cmd := loginCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleLogin(cmd); err != nil {
-		t.Fatalf("handleLogin failed: %v", err)
-	}
-
-	if cfg.SelectedAccount != "test-account" {
-		t.Errorf("expected SelectedAccount=test-account, got %s", cfg.SelectedAccount)
-	}
-
-	if _, err := os.Stat(configPath); err != nil {
-		t.Errorf("config file not created at %s: %v", configPath, err)
-	}
-}
-
-func TestLoginCommandWithoutToken(t *testing.T) {
-	t.Setenv("FIZZY_ACCESS_TOKEN", "")
-
-	cmd := loginCmd
-	err := handleLogin(cmd)
-
-	if err != nil {
-		t.Errorf("expected no error when token is missing, got %v", err)
-	}
-}

package/cmd/notification.go

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