From 94c8a54eadbf8552fa4d33dc349616d125e1a638 Mon Sep 17 00:00:00 2001 From: Scott Gasch Date: Fri, 4 Feb 2022 08:23:14 -0800 Subject: [PATCH] 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. --- smart_home/device.py | 4 ++-- smart_home/lights.py | 1 + tests/run_some_dependencies_test.py | 1 + 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: -- 2.45.2