handle LED prefs

tests/unit/test_commands.py

@@ -320,6 +320,31 @@ async def test_set_channel_invalid_secret_length(command_handler):
         await command_handler.set_channel(1, "Test", b"tooshort")
 
 
+async def test_set_led_params(command_handler, mock_connection):
+    """set_led_params sends CMD_SET_LED_PARAMS (0x3e) with both LED mode bytes."""
+    await command_handler.set_led_params(2, 1)
+
+    sent = mock_connection.send.call_args[0][0]
+    assert sent == b"\x3e\x02\x01"
+
+
+async def test_set_other_params_no_led_bytes(command_handler, mock_connection):
+    """set_other_params_from_infos should NOT include LED bytes."""
+    infos = {
+        "manual_add_contacts": 0,
+        "telemetry_mode_base": 0,
+        "telemetry_mode_loc": 0,
+        "telemetry_mode_env": 0,
+        "adv_loc_policy": 0,
+        "multi_acks": 0,
+    }
+    await command_handler.set_other_params_from_infos(infos)
+
+    sent = mock_connection.send.call_args[0][0]
+    assert sent.startswith(b"\x26")
+    assert len(sent) == 5  # cmd(1) + 4 params, no LED bytes
+
+
 async def test_send_chan_msg_with_str_timestamp(command_handler, mock_connection):
     ts = 1620000000
     await command_handler.send_chan_msg(3, "world", timestamp=ts)

tests/unit/test_reader.py

@@ -85,5 +85,66 @@ async def test_binary_response():
         else:
             print(f"⚠️ Unknown request type {request_type}, no specific event expected")
 
+@pytest.mark.asyncio
+async def test_device_info_v11_has_led_fields():
+    """Firmware v11+ includes led_ble_mode and led_status_mode in DEVICE_INFO."""
+    mock_dispatcher = MockDispatcher()
+    reader = MessageReader(mock_dispatcher)
+
+    # Build a DEVICE_INFO packet: PacketType.DEVICE_INFO (13) + fw_ver=11 + fields
+    import struct
+    packet = bytearray()
+    packet.append(13)        # RESP_CODE_DEVICE_INFO
+    packet.append(11)        # FIRMWARE_VER_CODE = 11
+    packet.append(25)        # max_contacts / 2
+    packet.append(8)         # max_channels
+    packet += struct.pack("<I", 123456)  # ble_pin
+    packet += b"20250305    "[:12]       # fw_build (12 bytes)
+    packet += b"TestModel".ljust(40, b"\x00")  # model (40 bytes)
+    packet += b"1.0.0".ljust(20, b"\x00")      # ver (20 bytes)
+    packet.append(0)         # repeat (v9+)
+    packet.append(1)         # path_hash_mode (v10+)
+    packet.append(2)         # led_ble_mode (v11+) - LED_BLE_CONN_ONLY
+    packet.append(1)         # led_status_mode (v11+) - LED_STATUS_DISABLED
+
+    await reader.handle_rx(packet)
+
+    device_events = [e for e in mock_dispatcher.dispatched_events if e.type == EventType.DEVICE_INFO]
+    assert len(device_events) == 1
+    info = device_events[0].payload
+    assert info["fw ver"] == 11
+    assert info["led_ble_mode"] == 2
+    assert info["led_status_mode"] == 1
+
+
+@pytest.mark.asyncio
+async def test_device_info_v10_no_led_fields():
+    """Firmware v10 should not have LED fields."""
+    mock_dispatcher = MockDispatcher()
+    reader = MessageReader(mock_dispatcher)
+
+    import struct
+    packet = bytearray()
+    packet.append(13)        # RESP_CODE_DEVICE_INFO
+    packet.append(10)        # FIRMWARE_VER_CODE = 10
+    packet.append(25)        # max_contacts / 2
+    packet.append(8)         # max_channels
+    packet += struct.pack("<I", 0)       # ble_pin
+    packet += b"20250101    "[:12]       # fw_build
+    packet += b"OldModel".ljust(40, b"\x00")
+    packet += b"0.9.0".ljust(20, b"\x00")
+    packet.append(0)         # repeat (v9+)
+    packet.append(0)         # path_hash_mode (v10+)
+
+    await reader.handle_rx(packet)
+
+    device_events = [e for e in mock_dispatcher.dispatched_events if e.type == EventType.DEVICE_INFO]
+    assert len(device_events) == 1
+    info = device_events[0].payload
+    assert info["fw ver"] == 10
+    assert "led_ble_mode" not in info
+    assert "led_status_mode" not in info
+
+
 if __name__ == "__main__":
     asyncio.run(test_binary_response())