address pr comments

Author: marcintyminski

tests/test_develco.py

@@ -182,15 +182,10 @@ async def test_frient_emi_current_summation_write_request(zigpy_device_from_v2_q
         await manufacturer_cluster.write_attributes({current_summation_attr_id: 1234})
 
         assert request_mock.call_count == 1
-        assert request_mock.call_args[0] == ()
-        assert request_mock.call_args[1]["cluster"] == Metering.cluster_id
-        assert (
-            request_mock.call_args[1]["data"]
-            == b"\x04\xd2\x04\x01\x02\x01\x03%\xd2\x04\x00\x00\x00\x00"
-        )
+        assert request_mock.call_args.kwargs["cluster"] == Metering.cluster_id
 
         zcl_header, attr_data = foundation.ZCLHeader.deserialize(
-            request_mock.call_args[1]["data"]
+            request_mock.call_args.kwargs["data"]
         )
         assert (
             zcl_header.frame_control.frame_type == foundation.FrameType.GLOBAL_COMMAND

zhaquirks/develco/emi_led.py

@@ -50,12 +50,13 @@ class AttributeDefs(BaseAttributeDefs):
         fallback_name="Pulse configuration",
     )
     .number(
-        # Allow a user to set the current summation value, so it can show the same value as the physical meter.
+        # Allow a user to set the current summation value,
+        # so it can show the same value as the physical meter.
         attribute_name=ManufacturerMetering.AttributeDefs.current_summation.name,
         cluster_id=ManufacturerMetering.cluster_id,
         endpoint_id=2,
         min_value=0,
-        max_value=281474976710655,
+        max_value=0xFFFFFFFFFFFF,  # uint48 max value
         step=1,
         unit="Wh",
         mode="box",