fix(protocol): forward music_segments in A3 device config

flux_led/protocol.py

@@ -2332,7 +2332,7 @@ def construct_device_config(
                 pixels_per_segment,
                 segments,
                 music_pixels_per_segment,
-                segments,
+                music_segments,
             ),
             inner_pre_constructed=True,
         )

tests/test_aio.py

@@ -37,6 +37,8 @@
     PowerRestoreState,
     PowerRestoreStates,
     ProtocolLEDENET25Byte,
+    ProtocolLEDENETAddressableA2,
+    ProtocolLEDENETAddressableA3,
     ProtocolLEDENETCCT,
     ProtocolLEDENETCCTWrapped,
     RemoteConfig,
@@ -1269,6 +1271,23 @@ def _updated_callback(*args, **kwargs):
     )
 
 
+def test_addressable_a3_device_config_forwards_music_segments():
+    """A3 must forward the caller's music_segments, not substitute segments.
+
+    A3 wraps A2's device-config message. With pixels_per_segment above
+    MUSIC_PIXELS_PER_SEGMENT_MAX the music-sync override in A2 is skipped, so a
+    distinct music_segments value survives to the wire. The A2 inner message
+    therefore appears verbatim inside the A3 wrapper only when A3 forwards every
+    parameter unchanged.
+    """
+    # segments=4, music_pixels_per_segment=100, music_segments=5 (distinct from
+    # segments). pixels_per_segment=300 (> 150) disables the music-sync merge.
+    args = (0, 0, 4, 300, 4, 100, 5)
+    inner = ProtocolLEDENETAddressableA2().construct_device_config(*args)
+    wrapped = ProtocolLEDENETAddressableA3().construct_device_config(*args)
+    assert bytes(inner) in bytes(wrapped)
+
+
 @pytest.mark.asyncio
 async def test_async_set_zones(mock_aio_protocol, caplog: pytest.LogCaptureFixture):
     """Test we can set set zone colors."""