Changeset e198832 in flexoentity

Timestamp:

10/26/25 17:35:48 (3 months ago)

Author:

Enrico Schwass <ennoausberlin@…>

Branches:

master

Children:

3bad43e

Parents:

c98728b

Message:

fix test errors

Files:

• flexoentity/flexo_entity.py (modified) (1 diff)
• tests/test_id_lifecycle.py (modified) (1 diff)
• tests/test_id_stress.py (modified) (1 diff)

flexoentity/flexo_entity.py

@@ -231 +231 @@
         return False
 
-   def _transition(self, target_state: EntityState):
-    """
-    Internal helper for state transitions with version/fingerprint checks
-    and forward-only enforcement.
-    """
-
-    allowed = self.allowed_transitions()
-
-    if target_state.name not in allowed:
-        raise ValueError(
-            f"Illegal state transition: {self.state.name} → {target_state.name}. "
-            f"Allowed: {', '.join(allowed) or 'none'}"
-        )
-
-    # special case: marking obsolete
-    if target_state == EntityState.OBSOLETE:
+    def _transition(self, target_state: EntityState):
+        """
+        Internal helper for state transitions with version/fingerprint checks
+        and forward-only enforcement.
+        """
+
+        allowed = self.allowed_transitions()
+
+        if target_state.name not in allowed:
+            raise ValueError(
+                f"Illegal state transition: {self.state.name} → {target_state.name}. "
+                f"Allowed: {', '.join(allowed) or 'none'}"
+            )
+
+        # special case: marking obsolete
+        if target_state == EntityState.OBSOLETE:
+            self.state = target_state
+            return
+
+        # version bump only for stable/external states
+        if self.should_version(target_state):
+            self._update_fingerprint()
+            self.flexo_id = FlexOID.next_version(self.flexo_id)
+
         self.state = target_state
-        return
-
-    # version bump only for stable/external states
-    if self.should_version(target_state):
-        self._update_fingerprint()
-        self.flexo_id = FlexOID.next_version(self.flexo_id)
-
-    self.state = target_state
 
     # ───────────────────────────────────────────────────────────────

tests/test_id_lifecycle.py

@@ -111 +111 @@
     q.approve()
     for _ in range(FlexOID.MAX_VERSION - 2):
-        q.sign()
+        q.bump_version()
     with pytest.raises(RuntimeError, match="mark obsolete"):
         q.sign()

tests/test_id_stress.py

@@ -93 +93 @@
 
     # 999 → 1000 should raise RuntimeError
-    with pytest.raises(RuntimeError):
-        q.sign()
+    with pytest.raises(ValueError):
+        q.publish()