fix: Fix test533, ordering of entering states and exit/enter of parallel states

Author: fgmacedo

statemachine/engines/base.py

@@ -403,7 +403,7 @@ def _enter_states(  # noqa: C901
         )
 
         ordered_states = sorted(
-            states_to_enter, key=lambda x: x.source and x.source.document_order or 0
+            states_to_enter, key=lambda x: x.target and x.target.document_order or 0
         )
 
         # We update the configuration atomically

statemachine/event.py

@@ -157,7 +157,7 @@ def __call__(self, *args, **kwargs):
         # an SM instance. Such SM instance is provided by `__get__` method when
         # used as a property descriptor.
         self.put(*args, **kwargs)
-        return self._sm._processing_loop()
+        return self._sm._processing_loop()  # type: ignore
 
     def split(  # type: ignore[override]
         self, sep: "str | None" = None, maxsplit: int = -1

statemachine/io/scxml/actions.py

@@ -145,6 +145,9 @@ def __init__(self):
     def __call__(self, *args, **kwargs):
         raise NotImplementedError
 
+    def __str__(self):
+        return f"{self.action}"
+
     def __repr__(self):
         return f"{self.__class__.__name__}({self.action!r})"
 

statemachine/io/scxml/schema.py

@@ -9,11 +9,17 @@
 class Action:
     pass
 
+    def __str__(self):
+        return f"{self.__class__.__name__}"
+
 
 @dataclass
 class ExecutableContent:
     actions: List[Action] = field(default_factory=list)
 
+    def __str__(self):
+        return ", ".join(str(action) for action in self.actions)
+
     @property
     def is_empty(self):
         return not self.actions
@@ -41,6 +47,9 @@ class IfBranch(Action):
     cond: "str | None"
     actions: List[Action] = field(default_factory=list)
 
+    def __str__(self):
+        return self.cond
+
     def append(self, action: Action):
         self.actions.append(action)
 

statemachine/state.py

@@ -302,7 +302,7 @@ def parallel(self):
 
     @property
     def is_compound(self):
-        return bool(self.states)
+        return bool(self.states) and not self.parallel
 
     def ancestors(self, parent: "State | None" = None) -> Generator["State", None, None]:  # noqa: UP043
         selected = self.parent