fizzy-cli 0.2.1 → 0.4.0

package/cmd/notification_list_test.go

@@ -0,0 +1,288 @@
+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/config"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestNotificationListCommand(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path != "/notifications" {
+			t.Errorf("expected /notifications, 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.Notification{
+			{
+				ID:        "notif-123",
+				Read:      false,
+				ReadAt:    "",
+				CreatedAt: "2025-01-01T00:00:00Z",
+				Title:     "Plain text mentions",
+				Body:      "Assigned to self",
+				Creator: api.User{
+					ID:        "user-123",
+					Name:      "David Heinemeier Hansson",
+					Email:     "david@example.com",
+					Role:      "owner",
+					Active:    true,
+					CreatedAt: "2025-12-05T19:36:35.401Z",
+					URL:       "http://fizzy.localhost:3006/897362094/users/03f5v9zjw7pz8717a4no1h8a7",
+				},
+				Card: api.CardReference{
+					ID:     "card-123",
+					Title:  "Plain text mentions",
+					Status: "published",
+					URL:    "http://fizzy.localhost:3006/897362094/cards/3",
+				},
+				URL: "http://fizzy.localhost:3006/897362094/notifications/03f5va03bpuvkcjemcxl73ho2",
+			},
+			{
+				ID:        "notif-456",
+				Read:      true,
+				ReadAt:    "2025-01-02T00:00:00Z",
+				CreatedAt: "2025-01-01T12:00:00Z",
+				Title:     "Comment reply",
+				Body:      "Someone replied to your comment",
+				Creator: api.User{
+					ID:        "user-456",
+					Name:      "Jason Fried",
+					Email:     "jason@example.com",
+					Role:      "member",
+					Active:    true,
+					CreatedAt: "2025-12-05T19:36:35.419Z",
+					URL:       "http://fizzy.localhost:3006/897362094/users/03f5v9zjysoy0fqs9yg0ei3hq",
+				},
+				Card: api.CardReference{
+					ID:     "card-456",
+					Title:  "Fix bug",
+					Status: "in_progress",
+					URL:    "http://fizzy.localhost:3006/897362094/cards/4",
+				},
+				URL: "http://fizzy.localhost:3006/897362094/notifications/notif-456",
+			},
+		}
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{
+		Client: client,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleListNotifications(cmd); err != nil {
+		t.Fatalf("handleListNotifications failed: %v", err)
+	}
+}
+
+func TestNotificationListCommandNoNotifications(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.Notification{})
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{
+		Client: client,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleListNotifications(cmd); err != nil {
+		t.Fatalf("handleListNotifications failed: %v", err)
+	}
+}
+
+func TestNotificationListCommandAPIError(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,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListNotifications(cmd)
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+	if err.Error() != "fetching notifications: unexpected status code 500: Internal Server Error" {
+		t.Errorf("expected API error, got %v", err)
+	}
+}
+
+func TestNotificationListCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := notificationListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleListNotifications(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)
+	}
+}
+
+func TestNotificationListCommandWithReadFilter(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Type", "application/json")
+		response := []api.Notification{
+			{
+				ID:        "notif-123",
+				Read:      false,
+				CreatedAt: "2025-01-01T00:00:00Z",
+				Title:     "Unread notification",
+				Body:      "This should be filtered out",
+				Creator: api.User{
+					ID:        "user-123",
+					Name:      "David Heinemeier Hansson",
+					Email:     "david@example.com",
+					Role:      "owner",
+					Active:    true,
+					CreatedAt: "2025-12-05T19:36:35.401Z",
+				},
+				Card: api.CardReference{
+					ID:     "card-123",
+					Title:  "Test card",
+					Status: "published",
+				},
+			},
+			{
+				ID:        "notif-456",
+				Read:      true,
+				ReadAt:    "2025-01-02T00:00:00Z",
+				CreatedAt: "2025-01-01T12:00:00Z",
+				Title:     "Read notification",
+				Body:      "This should be shown",
+				Creator: api.User{
+					ID:        "user-456",
+					Name:      "Jason Fried",
+					Email:     "jason@example.com",
+					Role:      "member",
+					Active:    true,
+					CreatedAt: "2025-12-05T19:36:35.419Z",
+				},
+				Card: api.CardReference{
+					ID:     "card-456",
+					Title:  "Another card",
+					Status: "in_progress",
+				},
+			},
+		}
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{
+		Client: client,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.Flags().Set("read", "true")
+
+	if err := handleListNotifications(cmd); err != nil {
+		t.Fatalf("handleListNotifications failed: %v", err)
+	}
+}
+
+func TestNotificationListCommandWithUnreadFilter(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.Header().Set("Content-Type", "application/json")
+		response := []api.Notification{
+			{
+				ID:        "notif-123",
+				Read:      false,
+				CreatedAt: "2025-01-01T00:00:00Z",
+				Title:     "Unread notification",
+				Body:      "This should be shown",
+				Creator: api.User{
+					ID:        "user-123",
+					Name:      "David Heinemeier Hansson",
+					Email:     "david@example.com",
+					Role:      "owner",
+					Active:    true,
+					CreatedAt: "2025-12-05T19:36:35.401Z",
+				},
+				Card: api.CardReference{
+					ID:     "card-123",
+					Title:  "Test card",
+					Status: "published",
+				},
+			},
+			{
+				ID:        "notif-456",
+				Read:      true,
+				ReadAt:    "2025-01-02T00:00:00Z",
+				CreatedAt: "2025-01-01T12:00:00Z",
+				Title:     "Read notification",
+				Body:      "This should be filtered out",
+				Creator: api.User{
+					ID:        "user-456",
+					Name:      "Jason Fried",
+					Email:     "jason@example.com",
+					Role:      "member",
+					Active:    true,
+					CreatedAt: "2025-12-05T19:36:35.419Z",
+				},
+				Card: api.CardReference{
+					ID:     "card-456",
+					Title:  "Another card",
+					Status: "in_progress",
+				},
+			},
+		}
+		json.NewEncoder(w).Encode(response)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{
+		Client: client,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationListCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+	cmd.Flags().Set("unread", "true")
+
+	if err := handleListNotifications(cmd); err != nil {
+		t.Fatalf("handleListNotifications failed: %v", err)
+	}
+}

package/cmd/notification_read.go

@@ -0,0 +1,51 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strings"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/ui"
+	"github.com/spf13/cobra"
+)
+
+var notificationReadCmd = &cobra.Command{
+	Use:   "read <notification_id>",
+	Short: "Mark notification as read and display it",
+	Long:  `Mark a notification as read and display its content`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleReadNotification(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleReadNotification(cmd *cobra.Command, notificationID string) error {
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	if _, err := a.Client.PostNotificationReading(context.Background(), notificationID); err != nil {
+		if strings.Contains(err.Error(), "404") {
+			return fmt.Errorf("notification not found")
+		}
+		return fmt.Errorf("marking notification as read: %w", err)
+	}
+
+	notification, err := a.Client.GetNotification(context.Background(), notificationID)
+	if err != nil {
+		if strings.Contains(err.Error(), "404") {
+			return fmt.Errorf("notification not found")
+		}
+		return fmt.Errorf("fetching notification: %w", err)
+	}
+
+	return ui.DisplayNotification(notification)
+}
+
+func init() {
+	notificationCmd.AddCommand(notificationReadCmd)
+}

package/cmd/notification_read_all.go

@@ -0,0 +1,38 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var notificationReadAllCmd = &cobra.Command{
+	Use:   "read-all",
+	Short: "Mark all unread notifications as read",
+	Long:  `Mark all unread notifications as read`,
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleReadAllNotifications(cmd); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleReadAllNotifications(cmd *cobra.Command) error {
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	if _, err := a.Client.PostBulkNotificationsReading(context.Background()); err != nil {
+		return fmt.Errorf("marking all notifications as read: %w", err)
+	}
+
+	fmt.Println("✓ All notifications marked as read successfully")
+	return nil
+}
+
+func init() {
+	notificationCmd.AddCommand(notificationReadAllCmd)
+}

package/cmd/notification_read_all_test.go

@@ -0,0 +1,75 @@
+package cmd
+
+import (
+	"context"
+	"net/http"
+	"net/http/httptest"
+	"testing"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/rogeriopvl/fizzy/internal/config"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestNotificationReadAllCommand(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path == "/notifications/bulk_reading" && r.Method == http.MethodPost {
+			w.WriteHeader(http.StatusNoContent)
+			return
+		}
+
+		t.Errorf("unexpected request: %s %s", r.Method, r.URL.Path)
+		w.WriteHeader(http.StatusNotFound)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{
+		Client: client,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationReadAllCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleReadAllNotifications(cmd); err != nil {
+		t.Fatalf("handleReadAllNotifications failed: %v", err)
+	}
+}
+
+func TestNotificationReadAllCommandAPIError(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,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationReadAllCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleReadAllNotifications(cmd)
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+}
+
+func TestNotificationReadAllCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := notificationReadAllCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleReadAllNotifications(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/notification_read_test.go

@@ -0,0 +1,138 @@
+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/config"
+	"github.com/rogeriopvl/fizzy/internal/testutil"
+)
+
+func TestNotificationReadCommand(t *testing.T) {
+	requestCount := 0
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		requestCount++
+
+		if r.URL.Path == "/notifications/notif-123/reading" && r.Method == http.MethodPost {
+			w.WriteHeader(http.StatusNoContent)
+			return
+		}
+
+		if r.URL.Path == "/notifications/notif-123" && r.Method == http.MethodGet {
+			w.Header().Set("Content-Type", "application/json")
+			response := api.Notification{
+				ID:        "notif-123",
+				Read:      true,
+				ReadAt:    "2025-01-01T00:00:00Z",
+				CreatedAt: "2025-01-01T00:00:00Z",
+				Title:     "Test Notification",
+				Body:      "This is a test notification",
+				Creator: api.User{
+					ID:        "user-123",
+					Name:      "David Heinemeier Hansson",
+					Email:     "david@example.com",
+					Role:      "owner",
+					Active:    true,
+					CreatedAt: "2025-12-05T19:36:35.401Z",
+				},
+				Card: api.CardReference{
+					ID:     "card-123",
+					Title:  "Test Card",
+					Status: "published",
+					URL:    "http://fizzy.localhost:3006/897362094/cards/1",
+				},
+				URL: "http://fizzy.localhost:3006/897362094/notifications/notif-123",
+			}
+			json.NewEncoder(w).Encode(response)
+			return
+		}
+
+		t.Errorf("unexpected request: %s %s", r.Method, r.URL.Path)
+		w.WriteHeader(http.StatusNotFound)
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{
+		Client: client,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationReadCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	if err := handleReadNotification(cmd, "notif-123"); err != nil {
+		t.Fatalf("handleReadNotification failed: %v", err)
+	}
+
+	if requestCount != 2 {
+		t.Errorf("expected 2 requests (POST and GET), got %d", requestCount)
+	}
+}
+
+func TestNotificationReadCommandNotFound(t *testing.T) {
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		w.WriteHeader(http.StatusNotFound)
+		w.Write([]byte("Notification not found"))
+	}))
+	defer server.Close()
+
+	client := testutil.NewTestClient(server.URL, "", "board-123", "test-token")
+	testApp := &app.App{
+		Client: client,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationReadCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleReadNotification(cmd, "notif-invalid")
+	if err == nil {
+		t.Errorf("expected error for invalid notification")
+	}
+	if err.Error() != "notification not found" {
+		t.Errorf("expected 'notification not found' error, got %v", err)
+	}
+}
+
+func TestNotificationReadCommandAPIError(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,
+		Config: &config.Config{SelectedBoard: "board-123"},
+	}
+
+	cmd := notificationReadCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleReadNotification(cmd, "notif-123")
+	if err == nil {
+		t.Errorf("expected error for API failure")
+	}
+}
+
+func TestNotificationReadCommandNoClient(t *testing.T) {
+	testApp := &app.App{}
+
+	cmd := notificationReadCmd
+	cmd.SetContext(testApp.ToContext(context.Background()))
+
+	err := handleReadNotification(cmd, "notif-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/notification_unread.go

@@ -0,0 +1,44 @@
+package cmd
+
+import (
+	"context"
+	"fmt"
+	"strings"
+
+	"github.com/rogeriopvl/fizzy/internal/app"
+	"github.com/spf13/cobra"
+)
+
+var notificationUnreadCmd = &cobra.Command{
+	Use:   "unread <notification_id>",
+	Short: "Mark notification as unread",
+	Long:  `Mark a notification as unread`,
+	Args:  cobra.ExactArgs(1),
+	Run: func(cmd *cobra.Command, args []string) {
+		if err := handleUnreadNotification(cmd, args[0]); err != nil {
+			fmt.Fprintf(cmd.OutOrStderr(), "Error: %v\n", err)
+		}
+	},
+}
+
+func handleUnreadNotification(cmd *cobra.Command, notificationID string) error {
+	a := app.FromContext(cmd.Context())
+	if a == nil || a.Client == nil {
+		return fmt.Errorf("API client not available")
+	}
+
+	_, err := a.Client.DeleteNotificationReading(context.Background(), notificationID)
+	if err != nil {
+		if strings.Contains(err.Error(), "404") {
+			return fmt.Errorf("notification not found")
+		}
+		return fmt.Errorf("marking notification as unread: %w", err)
+	}
+
+	fmt.Printf("✓ Notification marked as unread successfully\n")
+	return nil
+}
+
+func init() {
+	notificationCmd.AddCommand(notificationUnreadCmd)
+}