feat: add payment-method commands to CLI (#439)

* feat: add payment-method commands to CLI

Add top-level payment-method command for better discoverability:
- redisctl cloud payment-method list

This complements the existing 'account get-payment-methods' command
with a more intuitive command structure that matches other resource
types (database, subscription, etc.).

Changes:
- Added CloudPaymentMethodCommands enum with List subcommand
- Created payment_method.rs handler with table formatting
- Wired up command routing in main.rs and cloud/mod.rs
- Reuses existing AccountHandler.get_account_payment_methods() API

The command displays payment methods in a clean table format showing:
- ID
- Type (credit-card, etc.)
- Last 4 digits
- Expiration date

Supports all standard output formats (table, json, yaml) and
JMESPath query filtering.

Fixes #426

* test: add CLI tests for payment-method command

Add tests to verify:
- payment-method help text displays correctly
- payment-method list subcommand help works

Following best practice of adding CLI tests for all new commands
to catch argument parsing issues and help text regressions.

Author: joshrotenberg

crates/redisctl/src/cli.rs

@@ -1164,6 +1164,10 @@ pub enum CloudCommands {
     #[command(subcommand)]
     Account(CloudAccountCommands),
 
+    /// Payment method operations
+    #[command(subcommand, name = "payment-method")]
+    PaymentMethod(CloudPaymentMethodCommands),
+
     /// Subscription operations
     #[command(subcommand)]
     Subscription(CloudSubscriptionCommands),
@@ -1456,6 +1460,12 @@ pub enum CloudAccountCommands {
     GetSearchScaling,
 }
 
+#[derive(Subcommand, Debug)]
+pub enum CloudPaymentMethodCommands {
+    /// List payment methods configured for the account
+    List,
+}
+
 #[derive(Subcommand, Debug)]
 pub enum CloudSubscriptionCommands {
     /// List all subscriptions

crates/redisctl/src/commands/cloud/mod.rs

@@ -18,6 +18,7 @@ pub mod database;
 pub mod database_impl;
 pub mod fixed_database;
 pub mod fixed_subscription;
+pub mod payment_method;
 pub mod subscription;
 pub mod subscription_impl;
 pub mod task;
@@ -32,6 +33,8 @@ pub use connectivity::handle_connectivity_command;
 #[allow(unused_imports)]
 pub use database::handle_database_command;
 #[allow(unused_imports)]
+pub use payment_method::handle_payment_method_command;
+#[allow(unused_imports)]
 pub use subscription::handle_subscription_command;
 #[allow(unused_imports)]
 pub use user::handle_user_command;

crates/redisctl/src/commands/cloud/payment_method.rs

@@ -0,0 +1,113 @@
+//! Cloud payment method command implementations
+
+#![allow(dead_code)] // Used by binary target
+
+use anyhow::Context;
+use redis_cloud::AccountHandler;
+use serde_json::Value;
+use tabled::{Table, settings::Style};
+
+use crate::cli::{CloudPaymentMethodCommands, OutputFormat};
+use crate::connection::ConnectionManager;
+use crate::error::Result as CliResult;
+
+use super::utils::*;
+
+/// Handle cloud payment method commands
+pub async fn handle_payment_method_command(
+    conn_mgr: &ConnectionManager,
+    profile_name: Option<&str>,
+    command: &CloudPaymentMethodCommands,
+    output_format: OutputFormat,
+    query: Option<&str>,
+) -> CliResult<()> {
+    match command {
+        CloudPaymentMethodCommands::List => {
+            list_payment_methods(conn_mgr, profile_name, output_format, query).await
+        }
+    }
+}
+
+/// List payment methods configured for the account
+async fn list_payment_methods(
+    conn_mgr: &ConnectionManager,
+    profile_name: Option<&str>,
+    output_format: OutputFormat,
+    query: Option<&str>,
+) -> CliResult<()> {
+    let client = conn_mgr.create_cloud_client(profile_name).await?;
+    let handler = AccountHandler::new(client);
+
+    let response = handler
+        .get_account_payment_methods()
+        .await
+        .context("Failed to fetch payment methods")?;
+
+    let json_value = serde_json::to_value(response)?;
+    let data = handle_output(json_value, output_format, query)?;
+
+    match output_format {
+        OutputFormat::Auto | OutputFormat::Table => {
+            print_payment_methods_table(&data)?;
+        }
+        _ => print_formatted_output(data, output_format)?,
+    }
+
+    Ok(())
+}
+
+/// Print payment methods in table format
+fn print_payment_methods_table(data: &Value) -> CliResult<()> {
+    let methods = data.get("paymentMethods").and_then(|p| p.as_array());
+
+    if let Some(methods) = methods {
+        if methods.is_empty() {
+            println!("No payment methods configured");
+            return Ok(());
+        }
+
+        let mut rows = Vec::new();
+        for method in methods {
+            let id = method.get("id").and_then(|v| v.as_u64()).unwrap_or(0);
+            let type_ = method
+                .get("type")
+                .and_then(|v| v.as_str())
+                .unwrap_or("Unknown");
+            let last4 = method
+                .get("last4Digits")
+                .and_then(|v| v.as_str())
+                .unwrap_or("");
+            let exp = method
+                .get("expirationDate")
+                .and_then(|v| v.as_str())
+                .unwrap_or("");
+
+            rows.push(PaymentMethodRow {
+                id: id.to_string(),
+                payment_type: type_.to_string(),
+                last_4: last4.to_string(),
+                expiration: exp.to_string(),
+            });
+        }
+
+        let mut table = Table::new(&rows);
+        table.with(Style::blank());
+        output_with_pager(&table.to_string());
+    } else {
+        println!("No payment methods data available");
+    }
+    Ok(())
+}
+
+// Table row structure for formatting
+#[derive(tabled::Tabled)]
+struct PaymentMethodRow {
+    #[tabled(rename = "ID")]
+    id: String,
+    #[tabled(rename = "Type")]
+    payment_type: String,
+    #[tabled(rename = "Last 4")]
+    last_4: String,
+    #[tabled(rename = "Expiration")]
+    expiration: String,
+}

crates/redisctl/src/main.rs

@@ -858,6 +858,17 @@ async fn execute_cloud_command(
             .await
         }
 
+        PaymentMethod(payment_method_cmd) => {
+            commands::cloud::handle_payment_method_command(
+                conn_mgr,
+                cli.profile.as_deref(),
+                payment_method_cmd,
+                cli.output,
+                cli.query.as_deref(),
+            )
+            .await
+        }
+
         Subscription(sub_cmd) => {
             commands::cloud::handle_subscription_command(
                 conn_mgr,

crates/redisctl/tests/cli_basic_tests.rs

@@ -257,3 +257,27 @@ fn test_profile_remove_missing_name() {
         .failure()
         .stderr(predicate::str::contains("required"));
 }
+
+#[test]
+fn test_payment_method_help() {
+    redisctl()
+        .arg("cloud")
+        .arg("payment-method")
+        .arg("--help")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("Payment method operations"))
+        .stdout(predicate::str::contains("list"));
+}
+
+#[test]
+fn test_payment_method_list_help() {
+    redisctl()
+        .arg("cloud")
+        .arg("payment-method")
+        .arg("list")
+        .arg("--help")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("List payment methods"));
+}