TEST: Test for server issue 217: faulty list property XML.

Summary

The server does not like to change the description if there are list valued reference properties.

This is for caosdb-server#217 (closed)

Focus

Does this reproduce the integration test problem in caosadvancedtools?

Test Environment

In directory tests, do pytest -k test_issue_217 test_issues_server.py

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 -> No
• Reference related issues.
• Up-to-date CHANGELOG.md (or not necessary)
• 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)
• 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.

tests/test_issues_server.py

 # -*- coding: utf-8 -*-
-#
-# ** header v3.0
 # This file is a part of the CaosDB Project.
 #
-# Copyright (c) 2020 IndiScale GmbH <info@indiscale.com>
-# Copyright (c) 2020 Daniel Hornung <d.hornung@indiscale.com>
+# Copyright (c) 2022 IndiScale GmbH <info@indiscale.com>
+# Copyright (c) 2022 Daniel Hornung <d.hornung@indiscale.com>
 # Copyright (c) 2020 Florian Spreckelsen <f.spreckelsen@indiscale.com>
 #
 # This program is free software: you can redistribute it and/or modify
@@ -19,8 +17,6 @@
 #
 # You should have received a copy of the GNU Affero General Public License
 # along with this program. If not, see <https://www.gnu.org/licenses/>.
-#
-# ** end header
 
 """Tests for issues on gitlab.com, project caosdb-server."""
 
@@ -603,3 +599,17 @@ def test_issue_132():
     # this one has the wront scope of the conjunction.
     query = "FIND RECORD TestRT WITH TestRT_Foo.TestP_Bar = val1 AND TestP_Baz = 'the other one'"
     assert len(db.execute_query(query)) == 0
+
+
+def test_issue_217():
+    """Server gets list property datatype wrong if description is updated."""
+
+    rt = db.RecordType(name="TestRT").insert()
+    # prop = db.Property(name="LP", datatype=db.LIST("TestRT")).insert()
+    r = db.Record().add_parent(id=rt.id).add_property(name=rt.name,
+                                                      datatype=db.LIST(db.INTEGER),
+                                                      value=[1, 2]).insert()
+    r.retrieve()
+    r.description = "Changed description"
+    r.update()
+    r.retrieve()                # This line fails in the bug report with invalid XML.