chore: Only emit traits that have any values set

Author: allenporter

roborock/devices/traits/v1/__init__.py

@@ -254,7 +254,9 @@ def as_dict(self) -> dict[str, Any]:
             trait = getattr(self, item.name, None)
             if trait is None or not isinstance(trait, RoborockBase):
                 continue
-            result[item.name] = trait.as_dict()
+            data = trait.as_dict()
+            if data:  # Don't omit unset traits
+                result[item.name] = data
         return result
 
 

tests/devices/__snapshots__/test_v1_device.ambr

@@ -194,8 +194,6 @@
           1734458038,
         ]),
       }),
-      'consumables': dict({
-      }),
       'device_features': dict({
         'isActivateVideoChargingAndStandbySupported': False,
         'isAnalysisSupported': False,
@@ -370,23 +368,9 @@
         'isWifiManageSupported': False,
         'isWorkdayHolidaySupported': False,
       }),
-      'dnd': dict({
-      }),
-      'home': dict({
-      }),
-      'map_content': dict({
-      }),
-      'maps': dict({
-      }),
       'network_info': dict({
         'ip': '',
       }),
-      'rooms': dict({
-      }),
-      'sound_volume': dict({
-      }),
-      'status': dict({
-      }),
     }),
   })
 # ---
@@ -558,10 +542,6 @@
       ]),
     }),
     'traits': dict({
-      'clean_summary': dict({
-      }),
-      'consumables': dict({
-      }),
       'device_features': dict({
         'isActivateVideoChargingAndStandbySupported': False,
         'isAnalysisSupported': False,
@@ -743,21 +723,9 @@
         'startHour': 22,
         'startMinute': 0,
       }),
-      'home': dict({
-      }),
-      'map_content': dict({
-      }),
-      'maps': dict({
-      }),
       'network_info': dict({
         'ip': '',
       }),
-      'rooms': dict({
-      }),
-      'sound_volume': dict({
-      }),
-      'status': dict({
-      }),
     }),
   })
 # ---
@@ -929,10 +897,6 @@
       ]),
     }),
     'traits': dict({
-      'clean_summary': dict({
-      }),
-      'consumables': dict({
-      }),
       'device_features': dict({
         'isActivateVideoChargingAndStandbySupported': False,
         'isAnalysisSupported': False,
@@ -1107,21 +1071,9 @@
         'isWifiManageSupported': False,
         'isWorkdayHolidaySupported': False,
       }),
-      'dnd': dict({
-      }),
-      'home': dict({
-      }),
-      'map_content': dict({
-      }),
-      'maps': dict({
-      }),
       'network_info': dict({
         'ip': '',
       }),
-      'rooms': dict({
-      }),
-      'sound_volume': dict({
-      }),
       'status': dict({
         'battery': 100,
         'cleanArea': 91287500,
@@ -1319,10 +1271,6 @@
       ]),
     }),
     'traits': dict({
-      'clean_summary': dict({
-      }),
-      'consumables': dict({
-      }),
       'device_features': dict({
         'isActivateVideoChargingAndStandbySupported': False,
         'isAnalysisSupported': False,
@@ -1497,24 +1445,12 @@
         'isWifiManageSupported': False,
         'isWorkdayHolidaySupported': False,
       }),
-      'dnd': dict({
-      }),
-      'home': dict({
-      }),
-      'map_content': dict({
-      }),
-      'maps': dict({
-      }),
       'network_info': dict({
         'ip': '',
       }),
-      'rooms': dict({
-      }),
       'sound_volume': dict({
         'volume': 90,
       }),
-      'status': dict({
-      }),
     }),
   })
 # ---