Mirrored_Repos/python-kasa
1
0
Fork 0
mirror of https://github.com/python-kasa/python-kasa.git synced 2025-04-26 16:46:23 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Updated valid range to 360 (with passing tests) (#153)

Browse Source 
* Updated valid range to 360

with it set to 359 it will not show the color red. Just tested this with a buddies bulb - same model/fw
https://imgur.com/a/kSNZIuL

* Updated valid range to 360

with it set to 359 it will not show the color red. Just tested this with a buddies bulb - same model/fw
https://imgur.com/a/kSNZIuL

* Update valid range to 360

* Fix tests
This commit is contained in:
Annika Jacobs 2019-01-16 20:50:48 +00:00 committed by Teemu R
parent 6115d96c39
commit 4cb28a3b25
3 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
README.md
View File

@ -210,7 +210,7 @@ if bulb.is_variable_color_temp:
### Setting the color ### Setting the color
Hue is given in degrees (0-359) and saturation and value in percentage. Hue is given in degrees (0-360) and saturation and value in percentage.
```python ```python
print(bulb.hsv) print(bulb.hsv)

4
pyHS100/smartbulb.py
View File

@ -142,10 +142,10 @@ class SmartBulb(SmartDevice):
if not self.is_color: if not self.is_color:
return None return None
if not isinstance(state[0], int) or not (0 <= state[0] <= 359): if not isinstance(state[0], int) or not (0 <= state[0] <= 360):
raise SmartDeviceException( raise SmartDeviceException(
'Invalid hue value: {} ' 'Invalid hue value: {} '
'(valid range: 0-359)'.format(state[0])) '(valid range: 0-360)'.format(state[0]))
if not isinstance(state[1], int) or not (0 <= state[1] <= 100): if not isinstance(state[1], int) or not (0 <= state[1] <= 100):
raise SmartDeviceException( raise SmartDeviceException(

8
pyHS100/tests/test_bulb.py
View File

@ -51,7 +51,7 @@ class TestSmartBulb(TestCase):
'light_state': { 'light_state': {
'brightness': All(int, Range(min=0, max=100)), 'brightness': All(int, Range(min=0, max=100)),
'color_temp': int, 'color_temp': int,
'hue': All(int, Range(min=0, max=359)), 'hue': All(int, Range(min=0, max=360)),
'mode': str, 'mode': str,
'on_off': check_int_bool, 'on_off': check_int_bool,
'saturation': All(int, Range(min=0, max=255)), 'saturation': All(int, Range(min=0, max=255)),
@ -63,7 +63,7 @@ class TestSmartBulb(TestCase):
'preferred_state': [{ 'preferred_state': [{
'brightness': All(int, Range(min=0, max=100)), 'brightness': All(int, Range(min=0, max=100)),
'color_temp': int, 'color_temp': int,
'hue': All(int, Range(min=0, max=359)), 'hue': All(int, Range(min=0, max=360)),
'index': int, 'index': int,
'saturation': All(int, Range(min=0, max=255)), 'saturation': All(int, Range(min=0, max=255)),
}], }],
@ -210,10 +210,10 @@ class TestSmartBulb(TestCase):
def test_hsv(self): def test_hsv(self):
hue, saturation, brightness = self.bulb.hsv hue, saturation, brightness = self.bulb.hsv
self.assertTrue(0 <= hue <= 359) self.assertTrue(0 <= hue <= 360)
self.assertTrue(0 <= saturation <= 100) self.assertTrue(0 <= saturation <= 100)
self.assertTrue(0 <= brightness <= 100) self.assertTrue(0 <= brightness <= 100)
for invalid_hue in [-1, 360, 0.5]: for invalid_hue in [-1, 361, 0.5]:
with self.assertRaises(SmartDeviceException): with self.assertRaises(SmartDeviceException):
self.bulb.hsv = (invalid_hue, 0, 0) self.bulb.hsv = (invalid_hue, 0, 0)