fix/services list endpoint (#329)

* fix(enterprise): use correct endpoint for services list command

The services list command was using /v1/services which doesn't exist
for GET requests. Changed to use /v1/local/services which is the
correct documented endpoint.

Fixes #322

* test(enterprise): improve mock tests with realistic API responses

- Add comprehensive test fixtures with actual API response data
- Update tests to use realistic fixtures
- Fix User struct: email is primary identifier, not username
- Fix License struct: key field is optional, not required license_key
- Add explicit deserialization tests for type validation

Fixes #326

* fix: resolve CI compilation issues

- Add recursion limit to cluster_tests.rs for large JSON fixtures
- Fix test assertions to match actual ClusterInfo struct fields
- Allow dead code warnings in common test modules
- Remove problematic mod.rs that was causing duplicate module errors
- Fix integer overflow issues in test fixtures (use i64 suffix)

Author: joshrotenberg

crates/redis-enterprise/src/license.rs

@@ -14,7 +14,7 @@ use typed_builder::TypedBuilder;
 /// License information
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct License {
-    /// License key field
+    /// License key - the actual field name returned by API
     #[serde(skip_serializing_if = "Option::is_none")]
     pub key: Option<String>,
 

crates/redis-enterprise/src/users.rs

@@ -19,7 +19,7 @@ use typed_builder::TypedBuilder;
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct User {
     pub uid: u32,
-    /// User's email address (used as login identifier)
+    /// User's email address (used as login identifier) - was incorrectly named 'username'
     pub email: String,
     /// User's display name
     pub name: Option<String>,

crates/redis-enterprise/tests/cluster_tests.rs

@@ -1,6 +1,9 @@
 //! Cluster endpoint tests for Redis Enterprise
+#![recursion_limit = "256"]
 
-use redis_enterprise::{ClusterHandler, EnterpriseClient};
+mod common;
+
+use redis_enterprise::{ClusterHandler, ClusterInfo, EnterpriseClient};
 use serde_json::json;
 use wiremock::matchers::{basic_auth, method, path};
 use wiremock::{Mock, MockServer, ResponseTemplate};
@@ -10,15 +13,6 @@ fn success_response(body: serde_json::Value) -> ResponseTemplate {
     ResponseTemplate::new(200).set_body_json(body)
 }
 
-fn test_cluster() -> serde_json::Value {
-    json!({
-        "name": "test-cluster",
-        "nodes_count": 3,
-        "version": "6.4.2-30",
-        "license": "valid"
-    })
-}
-
 #[tokio::test]
 async fn test_cluster_actions_and_auditing() {
     let mock_server = MockServer::start().await;
@@ -139,7 +133,7 @@ async fn test_cluster_get() {
     Mock::given(method("GET"))
         .and(path("/v1/cluster"))
         .and(basic_auth("admin", "password"))
-        .respond_with(success_response(test_cluster()))
+        .respond_with(success_response(common::fixtures::cluster_info_response()))
         .mount(&mock_server)
         .await;
 
@@ -154,8 +148,37 @@ async fn test_cluster_get() {
     let result = handler.info().await;
 
     assert!(result.is_ok());
-    let _cluster = result.unwrap();
-    // ClusterInfo struct would have these fields available
+    let cluster = result.unwrap();
+    // Verify some key fields that had type mismatches
+    assert_eq!(cluster.name, "test-cluster.local");
+    assert!(cluster.sentinel_cipher_suites.is_some());
+}
+
+#[tokio::test]
+async fn test_cluster_info_deserialization() {
+    // This test explicitly validates that ClusterInfo can deserialize actual API responses
+    let cluster_json = common::fixtures::cluster_info_response();
+
+    // This would panic if deserialization fails with type mismatches
+    let cluster_info: ClusterInfo = serde_json::from_value(cluster_json.clone()).unwrap();
+
+    // Verify fields that previously had type mismatches
+    assert_eq!(cluster_info.name, "test-cluster.local");
+
+    // sentinel_cipher_suites was Option<String> but should be Option<Vec<String>>
+    assert!(cluster_info.sentinel_cipher_suites.is_some());
+    if let Some(cipher_suites) = &cluster_info.sentinel_cipher_suites {
+        assert_eq!(cipher_suites.len(), 0); // Empty array in fixture
+    }
+
+    // password_complexity was Option<Value> but should be Option<bool>
+    assert_eq!(cluster_info.password_complexity, Some(false));
+
+    // mtls_certificate_authentication was Option<String> but should be Option<bool>
+    assert_eq!(cluster_info.mtls_certificate_authentication, Some(false));
+
+    // upgrade_mode was Option<String> but should be Option<bool>
+    assert_eq!(cluster_info.upgrade_mode, Some(false));
 }
 
 #[tokio::test]