reload with global_ctx="*" now starts triggers, reported by Fabio C

Author: craigbarratt

custom_components/pyscript/__init__.py

@@ -130,7 +130,7 @@ def start_global_contexts(global_ctx_only: str = None) -> None:
         idx = global_ctx_name.find(".")
         if idx < 0 or global_ctx_name[0:idx] not in {"file", "apps", "scripts"}:
             continue
-        if global_ctx_only is not None:
+        if global_ctx_only is not None and global_ctx_only != "*":
             if global_ctx_name != global_ctx_only and not global_ctx_name.startswith(global_ctx_only + "."):
                 continue
         global_ctx.set_auto_start(True)

docs/new_features.rst

@@ -52,6 +52,7 @@ Breaking changes since 1.2.1 include:
 Bug fixes since 1.2.1 include:
 
 - Fixed ``@state_trigger`` with only a ``.old`` variable, which previously never triggered; reported by stigvi.
+- Reload with global_ctx="*" now starts triggers, reported by Fabio C.
 - Fixed subscripts when running python 3.9.x.
 - Timeouts that implement time triggers might infrequenctly occur a tiny time before the target time. A fix was added
   to do an additional short timeout when there is an early timeout, to make sure any time trigger occurs at or shortly

tests/test_reload.py

@@ -47,6 +47,10 @@ def func1():
 @service
 def func2():
     pass
+
+@time_trigger
+def startup():
+    log.info(f"{__name__} is starting up")
 """,
         #
         # This will load, since there is an apps/world2 config entry
@@ -266,6 +270,7 @@ def shutdown(trigger_time=None):
         #
         for i in range(3):
             assert caplog.text.count("world global_ctx=apps.world xyz=") == 2 + i
+            assert caplog.text.count("world is starting up") == 2 + i
             assert caplog.text.count("world2 global_ctx=apps.world2 var1=") == 3 + i
             assert caplog.text.count("hello global_ctx=file.hello xyz=") == 4 + i
             assert caplog.text.count("modules/xyz2/other global_ctx=modules.xyz2.other") == 2 + i