action #88612
closedBeautify the output of error when errors found during AutoYaST profile validation
0%
Description
Motivation¶
Currently, error output in validate_autoyast_profile
function is shown without readable separation between errors and list of executed queries. So that it is not so easy to get what is wrong by just looking in autoinst-log.txt
Please see the current_not_formatted.png screenshot for how it looks.
If format the output, then it is easy to read and clarify what is wrong. Please see the how_looks_formatted.png screenshot for how it looks when formatted.
Task¶
Beautify the output of errors found during AutoYaST profile validation, so that it is easy to determine of what is wrong.
Assumption: it can be simply made by replacing "\n" characters in the string with the actual 'new line' character (this is what I actually did when got the formatted output).
Further info¶
We could use data we have in yaml to show in the output which node we have expected and also print xpath for debugging the test, in case there is an issue with this part.
We already upload autoyast profile, so we can skip printing parts of the generated xml.
Files
Updated by oorlov about 3 years ago
- File how_looks_formatted.png how_looks_formatted.png added
- File current_not_formatted.png current_not_formatted.png added
Updated by riafarov about 3 years ago
- Description updated (diff)
- Status changed from New to Workable
Updated by JERiveraMoya about 3 years ago
- Status changed from Workable to In Progress
- Assignee set to JERiveraMoya
Updated by JERiveraMoya about 3 years ago
- Status changed from In Progress to Feedback
Updated by JERiveraMoya about 3 years ago
A bit of beautification was added in #88606, as agreed I left on feedback.
Updated by JERiveraMoya about 3 years ago
- Status changed from Feedback to Closed
Updated by JERiveraMoya about 3 years ago
- Status changed from Closed to Feedback
Updated by JERiveraMoya about 3 years ago
Agreed to leave it in feedback for a while so we can evaluate if needed more 'beautification' after tasks for unit testing/documentation have been completed.
Updated by JERiveraMoya about 3 years ago
- Status changed from Feedback to Closed
documentation added in #89539