update cert and minor test fixes (#67)

this fixes tests for the following issues:

- Updates the certs
- disable tls for integration tests because the mock uses a self-signed
cert
- fix liteon integration test (vendor was missing)

Author: Sinck

src/dell.rs

@@ -1037,7 +1037,7 @@ impl Redfish for Bmc {
                 }
             }
 
-            return Err(RedfishError::MissingBootOption(expected_boot_option_name));
+            Err(RedfishError::MissingBootOption(expected_boot_option_name))
         })
     }
 

src/jsonmap.rs

@@ -246,8 +246,8 @@ mod tests {
         });
         let map = value.as_object().unwrap();
 
-        assert_eq!(get_bool(map, "Enabled", "http://test/url").unwrap(), true);
-        assert_eq!(get_bool(map, "Disabled", "http://test/url").unwrap(), false);
+        assert!(get_bool(map, "Enabled", "http://test/url").unwrap());
+        assert!(!get_bool(map, "Disabled", "http://test/url").unwrap());
     }
 
     // test_get_i64_success tests that get_i64 correctly extracts an integer value.

src/lib.rs

@@ -222,10 +222,12 @@ pub trait Redfish: Send + Sync + 'static {
     /// - boot_interface_mac: MAC Address of the NIC you wish to boot from
     ///   If not given we look for a Mellanox Bluefield DPU and use that.
     ///   Not applicable to Supermicro and the DPU itself.
-    /// bios_profiles: Map of vendor/model (with spaces replaced by underscores)/profile/type
+    ///   bios_profiles: Map of vendor/model (with spaces replaced by underscores)/profile/type
     ///   to extra settings; expected to come from config rather than hardcoded.
-    /// selected_profile: Profile to use (if present)
+    ///   selected_profile: Profile to use (if present)
+    ///
     /// Returns Ok(Some(job_id)) when the vendor creates a job for the BIOS PATCH (e.g. Dell);
+    ///
     /// Ok(None) when no job is created. Caller should wait for job completion before configuring boot order.
     fn machine_setup<'a>(
         &'a self,

src/network.rs

@@ -781,7 +781,7 @@ fn truncate(s: &str, len: usize) -> &str {
 ///   `Password`, `OldPassword`, `NewPassword`   — standard Redfish account/BIOS ops
 ///   `CurrentUefiPassword`, `UefiPassword`       — NVIDIA DPU Bios/Settings PATCH
 ///   `ImportBuffer`                              — Dell ImportSystemConfiguration XML blob
-
+///
 /// A `Display` wrapper that redacts sensitive JSON fields on formatting.
 ///
 /// Passing this to `tracing::debug!` defers evaluation until the macro decides the

tests/cert.pem

@@ -1,22 +1,20 @@
 -----BEGIN CERTIFICATE-----
-MIIDkzCCAnugAwIBAgIUXVYkRCrM/ge03DVymDtXCuybp7gwDQYJKoZIhvcNAQEL
-BQAwWTELMAkGA1UEBhMCVVMxEzARBgNVBAgMClNvbWUtU3RhdGUxITAfBgNVBAoM
-GEludGVybmV0IFdpZGdpdHMgUHR5IEx0ZDESMBAGA1UEAwwJbG9jYWxob3N0MB4X
-DTIxMDczMTE0MjIxMloXDTIyMDczMTE0MjIxMlowWTELMAkGA1UEBhMCVVMxEzAR
-BgNVBAgMClNvbWUtU3RhdGUxITAfBgNVBAoMGEludGVybmV0IFdpZGdpdHMgUHR5
-IEx0ZDESMBAGA1UEAwwJbG9jYWxob3N0MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8A
-MIIBCgKCAQEA02V5ZjmqLB/VQwTarrz/35qsa83L+DbAoa0001+jVmmC+G9Nufi0
-daroFWj/Uicv2fZWETU8JoZKUrX4BK9og5cg5rln/CtBRWCUYIwRgY9R/CdBGPn4
-kp+XkSJaCw74ZIyLy/Zfux6h8ES1m9YRnBza+s7U+ImRBRf4MRPtXQ3/mqJxAZYq
-dOnKnvssRyD2qutgVTAxwMUvJWIivRhRYDj7WOpS4CEEeQxP1iH1/T5P7FdtTGdT
-bVBABCA8JhL96uFGPpOYHcM/7R5EIA3yZ5FNg931QzoDITjtXGtQ6y9/l/IYkWm6
-J67RWcN0IoTsZhz0WNU4gAeslVtJLofn8QIDAQABo1MwUTAdBgNVHQ4EFgQUzFnK
-NfS4LAYuKeWwHbzooER0yZ0wHwYDVR0jBBgwFoAUzFnKNfS4LAYuKeWwHbzooER0
-yZ0wDwYDVR0TAQH/BAUwAwEB/zANBgkqhkiG9w0BAQsFAAOCAQEAk4O+e9jia59W
-ZwetN4GU7OWcYhmOgSizRSs6u7mTfp62LDMt96WKU3THksOnZ44HnqWQxsSfdFVU
-XJD12tjvVU8Z4FWzQajcHeemUYiDze8EAh6TnxnUcOrU8IcwiKGxCWRY/908jnWg
-+MMscfMCMYTRdeTPqD8fGzAlUCtmyzH6KLE3s4Oo/r5+NR+Uvrwpdvb7xe0MwwO9
-Q/zR4N8ep/HwHVEObcaBofE1ssZLksX7ZgCP9wMgXRWpNAtC5EWxMbxYjBfWFH24
-fDJlBMiGJWg8HHcxK7wQhFh+fuyNzE+xEWPsI9VL1zDftd9x8/QsOagyEOnY8Vxr
-AopvZ09uEQ==
+MIIDSzCCAjOgAwIBAgIUJctSuhvufWDDxfQl4MKiCdjf+3EwDQYJKoZIhvcNAQEL
+BQAwFDESMBAGA1UEAwwJbG9jYWxob3N0MCAXDTI2MDUxMTE2MzU0N1oYDzIzMDAw
+MjI0MTYzNTQ3WjAUMRIwEAYDVQQDDAlsb2NhbGhvc3QwggEiMA0GCSqGSIb3DQEB
+AQUAA4IBDwAwggEKAoIBAQCpqpzhK3Gyn9l46pm4fFPk23kX6taXnQw8w6UWtFM3
+lqmsanhRSr+7RE2FDNcReXL01ftk7JZpz0yrsWXiHXrJRKJBry2btLjxwEcVE8Uy
+wbIDoLFbUwyIpW7YCv0Z+xkFNJXXZJFz1Ad74yMrGd3ob2KWcVvDXKnRRDFqDrC3
+anAFlHuxfIA+AOAIOGps6B/o4bBkwOZp7aWWaETd0XdkH5/R9x3osIZDkUaKk8GE
+XbzdipDWndBHTDr9myaWQvCutQVzScl5l+U+pX7pOpK2TzC2neeCZXmanXI+Vngt
+vlkTAT0BeWA3nBnybkymmHlXK8fVX9CDVb5YAJNAcAWxAgMBAAGjgZIwgY8wHQYD
+VR0OBBYEFBraZNxyzWkaWHiBzAZtbwHZGPE2MB8GA1UdIwQYMBaAFBraZNxyzWka
+WHiBzAZtbwHZGPE2MBoGA1UdEQQTMBGCCWxvY2FsaG9zdIcEfwAAATAMBgNVHRMB
+Af8EAjAAMA4GA1UdDwEB/wQEAwIFoDATBgNVHSUEDDAKBggrBgEFBQcDATANBgkq
+hkiG9w0BAQsFAAOCAQEAAdKJB77Isjv53fkJiCrL34RJ9LwFQLRiPL/lH7FQhenh
+bjoavDv39xEgIZ/rzvICUaBEXcyJPVFEkM8CrBUrf7U+wljqIrQRkwrxkPXF8eXE
+vJGvPfGboere/X1lOM/V+z5v8AQwxFR+W8sC2L1ec1UnqAeqh/7T9yYuhhKBr6zR
+3WTNbhet7wXNydA784iPV+5ftL9krBsiMoHAkgXcQnuqzjBy01OHKASMe0e7y66C
+WcZ6QEcgXlNBksg7l3bKOgT+/dxIbEM0Wa+t80YkkUc3SuT836qK+rwnuFNzzKpJ
+GPqvHm229Azp33cDIOZf2wncm3+9U/OK7eqzZyqdQQ==
 -----END CERTIFICATE-----

tests/integration_test.rs

@@ -142,7 +142,9 @@ async fn test_forbidden_error_handling() -> anyhow::Result<()> {
         ..Default::default()
     };
 
-    let pool = libredfish::RedfishClientPool::builder().build()?;
+    let pool = libredfish::RedfishClientPool::builder()
+        .danger_accept_invalid_certs()
+        .build()?;
     let redfish = pool.create_standard_client(endpoint)?;
 
     match redfish.get_chassis_all().await {
@@ -301,7 +303,9 @@ async fn run_integration_test(
         ..Default::default()
     };
 
-    let pool = libredfish::RedfishClientPool::builder().build()?;
+    let pool = libredfish::RedfishClientPool::builder()
+        .danger_accept_invalid_certs()
+        .build()?;
     let redfish = pool.create_client(endpoint).await?;
 
     if vendor_dir == "nvidia_dpu" {
@@ -443,7 +447,7 @@ async fn run_integration_test(
     if vendor_dir == "lenovo" {
         assert!(redfish.lockdown_status().await?.is_fully_enabled());
     }
-    if vendor_dir != "nvidia_gh200" {
+    if vendor_dir != "nvidia_gh200" && vendor_dir != "liteon_powershelf" {
         let tm = redfish.get_thermal_metrics().await?;
         if vendor_dir == "nvidia_gb200" {
             assert!(tm.leak_detectors.is_some());
@@ -452,7 +456,7 @@ async fn run_integration_test(
             _ = redfish.get_power_metrics().await?;
         }
     }
-    if vendor_dir != "supermicro" {
+    if vendor_dir != "supermicro" && vendor_dir != "liteon_powershelf" {
         _ = redfish.get_system_event_log().await?;
     }
 

tests/key.pem

@@ -1,28 +1,28 @@
 -----BEGIN PRIVATE KEY-----
-MIIEvQIBADANBgkqhkiG9w0BAQEFAASCBKcwggSjAgEAAoIBAQDTZXlmOaosH9VD
-BNquvP/fmqxrzcv4NsChrTTTX6NWaYL4b025+LR1qugVaP9SJy/Z9lYRNTwmhkpS
-tfgEr2iDlyDmuWf8K0FFYJRgjBGBj1H8J0EY+fiSn5eRIloLDvhkjIvL9l+7HqHw
-RLWb1hGcHNr6ztT4iZEFF/gxE+1dDf+aonEBlip06cqe+yxHIPaq62BVMDHAxS8l
-YiK9GFFgOPtY6lLgIQR5DE/WIfX9Pk/sV21MZ1NtUEAEIDwmEv3q4UY+k5gdwz/t
-HkQgDfJnkU2D3fVDOgMhOO1ca1DrL3+X8hiRabonrtFZw3QihOxmHPRY1TiAB6yV
-W0kuh+fxAgMBAAECggEADltu8k1qTFLhJgsXWxTFAAe+PBgfCT2WuaRM2So+qqjB
-12Of0MieYPt5hbK63HaC3nfHgqWt7yPhulpXfOH45C8IcgMXl93MMg0MJr58leMI
-+2ojFrIrerHSFm5R1TxwDEwrVm/mMowzDWFtQCc6zPJ8wNn5RuP48HKfTZ3/2fjw
-zEjSwPO2wFMfo1EJNTjlI303lFbdFBs67NaX6puh30M7Tn+gznHKyO5a7F57wkIt
-fkgnEy/sgMedQlwX7bRpUoD6f0fZzV8Qz4cHFywtYErczZJh3VGitJoO/VCIDdty
-RPXOAqVDd7EpP1UUehZlKVWZ0OZMEfRgKbRCel5abQKBgQDwgwrIQ5+BiZv6a0VT
-ETeXB+hRbvBinRykNo/RvLc3j1enRh9/zO/ShadZIXgOAiM1Jnr5Gp8KkNGca6K1
-myhtad7xYPODYzNXXp6T1OPgZxHZLIYzVUj6ypXeV64Te5ZiDaJ1D49czsq+PqsQ
-XRcgBJSNpFtDFiXWpjXWfx8PxwKBgQDhAnLY5Sl2eeQo+ud0MvjwftB/mN2qCzJY
-5AlQpRI4ThWxJgGPuHTR29zVa5iWNYuA5LWrC1y/wx+t5HKUwq+5kxvs+npYpDJD
-ZX/w0Glc6s0Jc/mFySkbw9B2LePedL7lRF5OiAyC6D106Sc9V2jlL4IflmOzt4CD
-ZTNbLtC6hwKBgHfIzBXxl/9sCcMuqdg1Ovp9dbcZCaATn7ApfHd5BccmHQGyav27
-k7XF2xMJGEHhzqcqAxUNrSgV+E9vTBomrHvRvrd5Ec7eGTPqbBA0d0nMC5eeFTh7
-wV0miH20LX6Gjt9G6yJiHYSbeV5G1+vOcTYBEft5X/qJjU7aePXbWh0BAoGBAJlV
-5tgCCuhvFloK6fHYzqZtdT6O+PfpW20SMXrgkvMF22h2YvgDFrDwqKRUB47NfHzg
-3yBpxNH1ccA5/w97QO8w3gX3h6qicpJVOAPusu6cIBACFZfjRv1hyszOZwvw+Soa
-Fj5kHkqTY1YpkREPYS9V2dIW1Wjic1SXgZDw7VM/AoGAP/cZ3ZHTSCDTFlItqy5C
-rIy2AiY0WJsx+K0qcvtosPOOwtnGjWHb1gdaVdfX/IRkSsX4PAOdnsyidNC5/l/m
-y8oa+5WEeGFclWFhr4dnTA766o8HrM2UjIgWWYBF2VKdptGnHxFeJWFUmeQC/xeW
-w37pCS7ykL+7gp7V0WShYsw=
+MIIEvQIBADANBgkqhkiG9w0BAQEFAASCBKcwggSjAgEAAoIBAQCpqpzhK3Gyn9l4
+6pm4fFPk23kX6taXnQw8w6UWtFM3lqmsanhRSr+7RE2FDNcReXL01ftk7JZpz0yr
+sWXiHXrJRKJBry2btLjxwEcVE8UywbIDoLFbUwyIpW7YCv0Z+xkFNJXXZJFz1Ad7
+4yMrGd3ob2KWcVvDXKnRRDFqDrC3anAFlHuxfIA+AOAIOGps6B/o4bBkwOZp7aWW
+aETd0XdkH5/R9x3osIZDkUaKk8GEXbzdipDWndBHTDr9myaWQvCutQVzScl5l+U+
+pX7pOpK2TzC2neeCZXmanXI+VngtvlkTAT0BeWA3nBnybkymmHlXK8fVX9CDVb5Y
+AJNAcAWxAgMBAAECggEAAXJ6GhWTx7FDWputqfwtfwM8Oi7/m5V0WERW8SWinBPF
+6LmTwyW/EurXOm5Mm1KM33e/yDGYatigz8oYk2Aq/vJ968+BtRVKio1YTd64TOGI
+Ei3ZtWy066l1k5Qsan9QzFGRgZ/S+cjyM2VRlHmJAgWJMr1OyhDNRgKt5Tiw2y+i
+rFj0erHT1WAFdyoJKozWsGsXvkOCkO8yMN8oGBXpKfeZhUdf2oolR58f4slRF0CL
+hZfc3ybNJ6s/MCfO7CS06UbboHH0EeJyB68pi715jJrML6QrV1ZpNViRcZfoa+iI
++qXRypZTS/NaS4tnhUb5jiLbg8yl2BO88gbtd0aTTQKBgQDXKbYlaxpU722VTn1b
+qvY/xdr1A0l4TKyW9NeYvQG5UBNQX86J3dVL5H0C5v1rPnmSZMu3R3oE/gKyMGU6
+o5MRtomYVPwNApDCyF1FVjKs0ZkxZxGyXMixu/isCJGH4WQE0k/PNWa+l95VI8av
+yklTSn5TvaqJafl4kh1DIxbhuwKBgQDJ3lQcu3sdebCspOSUq/XjOZD2S15R+aRg
+kzWNyrZwvSTRGrxwXF+pTfMPA6qLIsuKH0Rch4ET4vpfZqEMfD66zE+mOYP/ZsNl
+Q9GB+hACQCTratzRFFLMj4TQNhD0C8ze7lo3HRQO80Asz7VSW9pyBSlDP3nn+B7l
+QDi5z3nZgwKBgQClOIWDJERbFmQqs9P1UUm43ODvJjADrbUWUjtAMVsihwfdv0Ql
+f+/rUDBiyyesc5P9Q2NfgQaE0N0XIayKI9OqVWZBUHkgMGki+aMNvEw7gtPFKOVg
+9niyaqpgyE+gbQZV4X2hmtsj3hC/hjJrCMXXrKtm+Iz86iDEYkcQJ+DJQwKBgDL4
+tHxcyoVyyK2PJTh1Ay1Qx82o0jQYuOy5vvgCi3aEC9vQyGwGpxD04b6Uymc/be1+
+X6CNhoYi9d1I6LliVeSM2j+6nh3LMONM8IXFQm2PDMN0xDT20MFeK8b3H+i/YgeY
+X6cy7HGp8jZebAZTsyC/NOAKExoDiKmSQcCJJtdJAoGAYdck3veztSI8uP18gduh
+u4QIjwtZg9eLrhBxadxXr9yfd4hR3kJ+fObiqtIUTPVG1O8SZaQAZzyB65ArInPW
+PvbRDxtiSvaRclKDtgs4j2b4mtq3eJeSY7sv0nqwNHZBEa2hKjF+Ep5w6khl15KC
+7o+f3FaeSJdagWHJkJP+JTI=
 -----END PRIVATE KEY-----

tests/mockups/liteon_powershelf/redfish/v1/index.json

@@ -59,4 +59,4 @@
     "UpdateService": {
         "@odata.id": "/redfish/v1/UpdateService"
     }
-}
+}