test: add comprehensive CLI tests with assert_cmd (#435)

* test: add comprehensive CLI tests using assert_cmd

Add 39 new integration tests for CLI functionality:

Profile Commands (15 tests):
- Profile CRUD operations (set, show, remove, list)
- Cloud and Enterprise profile creation
- Default profile management
- Profile validation
- JSON output format
- Custom config file support
- Profile updates

Basic CLI Functionality (24 tests):
- Help flags (--help, -h)
- Version flags (--version, -V)
- Output formats (json, yaml, table)
- Invalid arguments handling
- Global flags (--verbose, --config-file, --profile, --query)
- Subcommand help (profile, cloud, enterprise, api)
- Error handling for missing required arguments

All tests use isolated temp directories via --config-file flag to avoid
interfering with system configuration. Tests validate argument parsing,
help text, error messages, and basic command functionality without
making actual API calls.

Total test count for redisctl: 72 tests (31 unit + 41 integration)

* test: expand mocked CLI integration tests to 17 tests

Add comprehensive mocked API tests covering:
- Cloud subscription operations (get, list, create)
- Enterprise database and cluster operations
- HTTP methods (GET, POST, PUT, DELETE)
- Error responses (401, 404, 500)
- Output formats (JSON, YAML)
- JMESPath query filtering
- Async operation task IDs
- Multiple requests to same mock server
- Verbose logging flag

All tests use wiremock to verify request construction without
hitting real APIs. Tests validate auth headers, request bodies,
and response handling.

* test: add 10 more mocked CLI integration tests (27 total)

Add comprehensive coverage for advanced Redis operations:
- Enterprise: nodes, backups, CRDBs, Redis ACLs, license info
- Cloud: VPC peering, account info, database modules, payment methods, maintenance windows

Tests cover critical functionality across both deployment types:
- Infrastructure operations (nodes, backups, peering)
- Access control (ACLs)
- Multi-region databases (CRDBs)
- Account management (payment, license)
- Database features (modules)
- Operational windows (maintenance)

All tests use wiremock to validate request/response handling
without hitting real APIs. Total mocked integration tests: 27.

* fix: format code and add missing dev dependencies

- Fix formatting in cli_basic_tests.rs
- Add wiremock, tokio, serde_json to dev-dependencies
- Update Cargo.lock

Author: joshrotenberg

Cargo.lock

@@ -76,6 +76,9 @@ predicates = "3.0"
 tempfile = "3.8"
 criterion = "0.5"
 serial_test = "3.1"
+wiremock = { workspace = true }
+tokio = { workspace = true }
+serde_json = { workspace = true }
 
 [[bench]]
 name = "api_performance"

crates/redisctl/Cargo.toml

@@ -0,0 +1,259 @@
+use assert_cmd::Command;
+use predicates::prelude::*;
+
+/// Helper to create a test command
+fn redisctl() -> Command {
+    Command::cargo_bin("redisctl").unwrap()
+}
+
+#[test]
+fn test_help_flag() {
+    redisctl()
+        .arg("--help")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("Redis management CLI"))
+        .stdout(predicate::str::contains("EXAMPLES:"));
+}
+
+#[test]
+fn test_help_short_flag() {
+    redisctl()
+        .arg("-h")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("Usage:"));
+}
+
+#[test]
+fn test_version_flag() {
+    redisctl()
+        .arg("--version")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("redisctl"))
+        .stdout(predicate::str::contains(env!("CARGO_PKG_VERSION")));
+}
+
+#[test]
+fn test_version_short_flag() {
+    redisctl()
+        .arg("-V")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("redisctl"));
+}
+
+#[test]
+fn test_no_args_shows_help() {
+    redisctl()
+        .assert()
+        .failure()
+        .code(2)
+        .stderr(predicate::str::contains("Usage:"));
+}
+
+#[test]
+fn test_invalid_subcommand() {
+    redisctl()
+        .arg("invalid-command")
+        .assert()
+        .failure()
+        .stderr(predicate::str::contains("unrecognized subcommand"));
+}
+
+#[test]
+fn test_profile_help() {
+    redisctl()
+        .arg("profile")
+        .arg("--help")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("Profile management"));
+}
+
+#[test]
+fn test_cloud_help() {
+    redisctl()
+        .arg("cloud")
+        .arg("--help")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("Cloud-specific"));
+}
+
+#[test]
+fn test_enterprise_help() {
+    redisctl()
+        .arg("enterprise")
+        .arg("--help")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("Enterprise-specific"));
+}
+
+#[test]
+fn test_api_help() {
+    redisctl()
+        .arg("api")
+        .arg("--help")
+        .assert()
+        .success()
+        .stdout(predicate::str::contains("Raw API access"));
+}
+
+#[test]
+fn test_output_format_json() {
+    // Test that -o json flag is accepted (doesn't test actual output)
+    redisctl()
+        .arg("profile")
+        .arg("list")
+        .arg("-o")
+        .arg("json")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_output_format_yaml() {
+    redisctl()
+        .arg("profile")
+        .arg("list")
+        .arg("-o")
+        .arg("yaml")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_output_format_table() {
+    redisctl()
+        .arg("profile")
+        .arg("list")
+        .arg("-o")
+        .arg("table")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_invalid_output_format() {
+    redisctl()
+        .arg("profile")
+        .arg("list")
+        .arg("-o")
+        .arg("invalid")
+        .assert()
+        .failure()
+        .stderr(predicate::str::contains("invalid value"));
+}
+
+#[test]
+fn test_verbose_flag() {
+    redisctl()
+        .arg("-v")
+        .arg("profile")
+        .arg("list")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_multiple_verbose_flags() {
+    redisctl()
+        .arg("-vvv")
+        .arg("profile")
+        .arg("list")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_config_file_flag() {
+    redisctl()
+        .arg("--config-file")
+        .arg("/tmp/test-config.toml")
+        .arg("profile")
+        .arg("list")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_profile_flag() {
+    // Just test that the flag is accepted, actual profile doesn't need to exist for this test
+    redisctl()
+        .arg("--profile")
+        .arg("nonexistent")
+        .arg("profile")
+        .arg("list")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_query_flag() {
+    redisctl()
+        .arg("profile")
+        .arg("list")
+        .arg("--query")
+        .arg("profiles")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_global_flags_before_subcommand() {
+    redisctl()
+        .arg("-v")
+        .arg("-o")
+        .arg("json")
+        .arg("profile")
+        .arg("list")
+        .assert()
+        .success();
+}
+
+#[test]
+fn test_profile_set_missing_required_args() {
+    redisctl()
+        .arg("profile")
+        .arg("set")
+        .arg("test-profile")
+        .assert()
+        .failure()
+        .stderr(predicate::str::contains("required"));
+}
+
+#[test]
+fn test_profile_set_missing_deployment_type() {
+    redisctl()
+        .arg("profile")
+        .arg("set")
+        .arg("test-profile")
+        .arg("--api-key")
+        .arg("key")
+        .assert()
+        .failure()
+        .stderr(predicate::str::contains("--deployment"));
+}
+
+#[test]
+fn test_profile_show_missing_name() {
+    redisctl()
+        .arg("profile")
+        .arg("show")
+        .assert()
+        .failure()
+        .stderr(predicate::str::contains("required"));
+}
+
+#[test]
+fn test_profile_remove_missing_name() {
+    redisctl()
+        .arg("profile")
+        .arg("remove")
+        .assert()
+        .failure()
+        .stderr(predicate::str::contains("required"));
+}