Address search review feedback

Author: robzolkos

internal/commands/help.go

@@ -405,7 +405,7 @@ var commandExamples = map[string]string{
 	"fizzy config show":       "$ fizzy config show\n$ fizzy config show --verbose",
 	"fizzy config explain":    "$ fizzy config explain\n$ fizzy config explain --profile acme",
 	"fizzy doctor":            "$ fizzy doctor\n$ fizzy doctor --profile acme\n$ fizzy doctor --all-profiles",
-	"fizzy search":            "$ fizzy search \"billing bug\"\n$ fizzy card list --search \"billing bug\" --board <id>",
+	"fizzy search":            "$ fizzy search \"billing bug\"\n$ fizzy search <card-id>",
 	"fizzy notification":      "$ fizzy notification tray\n$ fizzy notification list",
 	"fizzy notification tray": "$ fizzy notification tray",
 	"fizzy user":              "$ fizzy user list\n$ fizzy user show <id>",

internal/commands/search.go

@@ -8,7 +8,7 @@ import (
 )
 
 var searchCmd = &cobra.Command{
-	Use:   "search QUERY",
+	Use:   "search QUERY...",
 	Short: "Search cards",
 	Long: `Searches cards using the dedicated full-text search endpoint.
 
@@ -33,7 +33,7 @@ use 'fizzy card list' with --search and the relevant filter flags.`,
 
 		items := normalizeAny(raw)
 		count := dataCount(items)
-		summary := fmt.Sprintf("%d results for \"%s\"", count, query)
+		summary := fmt.Sprintf("%d results for %q", count, query)
 
 		breadcrumbs := []Breadcrumb{
 			breadcrumb("show", "fizzy card show <number>", "View card details"),

internal/commands/search_test.go

@@ -49,6 +49,9 @@ func TestSearch(t *testing.T) {
 		err := searchCmd.RunE(searchCmd, []string{"login", "error"})
 		assertExitCode(t, err, 0)
 
+		if len(mock.GetWithPaginationCalls) != 1 {
+			t.Fatalf("expected 1 GET call, got %d", len(mock.GetWithPaginationCalls))
+		}
 		if got := mock.GetWithPaginationCalls[0].Path; got != "/search.json?q=login+error" {
 			t.Errorf("expected '/search.json?q=login+error', got '%s'", got)
 		}
@@ -68,6 +71,9 @@ func TestSearch(t *testing.T) {
 		err := searchCmd.RunE(searchCmd, []string{"foo&bar=baz"})
 		assertExitCode(t, err, 0)
 
+		if len(mock.GetWithPaginationCalls) != 1 {
+			t.Fatalf("expected 1 GET call, got %d", len(mock.GetWithPaginationCalls))
+		}
 		if got := mock.GetWithPaginationCalls[0].Path; got != "/search.json?q=foo%26bar%3Dbaz" {
 			t.Errorf("expected URL-encoded q, got '%s'", got)
 		}
@@ -88,6 +94,9 @@ func TestSearch(t *testing.T) {
 		err := searchCmd.RunE(searchCmd, []string{"bug"})
 		assertExitCode(t, err, 0)
 
+		if len(mock.GetWithPaginationCalls) != 1 {
+			t.Fatalf("expected 1 GET call, got %d", len(mock.GetWithPaginationCalls))
+		}
 		if got := mock.GetWithPaginationCalls[0].Path; got != "/search.json?q=bug" {
 			t.Errorf("expected no board params in path, got '%s'", got)
 		}