fix: correct decorator order for deprecated _advance_pointer property and update tests to use public API

Author: JohnRichard4096

src/amrita_sense/runtime/workflow.py

@@ -407,12 +407,11 @@ async def run_step_by(self) -> AsyncGenerator[Any, None]:
             self._ret_addr_stack.clear()
             self._pointer.clear()
             self._jump_marked = False
-
+    @property
     @deprecated(
         "Method of `_advance_pointer` is now `advance_pointer`, this compile method will be removed in `v0.3.0`",
         category=DeprecationWarning,
     )
-    @property
     def _advance_pointer(self):
         return self.advance_pointer
 

tests/test_runtime.py

@@ -197,7 +197,7 @@ def simple_node():
         interpreter = WorkflowInterpreter(rendered)
         interpreter._pointer = PointerVector([0])
 
-        assert interpreter._advance_pointer()
+        assert interpreter.advance_pointer()
         assert interpreter._pointer == PointerVector([1, 0])
 
     def test_advance_pointer_backtrack_to_nested_sibling(self):
@@ -211,7 +211,7 @@ def simple_node():
         interpreter = WorkflowInterpreter(rendered)
         interpreter._pointer = PointerVector([0, 0])
 
-        assert interpreter._advance_pointer()
+        assert interpreter.advance_pointer()
         assert interpreter._pointer == PointerVector([1, 0])
 
     @pytest.mark.asyncio
@@ -279,10 +279,10 @@ def simple_node():
 
         interpreter = WorkflowInterpreter(NodeCompose(simple_node).render())
         interpreter._pointer = PointerVector()
-        assert not interpreter._advance_pointer()
+        assert not interpreter.advance_pointer()
 
         interpreter._pointer = PointerVector([0, 0])
-        assert not interpreter._advance_pointer()
+        assert not interpreter.advance_pointer()
 
     def test_advance_pointer_nested_and_backtrack(self):
         @NodeDecorator()
@@ -293,11 +293,11 @@ def simple_node():
         interpreter = WorkflowInterpreter(rendered)
 
         interpreter._pointer = PointerVector([0])
-        assert interpreter._advance_pointer()
+        assert interpreter.advance_pointer()
         assert interpreter._pointer == PointerVector([0, 0])
 
         interpreter._pointer = PointerVector([0, 0])
-        assert interpreter._advance_pointer()
+        assert interpreter.advance_pointer()
         assert interpreter._pointer == PointerVector([1])
 
     def test_advance_pointer_backtrack_from_nested_end(self):
@@ -309,4 +309,4 @@ def simple_node():
         interpreter = WorkflowInterpreter(rendered)
         interpreter._pointer = PointerVector([0, 0])
 
-        assert not interpreter._advance_pointer()
+        assert not interpreter.advance_pointer()