F merge missing list datatype

Summary

For #151 (closed). Does not fix the actual problem because this is a CloseWon't Fix (or has to be fixed in the crawler) but raises a better error message instead.

Focus

Check that the correct error message is shown.

Test Environment

Unit tests should be 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)
• 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.

src/caosdb/common/models.py

@@ -62,7 +62,6 @@ from caosdb.exceptions import (AmbiguousEntityError, AuthorizationError,
                                EntityDoesNotExistError, EntityError,
                                EntityHasNoDatatypeError, HTTPURITooLongError,
                                MismatchingEntitiesError, QueryNotUniqueError,
-                               ServerConfigurationException,
                                TransactionError, UniqueNamesError,
                                UnqualifiedParentsError,
                                UnqualifiedPropertiesError)
@@ -1381,12 +1380,15 @@ def _parse_value(datatype, value):
             # reference via name
 
             return str(value)
-        except TypeError:
+        except TypeError as te:
             # deal with invalid XML: List of values without appropriate datatype
             if isinstance(value, list):
-                raise ServerConfigurationException(
-                    "The server sent an invalid XML: List valued properties must be announced by "
+                raise TypeError(
+                    "Invalid datatype: List valued properties must be announced by "
                     "the datatype.\n" + f"Datatype: {datatype}\nvalue: {value}")
+            else:
+                # Everything else that's not related to wrong list assignments
+                raise te
 
 
 def _log_request(request, xml_body=None):