Changeset 5c72356 in flexoentity for tests/test_id_lifecycle.py

Timestamp:

11/02/25 18:49:14 (2 months ago)

Author:

Enrico Schwass <ennoausberlin@…>

Branches:

master

Children:

bf30018

Parents:

8aa20c7

Message:

fix tests due to simplifying state and type

File:

• tests/test_id_lifecycle.py (modified) (4 diffs)

tests/test_id_lifecycle.py

@@ -20 +20 @@
     assert q.flexo_id.version == 1
 
-
-def test_signing_bumps_version(sample_question):
+def test_signing_does_not_bump_version(sample_question):
     q = sample_question
     q.approve()
-    v_before = str(q.flexo_id)
+    before = q.flexo_id
     q.sign()
+    after = q.flexo_id
+
+    # state changed
     assert q.state == EntityState.APPROVED_AND_SIGNED
-    assert str(q.flexo_id) != v_before
+
+    # version unchanged
+    assert before.version == after.version
+
+    # only suffix letter differs
+    assert before.prefix == after.prefix
+    assert before.state_code == "A"
+    assert after.state_code == "S"
 
 
-def test_publish_bumps_version(sample_question):
+def test_publish_does_not_bump_version(sample_question):
     q = sample_question
     q.approve()
@@ -37 +46 @@
     q.publish()
     assert q.state == EntityState.PUBLISHED
-    assert q.flexo_id.version == v_before + 1
+    assert q.flexo_id.version == v_before
 
 
@@ -70 +79 @@
     # simulate tampering
     q.text = "Tampered text"
+    print(FlexoEntity.debug_integrity(q))
     assert not FlexoEntity.verify_integrity(q)
 
@@ -112 +122 @@
     for _ in range(FlexOID.MAX_VERSION - 1):
         q.bump_version()
+
+    # Next one must raise
     with pytest.raises(RuntimeError, match="mark obsolete"):
-        q.sign()
+        q.bump_version()