@@ -1,15 +1,17 @@
+"""Module for tapo-branded smart bulbs (L5**)."""
 from typing import Dict, List, Optional
 
+from ..exceptions import SmartDeviceException
 from ..smartbulb import HSV, ColorTempRange, SmartBulb, SmartBulbPreset
 from .tapodevice import TapoDevice
 
 AVAILABLE_EFFECTS = {
-    'L1': "Party",
+    "L1": "Party",
     "L2": "Relax",
 }
 
-class TapoBulb(TapoDevice, SmartBulb):
 
+class TapoBulb(TapoDevice, SmartBulb):
     @property
     def is_color(self) -> bool:
         # TODO: this makes an assumption that only color bulbs report this
@@ -27,11 +29,16 @@ def is_variable_color_temp(self) -> bool:
 
     @property
     def valid_temperature_range(self) -> ColorTempRange:
-        ct_range = self._info.get("color_temp_range")
+        """Return the device-specific white temperature range (in Kelvin).
+
+        :return: White temperature range in Kelvin (minimum, maximum)
+        """
+        ct_range = self._info.get("color_temp_range", [0, 0])
         return ColorTempRange(min=ct_range[0], max=ct_range[1])
 
     @property
     def has_effects(self) -> bool:
+        """Return True if the device supports effects."""
         return "dynamic_light_effect_enable" in self._info
 
     @property
@@ -65,40 +72,83 @@ def effect_list(self) -> Optional[List[str]]:
         Example:
             ['Party', 'Relax', ...]
         """
-        return AVAILABLE_EFFECTS.keys() if self.has_effects else None
-
+        return list(AVAILABLE_EFFECTS.keys()) if self.has_effects else None
 
     @property
     def hsv(self) -> HSV:
-        h, s, v = self._info.get("hue"), self._info.get("saturation"), self._info.get("brightness")
+        """Return the current HSV state of the bulb.
+
+        :return: hue, saturation and value (degrees, %, %)
+        """
+        if not self.is_color:
+            raise SmartDeviceException("Bulb does not support color.")
+
+        h, s, v = (
+            self._info.get("hue", 0),
+            self._info.get("saturation", 0),
+            self._info.get("brightness", 0),
+        )
 
         return HSV(hue=h, saturation=s, value=v)
 
     @property
     def color_temp(self) -> int:
-        return self._info.get("color_temp")
+        """Whether the bulb supports color temperature changes."""
+        if not self.is_variable_color_temp:
+            raise SmartDeviceException("Bulb does not support colortemp.")
+
+        return self._info.get("color_temp", -1)
 
     @property
     def brightness(self) -> int:
-        return self._info.get("brightness")
+        """Return the current brightness in percentage."""
+        if not self.is_dimmable:  # pragma: no cover
+            raise SmartDeviceException("Bulb is not dimmable.")
 
-    async def set_hsv(self, hue: int, saturation: int, value: Optional[int] = None, *,
-                      transition: Optional[int] = None) -> Dict:
-        return await self.protocol.query({"set_device_info": {
-            "hue": hue,
-            "saturation": saturation,
-            "brightness": value,
-        }})
+        return self._info.get("brightness", -1)
 
-    async def set_color_temp(self, temp: int, *, brightness=None, transition: Optional[int] = None) -> Dict:
+    async def set_hsv(
+        self,
+        hue: int,
+        saturation: int,
+        value: Optional[int] = None,
+        *,
+        transition: Optional[int] = None,
+    ) -> Dict:
+        if not self.is_color:
+            raise SmartDeviceException("Bulb does not support color.")
+
+        return await self.protocol.query(
+            {
+                "set_device_info": {
+                    "hue": hue,
+                    "saturation": saturation,
+                    "brightness": value,
+                }
+            }
+        )
+
+    async def set_color_temp(
+        self, temp: int, *, brightness=None, transition: Optional[int] = None
+    ) -> Dict:
         # TODO: Decide how to handle brightness and transition
-        # TODO: Note, trying to set brightness at the same time with color_temp causes error -1008
-        return await self.protocol.query({"set_device_info": {"color_temp": temp}})
+        # TODO: Note, trying to set brightness at the same time
+        #  with color_temp causes error -1008
+        if not self.is_variable_color_temp:
+            raise SmartDeviceException("Bulb does not support colortemp.")
 
+        return await self.protocol.query({"set_device_info": {"color_temp": temp}})
 
-    async def set_brightness(self, brightness: int, *, transition: Optional[int] = None) -> Dict:
+    async def set_brightness(
+        self, brightness: int, *, transition: Optional[int] = None
+    ) -> Dict:
         # TODO: Decide how to handle transitions
-        return await self.protocol.query({"set_device_info": {"brightness": brightness}})
+        if not self.is_dimmable:  # pragma: no cover
+            raise SmartDeviceException("Bulb is not dimmable.")
+
+        return await self.protocol.query(
+            {"set_device_info": {"brightness": brightness}}
+        )
 
     # Default state information, should be made to settings
     """
@@ -123,10 +173,16 @@ async def set_effect(
         transition: Optional[int] = None,
     ) -> None:
         """Set an effect on the device."""
-        raise NotImplementedError()  # TODO: the code above does not seem to activate the effect
-        return await self.protocol.query({"set_device_info": {"dynamic_light_effect_enable": 1,
-                                                              "dynamic_light_effect_id": effect}})
-
+        raise NotImplementedError()
+        # TODO: the code below does to activate the effect but gives no error
+        return await self.protocol.query(
+            {
+                "set_device_info": {
+                    "dynamic_light_effect_enable": 1,
+                    "dynamic_light_effect_id": effect,
+                }
+            }
+        )
 
     @property
     def presets(self) -> List[SmartBulbPreset]: