clamp effect brightness to ensure it cannot be set above the max value (#437)

clamp brightness to ensure it cannot be set above the max value

Author: bdraco

flux_led/aiodevice.py

@@ -428,6 +428,7 @@ async def async_set_effect(
         if effect == EFFECT_RANDOM:
             await self.async_set_random()
             return
+        brightness = min(100, max(0, brightness))
         if effect == EFFECT_MUSIC:
             await self.async_set_music_mode(brightness=brightness)
             return

tests/test_aio.py

@@ -934,6 +934,15 @@ def _updated_callback(*args, **kwargs):
     counter_byte = transport.mock_calls[0][1][0][7]
     assert counter_byte == 0
 
+    transport.reset_mock()
+    # Verify brightness clamped
+    await light.async_set_effect("RBM 1", 50, brightness=500)
+    assert transport.mock_calls[0][0] == "write"
+    assert (
+        transport.mock_calls[0][1][0]
+        == b"\xb0\xb1\xb2\xb3\x00\x01\x01\x01\x00\x05B\x012d\xd9\x80"
+    )
+
 
 @pytest.mark.asyncio
 async def test_SK6812RGBW(mock_aio_protocol, caplog: pytest.LogCaptureFixture):