XLSX-Konverter: Bessere Fehlermeldung bei inkorrektem Typ in Spalte, zusätzlicher Spalte

Summary

Type and Validation errors are now collected and formatted as a table. The error raised when there is an additional column not present in the schema now describes the problem. Addresses https://gitlab.indiscale.com/caosdb/customers/dimr/management/-/issues/251

Focus

• Does this solution improve readability and error identification? (is it a 'better output'?)
• Are all errors covered?

Test Environment

Manual testing.

Check List for the Author

• All automated tests pass
• Reference related issues
• Up-to-date CHANGELOG.md (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?

assigned to @i.nueske

added 1 commit

• a6910d39 - MNT: Added a warning when column metadata is not configured, and a better...

Compare with previous version

added 1 commit

• 0519fc38 - STY: style fix

Compare with previous version

marked the checklist item Annotations in code (Gitlab comments) as completed

requested review from @daniel

added 1 commit

• 22dac5d1 - WIP: XLSX converter errors.

Compare with previous version

added 1 commit

• d0f4f3f5 - MNT: Implement review feedback, extend test, some more comments

Compare with previous version

added 10 commits

• d0f4f3f5...dbd18de1 - 9 commits from branch dev
• 969f32d4 - Merge branch 'dev' into f-xlsx-converter-error-messages

Compare with previous version

added 1 commit

• 1a6a00ab - ENH: Even better error message.

Compare with previous version

added 1 commit

• 008b1bcf - DOC: if and only if

Compare with previous version

resolved all threads

marked the checklist item I understand the intent of this MR as completed

marked the checklist item Up-to-date CHANGELOG.md (or not necessary) as completed

marked the checklist item Appropriate user and developer documentation (or not necessary) as completed

marked the checklist item The test environment setup works and the intended behavior is reproducible in the test as completed

marked the checklist item In-code documentation and comments are up-to-date. as completed

marked the checklist item All automated tests pass as completed

mentioned in commit dccc2ce8

merged