fix state trigger hold on unrelated state updates

custom_components/pyscript/decorators/state.py

@@ -281,7 +281,7 @@ async def _cycle(self) -> None:
                 elif ident_values_changed(self.last_func_args, self.state_trig_ident):
                     trig_ok = await self._is_trig_ok()
                 else:
-                    trig_ok = False
+                    continue
                 await self._check_new_state(trig_ok)
             except TimeoutError:
                 await self._check_state_hold()

tests/test_function.py

@@ -440,6 +440,13 @@ def func10d(var_name=None, value=None, trigger_type=None, context=None, old_valu
     log.info(f"func10d var = {var_name}, value = {value}, kwargs = {kwargs}")
     pyscript.done = [seq_num, var_name, kwargs]
 
+@state_trigger("pyscript.f11var1 == 'playing'", state_hold=0.05)
+def func11(var_name=None, value=None):
+    global seq_num
+
+    seq_num += 1
+    pyscript.done = [seq_num, var_name, value, pyscript.f11var1.position]
+
 """,
     )
     # initialize the trigger and active variables
@@ -672,6 +679,19 @@ def func10d(var_name=None, value=None, trigger_type=None, context=None, old_valu
     hass.states.async_set("pyscript.f8bvar1", 30)
     hass.states.async_set("pyscript.f8bvar1", 31)
 
+    #
+    # check that state_hold isn't cancelled by unrelated attribute-only updates
+    #
+    seq_num += 1
+    hass.states.async_set("pyscript.f11var1", "playing", {"position": 1})
+    hass.states.async_set("pyscript.f11var1", "playing", {"position": 2})
+    assert literal_eval(await wait_until_done(notify_q)) == [
+        seq_num,
+        "pyscript.f11var1",
+        "playing",
+        2,
+    ]
+
     #
     # check that state_var.old is None first time
     #