S500D support

[smatterchoo]

S500D support

[smatterchoo]

Actually although get_device_info() fails, get_device_info_json() does work. it looks like the "overheated" field is now named "overheat_status" for this device. Here is the json with all keys and most values (some redacted for privacy).

{'avatar': 'switch_s500d', 'brightness': 50, 
'default_states': {'re_power_type': 'always_off', 're_power_type_capability': ['last_states', 'always_on', 'always_off'], 'type': 'last_states'}, 
'device_id': 'xxxx', 'device_on': False, 'fw_id': '00000000000000000000000000000000', 
'fw_ver': '1.1.1 Build 240130 Rel.090201', 'has_set_location_info': True, 
'hw_id': 'xxxx', 'hw_ver': '1.0', 'ip': 'xxxx', 'lang': 'en_US', 
'latitude': xxx, 'longitude': -xxx, 'mac': 'xxxx', 'model': 'S500D', 
'nickname': 'xxxxx=', 'oem_id': 'xxxxx', 'on_time': 0, 
'overheat_status': 'normal', 'region': 'America/Los_Angeles', 
'rssi': -43, 'signal_level': 3, 'specs': '', 'ssid': 'xxxxx', 'time_diff': -48\0, 
'type': 'SMART.TAPOSWITCH'}  

In case it's helpful to other users, you can roll your own device info from the json like this:

    device_info_dict = await device.get_device_info_json()
    DeviceInfoRecord = namedtuple('DeviceInfoRecord', device_info_dict.keys())
    device_info = DeviceInfoRecord(**device_info_dict)

    print(f"Device is {'on' if device_info.device_on else 'off'}")
    if device_info.device_on: 

..etc

[mihai-dinculescu]

Thank you for raising this issue.

In the short term, I have removed the overheated property from DeviceInfoGenericResult.

In the long term, I think it's worth adding first-class support for S500D devices.
It looks very similar to light bulb devices, which can be turned on/off and have their brightness adjusted.

Thank you for posting the device info, that's very helpful!

[smatterchoo]

As one user of your library, I would be very happy if you do not first-class the S500D but instead make the generic tapo device more functional. get_device_info_json() is perfect but if you could add set_device_info_json(<key/val dict>), it would support all devices (that use the same authentication method) and there's no new work for you every time Tapo introduces a new device or change.