fix(dell): Tolerate null values for TempRollupStatus on dell machines

src/model/oem/dell.rs

@@ -116,7 +116,7 @@ pub struct System {
     pub populated_pcie_slots: i64,
     pub power_cap_enabled_state: Option<String>, // We see this field explicitly returned as null by Dell XE9680s
     pub system_generation: String,
-    pub temp_rollup_status: String,
+    pub temp_rollup_status: Option<String>, // Observed as null if machine is off
     #[serde(rename = "UUID")]
     pub uuid: String,
     pub volt_rollup_status: String,

src/model/system.rs

@@ -382,6 +382,18 @@ mod test {
         assert_eq!(result.part_number.as_deref(), Some("0H28RRA06"));
     }
 
+    #[test]
+    fn test_system_dell_null_temp_rollup() {
+        let data = include_str!("testdata/system_dell_null_temp_rollup.json");
+        let result: super::ComputerSystem = serde_json::from_str(data).unwrap();
+        let dell = result
+            .oem
+            .expect("should contain OEM data")
+            .dell
+            .expect("should contain dell-specific OEM data");
+        assert!(dell.dell_system.temp_rollup_status.is_none());
+    }
+
     #[test]
     fn test_system_bluefield_boot_valid() {
         // Old firmware versions of Bluefield deliver empty values for Boot fields