action #7476

Support comments in tests

Added by coolo almost 5 years ago. Updated about 4 years ago.

Status:ResolvedStart date:12/05/2015
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:Feature requests
Target version:-
Difficulty:
Duration:

Description

We need to add and edit one comment per test - preferably rich text to be able to add links to screenshots


Related issues

Related to openQA Project - action #10212: labels and badges for builds Resolved 13/01/2016
Related to openQA Project - action #10148: better notification and user feedback Resolved 13/11/2015

History

#1 Updated by coolo almost 5 years ago

I forgot to mention: the comments need to have a flag "outdated"

#2 Updated by oholecek almost 5 years ago

One comment only? As max. one comment?

#3 Updated by coolo almost 5 years ago

Perhaps we shouldn't call it comment - but one "fail reason description" per test :)

#4 Updated by coolo almost 5 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.

#5 Updated by coolo almost 5 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

#6 Updated by coolo over 4 years ago

  • Status changed from In Progress to Resolved
  • Assignee set to coolo

#7 Updated by okurz about 4 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.

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

#8 Updated by okurz about 4 years ago

#9 Updated by okurz about 4 years ago

  • Related to action #10148: better notification and user feedback added

#10 Updated by coolo about 4 years ago

  • Assignee changed from coolo to okurz
  • Target version deleted (Sprint 17)

#11 Updated by RBrownSUSE about 4 years ago

  • Assignee deleted (okurz)

#12 Updated by okurz about 4 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.

Also available in: Atom PDF