Fix reporting available scenes on RGB devices

- Add coverage for scenes

pywizlight/bulb.py

@@ -6,6 +6,8 @@
 import time
 from typing import Any, Callable, Dict, List, Optional, Tuple, Union, cast
 
+from pywizlight.exceptions import WizLightNotKnownBulb
+
 from pywizlight.bulblibrary import BulbType
 from pywizlight.exceptions import (
     WizLightConnectionError,
@@ -453,7 +455,9 @@ async def get_bulbtype(self) -> BulbType:
         bulb_config = await self.getBulbConfig()
         result = bulb_config["result"]
         if "moduleName" not in result:
-            raise ValueError("Unable to determine bulb type.")
+            raise WizLightNotKnownBulb(
+                "Unable to determine the bulb type; moduleName is missing from getSystemConfig"
+            )
         white_range = await self.getExtendedWhiteRange()
         white_to_color_ratio = None
         white_channels = None

pywizlight/bulblibrary.py

@@ -82,19 +82,14 @@ def from_data(
         white_channels: Optional[int],
         white_to_color_ratio: Optional[int],
     ) -> "BulbType":
-        if kelvin_list:
-            kelvin_range: Optional[KelvinRange] = KelvinRange(
-                min=int(min(kelvin_list)), max=int(max(kelvin_list))
-            )
-        else:
-            kelvin_range = None
-
         try:
             # parse the features from name
             _identifier = module_name.split("_")[1]
         # Throw exception if index can not be found
         except IndexError:
-            raise WizLightNotKnownBulb("The bulb type can not be determined!")
+            raise WizLightNotKnownBulb(
+                f"The bulb type could not be determined from the module name: {module_name}"
+            )
 
         if "RGB" in _identifier:  # full RGB bulb
             bulb_type = BulbClass.RGB
@@ -105,6 +100,17 @@ def from_data(
         else:  # Plain brightness-only bulb
             bulb_type = BulbClass.DW
 
+        if kelvin_list:
+            kelvin_range: Optional[KelvinRange] = KelvinRange(
+                min=int(min(kelvin_list)), max=int(max(kelvin_list))
+            )
+        elif bulb_type in (BulbClass.RGB, BulbClass.TW):
+            raise WizLightNotKnownBulb(
+                f"Unable to determine required kelvin range for a {bulb_type.value} device"
+            )
+        else:
+            kelvin_range = None
+
         features = FEATURE_MAP[bulb_type]
 
         return BulbType(

pywizlight/scenes.py

@@ -44,7 +44,7 @@
 DW_SCENES = [9, 10, 13, 14, 29, 30, 31, 32]
 
 SCENES_BY_CLASS: Dict[BulbClass, List[str]] = {
-    BulbClass.RGB: list(cast(Iterable, SCENES)),
+    BulbClass.RGB: list(cast(Iterable, SCENES.values())),
     BulbClass.TW: [SCENES[key] for key in TW_SCENES],
     BulbClass.DW: [SCENES[key] for key in DW_SCENES],
 }

pywizlight/tests/fake_bulb.py

@@ -58,6 +58,20 @@
             "drvIface": 0,
         },
     },
+    ("ESP21_SHTW_01", "1.25.0"): {
+        "method": "getModelConfig",
+        "env": "pro",
+        "result": {
+            "ps": 2,
+            "pwmFreq": 5000,
+            "pwmRange": [1, 100],
+            "wcr": 20,
+            "nowc": 1,
+            "cctRange": [2700, 2700, 5000, 5000],
+            "renderFactor": [255, 0, 255, 255, 0, 0, 0, 0, 0, 0],
+            "drvIface": 0,
+        },
+    },
 }
 
 SYSTEM_CONFIGS = {
@@ -157,6 +171,36 @@
             "drvConf": [20, 1],
         },
     },
+    ("ESP20_SHRGBC_01", "1.21.4"): {
+        "method": "getSystemConfig",
+        "env": "pro",
+        "result": {
+            "mac": "6c2990e493bc",
+            "homeId": 5385975,
+            "roomId": 8016844,
+            "moduleName": "ESP20_SHRGBC_01",
+            "fwVersion": "1.21.4",
+            "groupId": 0,
+            "drvConf": [30, 1],
+            "ewf": [200, 255, 150, 255, 0, 0, 40],
+            "ewfHex": "c8ff96ff000028",
+            "ping": 0,
+        },
+    },
+    ("ESP21_SHTW_01", "1.25.0"): {
+        "method": "getSystemConfig",
+        "env": "pro",
+        "result": {
+            "mac": "6c29905a067c",
+            "homeId": 5385975,
+            "roomId": 8016844,
+            "rgn": "eu",
+            "moduleName": "ESP21_SHTW_01",
+            "fwVersion": "1.25.0",
+            "groupId": 0,
+            "ping": 0,
+        },
+    },
 }
 
 USER_CONFIGS = {
@@ -189,6 +233,20 @@
             "po": False,
         },
     },
+    ("ESP20_SHRGBC_01", "1.21.4"): {
+        "method": "getUserConfig",
+        "env": "pro",
+        "result": {
+            "fadeIn": 500,
+            "fadeOut": 500,
+            "dftDim": 100,
+            "pwmRange": [0, 100],
+            "whiteRange": [2700, 6500],
+            "extRange": [2200, 6500],
+            "opMode": 0,
+            "po": True,
+        },
+    },
 }
 
 

pywizlight/tests/test_bulb_dimmable_white.py

@@ -30,3 +30,18 @@ async def test_model_description_dimmable_bulb(dimmable_bulb: wizlight) -> None:
         white_channels=1,
         white_to_color_ratio=20,
     )
+
+
+@pytest.mark.asyncio
+async def test_supported_scenes(dimmable_bulb: wizlight) -> None:
+    """Test supported scenes."""
+    assert await dimmable_bulb.getSupportedScenes() == [
+        "Wake up",
+        "Bedtime",
+        "Cool white",
+        "Night light",
+        "Candlelight",
+        "Golden white",
+        "Pulse",
+        "Steampunk",
+    ]

pywizlight/tests/test_bulb_light_strip_1_25_0.py

@@ -25,6 +25,46 @@ async def test_setting_rgbww(light_strip: wizlight) -> None:
     assert state and state.get_rgbww() == (1, 2, 3, 4, 5)
 
 
+@pytest.mark.asyncio
+async def test_supported_scenes(light_strip: wizlight) -> None:
+    """Test supported scenes."""
+    assert await light_strip.getSupportedScenes() == [
+        "Ocean",
+        "Romance",
+        "Sunset",
+        "Party",
+        "Fireplace",
+        "Cozy",
+        "Forest",
+        "Pastel Colors",
+        "Wake up",
+        "Bedtime",
+        "Warm White",
+        "Daylight",
+        "Cool white",
+        "Night light",
+        "Focus",
+        "Relax",
+        "True colors",
+        "TV time",
+        "Plantgrowth",
+        "Spring",
+        "Summer",
+        "Fall",
+        "Deepdive",
+        "Jungle",
+        "Mojito",
+        "Club",
+        "Christmas",
+        "Halloween",
+        "Candlelight",
+        "Golden white",
+        "Pulse",
+        "Steampunk",
+        "Rhythm",
+    ]
+
+
 @pytest.mark.asyncio
 async def test_model_description_light_strip(light_strip: wizlight) -> None:
     """Test fetching the model description for a light strip."""

pywizlight/tests/test_bulb_rgbtw_1_21_4.py

@@ -0,0 +1,72 @@
+"""Tests for the Bulb API with a rgbtw bulb."""
+from typing import AsyncGenerator
+
+import pytest
+
+from pywizlight import wizlight
+from pywizlight.bulblibrary import BulbClass, BulbType, Features, KelvinRange
+from pywizlight.tests.fake_bulb import startup_bulb
+
+
+@pytest.fixture()
+async def dimmable_bulb() -> AsyncGenerator[wizlight, None]:
+    shutdown = startup_bulb(module_name="ESP20_SHRGBC_01", firmware_version="1.21.4")
+    bulb = wizlight(ip="127.0.0.1")
+    yield bulb
+    await bulb.async_close()
+    shutdown()
+
+
+@pytest.mark.asyncio
+async def test_model_description_dimmable_bulb(dimmable_bulb: wizlight) -> None:
+    """Test fetching the model description dimmable bulb."""
+    bulb_type = await dimmable_bulb.get_bulbtype()
+    assert bulb_type == BulbType(
+        features=Features(color=True, color_tmp=True, effect=True, brightness=True),
+        name="ESP20_SHRGBC_01",
+        kelvin_range=KelvinRange(max=6500, min=2200),
+        bulb_type=BulbClass.RGB,
+        fw_version="1.21.4",
+        white_channels=1,
+        white_to_color_ratio=30,
+    )
+
+
+@pytest.mark.asyncio
+async def test_supported_scenes(dimmable_bulb: wizlight) -> None:
+    """Test supported scenes."""
+    assert await dimmable_bulb.getSupportedScenes() == [
+        "Ocean",
+        "Romance",
+        "Sunset",
+        "Party",
+        "Fireplace",
+        "Cozy",
+        "Forest",
+        "Pastel Colors",
+        "Wake up",
+        "Bedtime",
+        "Warm White",
+        "Daylight",
+        "Cool white",
+        "Night light",
+        "Focus",
+        "Relax",
+        "True colors",
+        "TV time",
+        "Plantgrowth",
+        "Spring",
+        "Summer",
+        "Fall",
+        "Deepdive",
+        "Jungle",
+        "Mojito",
+        "Club",
+        "Christmas",
+        "Halloween",
+        "Candlelight",
+        "Golden white",
+        "Pulse",
+        "Steampunk",
+        "Rhythm",
+    ]

pywizlight/tests/test_bulb_socket.py

@@ -30,3 +30,9 @@ async def test_model_description_socket(socket: wizlight) -> None:
         white_channels=2,
         white_to_color_ratio=20,
     )
+
+
+@pytest.mark.asyncio
+async def test_supported_scenes(socket: wizlight) -> None:
+    """Test supported scenes."""
+    assert await socket.getSupportedScenes() == []

pywizlight/tests/test_bulb_turnable_white.py

@@ -0,0 +1,53 @@
+"""Tests for the Bulb API with a light strip."""
+from typing import AsyncGenerator
+
+import pytest
+
+from pywizlight import wizlight
+from pywizlight.bulblibrary import BulbClass, BulbType, Features, KelvinRange
+from pywizlight.tests.fake_bulb import startup_bulb
+
+
+@pytest.fixture()
+async def turnable_bulb() -> AsyncGenerator[wizlight, None]:
+    shutdown = startup_bulb(module_name="ESP21_SHTW_01", firmware_version="1.25.0")
+    bulb = wizlight(ip="127.0.0.1")
+    yield bulb
+    await bulb.async_close()
+    shutdown()
+
+
+@pytest.mark.asyncio
+async def test_model_description_dimmable_bulb(turnable_bulb: wizlight) -> None:
+    """Test fetching the model description dimmable bulb."""
+    bulb_type = await turnable_bulb.get_bulbtype()
+    assert bulb_type == BulbType(
+        features=Features(color=False, color_tmp=True, effect=True, brightness=True),
+        name="ESP21_SHTW_01",
+        kelvin_range=KelvinRange(max=5000, min=2700),
+        bulb_type=BulbClass.TW,
+        fw_version="1.25.0",
+        white_channels=1,
+        white_to_color_ratio=20,
+    )
+
+
+@pytest.mark.asyncio
+async def test_supported_scenes(turnable_bulb: wizlight) -> None:
+    """Test supported scenes."""
+    assert await turnable_bulb.getSupportedScenes() == [
+        "Cozy",
+        "Wake up",
+        "Bedtime",
+        "Warm White",
+        "Daylight",
+        "Cool white",
+        "Night light",
+        "Focus",
+        "Relax",
+        "TV time",
+        "Candlelight",
+        "Golden white",
+        "Pulse",
+        "Steampunk",
+    ]