TST: Added test for https://gitlab.com/linkahead/linkahead-server/-/issues/280 and update changelog

Summary

Test that the server returns identifying information (name or id) of a missing obligatory property.
Test for https://gitlab.indiscale.com/caosdb/customers/f-fit/management/-/issues/95 / https://gitlab.com/linkahead/linkahead-server/-/issues/280

Check List for the Author

• 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)
• Annotations in code (Gitlab comments)

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?

tests/test_issues_server.py

@@ -1588,3 +1588,25 @@ def test_268_insert_nonexistent_file():
     with pytest.raises(TransactionError) as tre:
         db.File(path='IDoNotExist').insert()
     assert repr(tre) == "File does not exist: IDoNotExist"  # TODO Replace by proper string.
+
+
+@pytest.mark.xfail(reason="https://gitlab.com/linkahead/linkahead-server/-/issues/280")
+def test_issue_280():
+    """
+    Test that the error message returned when an obligatory property is missing
+    contains information on the missing property.
+
+    See https://gitlab.com/linkahead/linkahead-server/-/issues/280
+    """
+    prop = db.Property(name="TestProp", datatype=db.TEXT)
+    prop.insert()
+
+    rt = db.RecordType(name="TestRT")
+    rt.add_property(prop, importance=db.OBLIGATORY)
+    rt.insert()
+
+    # Check that error message contains name or id of missing property
+    rec = db.Record(name="TestRec").add_parent(rt)
+    with pytest.raises(TransactionError) as e:
+        rec.insert()
+    assert (prop.name in str(e.value)) or (str(prop.id) in str(e.value))