From ed5d8341e07cee30de0959496f1db076d7e296c4 Mon Sep 17 00:00:00 2001 From: JarbasAi Date: Tue, 8 Sep 2026 00:37:22 +0100 Subject: [PATCH] fix: read record[skill_id] not record[owner] in scheduler delegation test ovos-bus-client 2.11.14a1 renamed the SCHEDULER-1 record identity key from owner to skill_id in the live get()/list() view a consumer sees; the from_stored migration shim in schedules.py only covers the on-disk read boundary, not this view. ovos_workshop has no "owner" reader at all, so the old assertion was asserting a key the current library never produces. Bisected the published wheels on a clean origin/dev worktree: 2.11.12a1, 2.11.13a1, 2.11.13a3 and 2.11.13a5 give 44 passed in this file; 2.11.14a1 and 2.11.14a2 give 1 failed, 43 passed with KeyError owner at the assertion. After this change: 44 passed in the file and 702 passed with 18 subtests across test/unittests against ovos-bus-client 2.11.14a2. Co-Authored-By: Claude Sonnet 5 --- test/unittests/test_scheduler_delegation.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/unittests/test_scheduler_delegation.py b/test/unittests/test_scheduler_delegation.py index b95785a7..18189067 100644 --- a/test/unittests/test_scheduler_delegation.py +++ b/test/unittests/test_scheduler_delegation.py @@ -193,7 +193,7 @@ def test_a_delay_in_seconds_becomes_a_relative_timing(self): record = self.record("ring") self.assertEqual(record["in"], {"seconds": 60}) self.assertEqual(record["event"], "test.scheduler.ring") - self.assertEqual(record["owner"], self.skill_id) + self.assertEqual(record["skill_id"], self.skill_id) self.assertEqual(record["data"], {"k": 1}) def test_an_aware_datetime_becomes_the_instant_it_names(self):