F remove deprecated parent

Summary

For #94 (closed). Removes the deprecated parent keyword for good.

Focus

Simply removed the special treatment; it now raises a ValueError like any other unknown keyword would.

Test Environment

Unit test is sufficient.

Check List for the Author

Please, prepare your MR for a review. Be sure to write a summary and a focus and create gitlab comments for the reviewer. They should guide the reviewer through the changes, explain your changes and also point out open questions. For further good practices have a look at our review guidelines

• All automated tests pass
• Reference related issues
• Up-to-date CHANGELOG.md (or not necessary)
• Up-to-date JSON schema (or not necessary)
• Appropriate user and developer documentation (or not necessary)
  • How do I use the software? Assume "stupid" users.
  • How do I develop or debug the software? Assume novice developers.
• Annotations in code (Gitlab comments)
  • Intent of new code
  • Problems with old code
  • Why this implementation?

Check List for the Reviewer

• I understand the intent of this MR
• All automated tests pass
• Up-to-date CHANGELOG.md (or not necessary)
• Appropriate user and developer documentation (or not necessary)
• The test environment setup works and the intended behavior is reproducible in the test environment
• In-code documentation and comments are up-to-date.
• Check: Are there specifications? Are they satisfied?

For further good practices have a look at our review guidelines.

unittests/test_yaml_model_parser.py

@@ -19,7 +19,7 @@
 import unittest
 from datetime import date
 from tempfile import NamedTemporaryFile
-from pytest import deprecated_call, raises, mark
+from pytest import raises, mark
 
 import linkahead as db
 from caosadvancedtools.models.parser import (TwiceDefinedException,
@@ -527,7 +527,7 @@ F:
 
 
 def test_issue_36():
-    """Test whether the `parent` keyword is deprecated.
+    """Test whether the `parent` keyword is removed.
 
     See https://gitlab.com/caosdb/caosdb-advanced-user-tools/-/issues/36.
 
@@ -550,17 +550,12 @@ R3:
   inherit_from_obligatory:
   - R1
 """
-    with deprecated_call():
-        # Check whether this is actually deprecated
+    with raises(ValueError) as ve:
+        # The keyword has been removed, so it should raise a regular ValueError.
         model = parse_model_from_string(model_string)
 
-    assert "R3" in model
-    r3 = model["R3"]
-    assert isinstance(r3, db.RecordType)
-    for par in ["R1", "R2"]:
-        # Until removal, both do the same
-        assert has_parent(r3, par)
-        assert r3.get_parent(par)._flags["inheritance"] == db.OBLIGATORY
+    assert "invalid keyword" in str(ve.value)
+    assert "parent" in str(ve.value)
 
 
 def test_yaml_error():