fizzy-cli 0.8.0 → 0.9.0

package/cmd/tag_list_test.go

@@ -1,109 +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 TestTagListCommand(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/tags" {
-			t.Errorf("expected /tags, 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.Tag{
-			{
-				ID:        "tag-123",
-				Title:     "bug",
-				CreatedAt: "2025-01-01T00:00:00Z",
-				URL:       "http://fizzy.localhost:3006/897362094/cards?tag_ids[]=tag-123",
-			},
-			{
-				ID:        "tag-456",
-				Title:     "feature",
-				CreatedAt: "2025-01-02T00:00:00Z",
-				URL:       "http://fizzy.localhost:3006/897362094/cards?tag_ids[]=tag-456",
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := tagListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListTags(cmd); err != nil {
-		t.Fatalf("handleListTags failed: %v", err)
-	}
-}
-
-func TestTagListCommandNoTags(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.Header().Set("Content-Type", "application/json")
-		w.Write([]byte("[]"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := tagListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleListTags(cmd); err != nil {
-		t.Fatalf("handleListTags with no tags failed: %v", err)
-	}
-}
-
-func TestTagListCommandAPIError(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 := tagListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListTags(cmd)
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-}
-
-func TestTagListCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := tagListCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleListTags(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/use.go

@@ -1,85 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/rogeriopvl/fizzy/internal/config"
-	"github.com/spf13/cobra"
-)
-
-var useCmd = &cobra.Command{
-	Use:   "use",
-	Short: "Set the active board or account",
-	Long:  `Set the active board or account to use for subsequent commands`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleUse(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleUse(cmd *cobra.Command) error {
-	board, _ := cmd.Flags().GetString("board")
-	account, _ := cmd.Flags().GetString("account")
-
-	if board == "" && account == "" {
-		return fmt.Errorf("must specify either --board or --account")
-	}
-
-	if board != "" && account != "" {
-		return fmt.Errorf("cannot specify both --board and --account")
-	}
-
-	cfg, err := config.Load()
-	if err != nil {
-		return fmt.Errorf("loading config: %w", err)
-	}
-
-	if board != "" {
-		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)
-		}
-
-		var boardID string
-		for _, b := range boards {
-			if b.Name == board {
-				boardID = b.ID
-				break
-			}
-		}
-
-		if boardID == "" {
-			return fmt.Errorf("board '%s' not found", board)
-		}
-
-		cfg.SelectedBoard = boardID
-		if err := cfg.Save(); err != nil {
-			return fmt.Errorf("saving config: %w", err)
-		}
-		fmt.Printf("Selected board: %s\n", board)
-	}
-
-	if account != "" {
-		cfg.SelectedAccount = account
-		if err := cfg.Save(); err != nil {
-			return fmt.Errorf("saving config: %w", err)
-		}
-		fmt.Printf("Selected account: %s\n", account)
-	}
-
-	return nil
-}
-
-func init() {
-	rootCmd.AddCommand(useCmd)
-	useCmd.Flags().String("board", "", "Board name to use")
-	useCmd.Flags().String("account", "", "Account slug to use")
-}

package/cmd/use_test.go

@@ -1,186 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"encoding/json"
-	"fmt"
-	"net/http"
-	"net/http/httptest"
-	"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"
-	"github.com/spf13/cobra"
-)
-
-func newUseCmd() *cobra.Command {
-	cmd := &cobra.Command{
-		Use:   "use",
-		Short: "Set the active board or account",
-		Long:  `Set the active board or account to use for subsequent commands`,
-		Run: func(cmd *cobra.Command, args []string) {
-			if err := handleUse(cmd); err != nil {
-				fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-			}
-		},
-	}
-	cmd.Flags().String("board", "", "Board name to use")
-	cmd.Flags().String("account", "", "Account slug to use")
-	return cmd
-}
-
-func TestUseCommandSetBoard(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: "My Project",
-			},
-			{
-				ID:   "board-456",
-				Name: "Other Board",
-			},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	tmpDir := t.TempDir()
-	t.Setenv("HOME", tmpDir)
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	cfg := &config.Config{}
-	testApp := &app.App{
-		Client: client,
-		Config: cfg,
-	}
-
-	cmd := newUseCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--board", "My Project"})
-
-	if err := handleUse(cmd); err != nil {
-		t.Fatalf("handleUse failed: %v", err)
-	}
-
-	savedCfg, err := config.Load()
-	if err != nil {
-		t.Fatalf("failed to load config: %v", err)
-	}
-
-	if savedCfg.SelectedBoard != "board-123" {
-		t.Errorf("expected SelectedBoard=board-123, got %s", savedCfg.SelectedBoard)
-	}
-}
-
-func TestUseCommandSetAccount(t *testing.T) {
-	tmpDir := t.TempDir()
-	t.Setenv("HOME", tmpDir)
-
-	cmd := newUseCmd()
-	cmd.ParseFlags([]string{"--account", "my-company"})
-
-	cfg := &config.Config{}
-	testApp := &app.App{Config: cfg}
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleUse(cmd); err != nil {
-		t.Fatalf("handleUse failed: %v", err)
-	}
-
-	savedCfg, err := config.Load()
-	if err != nil {
-		t.Fatalf("failed to load config: %v", err)
-	}
-
-	if savedCfg.SelectedAccount != "my-company" {
-		t.Errorf("expected SelectedAccount=my-company, got %s", savedCfg.SelectedAccount)
-	}
-}
-
-func TestUseCommandBoardNotFound(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		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: "Existing Board"},
-		}
-		json.NewEncoder(w).Encode(response)
-	}))
-	defer server.Close()
-
-	tmpDir := t.TempDir()
-	t.Setenv("HOME", tmpDir)
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{
-		Client: client,
-		Config: &config.Config{},
-	}
-
-	cmd := newUseCmd()
-	cmd.SetContext(testApp.ToContext(context.Background()))
-	cmd.ParseFlags([]string{"--board", "Nonexistent Board"})
-
-	err := handleUse(cmd)
-	if err == nil {
-		t.Errorf("expected error for nonexistent board")
-	}
-	if err.Error() != "board 'Nonexistent Board' not found" {
-		t.Errorf("expected 'not found' error, got %v", err)
-	}
-}
-
-func TestUseCommandBothFlagsError(t *testing.T) {
-	cmd := newUseCmd()
-	cmd.ParseFlags([]string{"--board", "Board", "--account", "Account"})
-	cmd.SetContext(context.Background())
-
-	err := handleUse(cmd)
-	if err == nil {
-		t.Errorf("expected error when both flags provided")
-	}
-	if err.Error() != "cannot specify both --board and --account" {
-		t.Errorf("expected 'both flags' error, got %v", err)
-	}
-}
-
-func TestUseCommandNoFlagsError(t *testing.T) {
-	cmd := newUseCmd()
-	cmd.SetContext(context.Background())
-	cmd.ParseFlags([]string{})
-
-	err := handleUse(cmd)
-	if err == nil {
-		t.Errorf("expected error when no flags provided")
-	}
-	if err.Error() != "must specify either --board or --account" {
-		t.Errorf("expected 'no flags' error, got %v", err)
-	}
-}

package/cmd/user.go

@@ -1,22 +0,0 @@
-// Package cmd
-package cmd
-
-import (
-	"github.com/spf13/cobra"
-)
-
-var userCmd = &cobra.Command{
-	Use:   "user",
-	Short: "Manage users",
-	Long: `Manage users in your account.
-
-Use subcommands to list, view, or manage users:
-  fizzy user list         List all users
-  fizzy user show <id>    Show user details
-  fizzy user update <id>  Update user settings
-  fizzy user deactivate <id>  Deactivate a user`,
-}
-
-func init() {
-	rootCmd.AddCommand(userCmd)
-}

package/cmd/user_deactivate.go

@@ -1,40 +0,0 @@
-package cmd
-
-import (
-	"context"
-	"fmt"
-
-	"github.com/rogeriopvl/fizzy/internal/app"
-	"github.com/spf13/cobra"
-)
-
-var userDeactivateCmd = &cobra.Command{
-	Use:   "deactivate <user_id>",
-	Short: "Deactivate a user",
-	Long:  `Deactivate a user. Only account administrators can deactivate users.`,
-	Args:  cobra.ExactArgs(1),
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleDeactivateUser(cmd, args[0]); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleDeactivateUser(cmd *cobra.Command, userID string) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	err := a.Client.DeleteUser(context.Background(), userID)
-	if err != nil {
-		return fmt.Errorf("deactivating user: %w", err)
-	}
-
-	fmt.Fprintf(cmd.OutOrStdout(), "✓ User '%s' deactivated successfully\n", userID)
-	return nil
-}
-
-func init() {
-	userCmd.AddCommand(userDeactivateCmd)
-}

package/cmd/user_deactivate_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 TestUserDeactivateCommandSuccess(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		if r.URL.Path != "/users/user-123" {
-			t.Errorf("expected /users/user-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 := userDeactivateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	if err := handleDeactivateUser(cmd, "user-123"); err != nil {
-		t.Fatalf("handleDeactivateUser failed: %v", err)
-	}
-}
-
-func TestUserDeactivateCommandNotFound(t *testing.T) {
-	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		w.WriteHeader(http.StatusNotFound)
-		w.Write([]byte("User not found"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := userDeactivateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeactivateUser(cmd, "nonexistent-user")
-	if err == nil {
-		t.Errorf("expected error for user not found")
-	}
-	if err.Error() != "deactivating user: unexpected status code 404: User not found" {
-		t.Errorf("expected user not found error, got %v", err)
-	}
-}
-
-func TestUserDeactivateCommandForbidden(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 deactivate this user"))
-	}))
-	defer server.Close()
-
-	client := testutil.NewTestClient(server.URL, "", "", "test-token")
-	testApp := &app.App{Client: client}
-
-	cmd := userDeactivateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeactivateUser(cmd, "user-123")
-	if err == nil {
-		t.Errorf("expected error for forbidden access")
-	}
-	if err.Error() != "deactivating user: unexpected status code 403: You don't have permission to deactivate this user" {
-		t.Errorf("expected permission error, got %v", err)
-	}
-}
-
-func TestUserDeactivateCommandAPIError(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 := userDeactivateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeactivateUser(cmd, "user-123")
-	if err == nil {
-		t.Errorf("expected error for API failure")
-	}
-	if err.Error() != "deactivating user: unexpected status code 500: Internal Server Error" {
-		t.Errorf("expected API error, got %v", err)
-	}
-}
-
-func TestUserDeactivateCommandNoClient(t *testing.T) {
-	testApp := &app.App{}
-
-	cmd := userDeactivateCmd
-	cmd.SetContext(testApp.ToContext(context.Background()))
-
-	err := handleDeactivateUser(cmd, "user-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/user_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 userListCmd = &cobra.Command{
-	Use:   "list",
-	Short: "List all users",
-	Long:  `Retrieve and display all users from the current account`,
-	Run: func(cmd *cobra.Command, args []string) {
-		if err := handleListUsers(cmd); err != nil {
-			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
-		}
-	},
-}
-
-func handleListUsers(cmd *cobra.Command) error {
-	a := app.FromContext(cmd.Context())
-	if a == nil || a.Client == nil {
-		return fmt.Errorf("API client not available")
-	}
-
-	users, err := a.Client.GetUsers(context.Background())
-	if err != nil {
-		return fmt.Errorf("fetching users: %w", err)
-	}
-
-	if len(users) == 0 {
-		fmt.Println("No users found")
-		return nil
-	}
-
-	return ui.DisplayUsers(cmd.OutOrStdout(), users)
-}
-
-func init() {
-	userCmd.AddCommand(userListCmd)
-}