summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorScott Gasch <[email protected]>2022-02-04 08:23:14 -0800
committerScott Gasch <[email protected]>2022-02-04 08:23:14 -0800
commit94c8a54eadbf8552fa4d33dc349616d125e1a638 (patch)
treeb72e672361e36b0c4c9d6823e98b8dafb75d3881
parent90dc74b2ff8f36dd2495632a8150ce37dab61f1c (diff)
Fix a bug in device.py around the type of keywords, add logging and
add manage_switch_off_timers to the deps run at each checkin.
-rw-r--r--smart_home/device.py4
-rw-r--r--smart_home/lights.py1
-rwxr-xr-xtests/run_some_dependencies_test.py1
3 files changed, 4 insertions, 2 deletions
diff --git a/smart_home/device.py b/smart_home/device.py
index 02717a3..06187d7 100644
--- a/smart_home/device.py
+++ b/smart_home/device.py
@@ -11,14 +11,14 @@ class Device(object):
self,
name: str,
mac: str,
- keywords: Optional[List[str]],
+ keywords: Optional[str] = "",
):
self.name = name
self.mac = mac
self.keywords = keywords
self.arper = arper.Arper()
if keywords is not None:
- self.kws = keywords
+ self.kws: List[str] = keywords.split(' ')
else:
self.kws = []
diff --git a/smart_home/lights.py b/smart_home/lights.py
index 80bfffa..ac1cc88 100644
--- a/smart_home/lights.py
+++ b/smart_home/lights.py
@@ -329,6 +329,7 @@ class TPLinkLight(BaseLight):
@timeout(10.0, use_signals=False, error_message="Timed out waiting for tplink.py")
def get_info(self) -> Optional[Dict]:
cmd = self.get_cmdline() + "-c info"
+ logger.debug(f'Getting status of {self.mac} via "{cmd}"...')
out = subprocess.getoutput(cmd)
logger.debug(f'RAW OUT> {out}')
out = re.sub("Sent:.*\n", "", out)
diff --git a/tests/run_some_dependencies_test.py b/tests/run_some_dependencies_test.py
index 7e87d44..20f44cb 100755
--- a/tests/run_some_dependencies_test.py
+++ b/tests/run_some_dependencies_test.py
@@ -26,6 +26,7 @@ class RunSomeDependenciesTest(unittest.TestCase):
"/home/scott/bin/tplink.py -a office_lights -c info --audit_import_events >& /dev/null",
"/home/scott/bin/unscramble.py ethyropadratoyzrhoiectmi --trace_memory >& /dev/null",
"/home/scott/bin/cron.py --command='sleep 0' --lockfile=/tmp/deleteme_lock >& /dev/null",
+ "/home/scott/cron/manage_switch_off_timers.py >& /dev/null",
]
for command in commands:
try: