Data validation (#109)

home-assistant-libs · Nov 12, 2017 · 3898172 · 3898172
1 parent ec6d395
commit 3898172
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/pytradfri/device.py b/pytradfri/device.py
@@ -191,11 +191,8 @@ def set_dimmer(self, dimmer, *, index=0, transition_time=None):
         dimmer: Integer between 0..254
         transition_time: Integer representing tenth of a second (default None)
         """
-        # Set bounds for safety.
-        if dimmer < 0:
-            dimmer = 0
-        elif dimmer > 254:
-            dimmer = 254
+        if dimmer < 0 or dimmer > 254:
+            raise ValueError('Dimmer value must be between 0 and 254.')
 
         values = {
             ATTR_LIGHT_DIMMER: dimmer