action #7476
closedSupport comments in tests
100%
Description
We need to add and edit one comment per test - preferably rich text to be able to add links to screenshots
Updated by coolo over 9 years ago
I forgot to mention: the comments need to have a flag "outdated"
Updated by coolo over 9 years ago
Perhaps we shouldn't call it comment - but one "fail reason description" per test :)
Updated by coolo over 9 years ago
In discussion with Ludwig and Anja, I got my mind changed and I added multiple comments per test. How to show that nicely per job group is still up to experiments.
Updated by coolo over 9 years ago
- Status changed from New to In Progress
- % Done changed from 0 to 60
https://github.com/os-autoinst/openQA/pull/373 adds a database model and a start of the UI change.
What's left to do:
- write test cases
- hide comments - the database field is there, but that's it
- show job comments within the job group. right now only job group specific comments are displayed. I'm not so sure how it should look like, but you want to see what test the comment is from
Updated by coolo over 9 years ago
- Status changed from In Progress to Resolved
- Assignee set to coolo
Updated by okurz almost 9 years ago
- Status changed from Resolved to Feedback
we have comments and we use them. But the view gets cluttered and many comments are incorrectly rendered and we can not correct them. In discussions I mentioned there should be a "fail reason description" which is exactly what coolo mentioned here https://progress.opensuse.org/issues/7476#note-3
Can we please try to do this? E.g.
- have comments editable
- have a test description field, e.g. a field just below the "Overall summary" which we can edit to describe the current state, add bug links, describe who is working on it, etc. (see https://openqa.opensuse.org/tests/overview?distri=opensuse&version=Tumbleweed&build=20160116&groupid=1 for an example)
One might have remarks about "but we need an unchanged history" but
- I don't see a benefit of an "unchanged history" of broken comments. Without some hashsum we can never be sure anyway I did not mangle with the database
- With #7478 we have a history of changes
Updated by okurz almost 9 years ago
- Related to action #10212: labels and badges for builds added
Updated by okurz almost 9 years ago
- Related to action #10148: better notification and user feedback added
Updated by coolo almost 9 years ago
- Assignee changed from coolo to okurz
- Target version deleted (
Sprint 17)
Updated by okurz almost 9 years ago
- Status changed from Feedback to Resolved
- % Done changed from 60 to 100
ok, comments are not-editable because no one did it yet, not because of "political reasons" or similar. Work on "editable comments" is tracked in #10622 so we can close the current issue again.