Bug 9401 - patchtest: show assertion messages on summary
Summary: patchtest: show assertion messages on summary
Status: RESOLVED FIXED
Alias: None
Product: Patchwork/Patchtest
Classification: Yocto Project Subprojects
Component: Patchtest (show other bugs)
Version: 2.2
Hardware: x86 Multiple
: Medium+ enhancement
Target Milestone: 2.2 M1
Assignee: Leonardo Sandoval Gonzalez
QA Contact:
URL:
Whiteboard:
Depends on:
Blocks: 9403
  Show dependency tree
 
Reported: 2016-04-06 15:06 UTC by Leonardo Sandoval Gonzalez
Modified: 2017-12-06 17:30 UTC (History)
2 users (show)

See Also:
OS type for building Yocto: ---
Type of Regression: ---
Verified:
Documentation change: No (bug/feature does not impact docs)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Leonardo Sandoval Gonzalez 2016-04-06 15:06:56 UTC 
Currently, patchtest builds the summary based on test suite docstrings. This is useful but it does not give the precise information for the test failing. In order to show this info, we can use assertion messages; this have to be caught by patchtest and use this info on the summary.
Comment 1 Leonardo Sandoval Gonzalez 2016-06-14 14:18:24 UTC 
Besides the error, patchtest should provide the solution for the problem. So this would be the output for each test:

ID (id of the test)
RESULT (pass or fail)
Purpose (the reason for this test)
Failure (the exception msg)
Fix (the proposed fix)
Comment 2 Leonardo Sandoval Gonzalez 2016-06-23 19:52:15 UTC 
My fault, moving it to M1 again. CCB should agree on any milestone movement.
Comment 3 Leonardo Sandoval Gonzalez 2016-06-30 20:40:42 UTC 
Completed on the patchtest (ssh) repo.