Project

General

Profile

Actions

action #7476

closed

Support comments in tests

Added by coolo over 9 years ago. Updated almost 9 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Feature requests
Target version:
-
Start date:
2015-05-12
Due date:
% Done:

100%

Estimated time:

Description

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


Related issues 2 (0 open2 closed)

Related to openQA Project - action #10212: labels and badges for buildsResolvedokurz2016-01-13

Actions
Related to openQA Project - action #10148: better notification and user feedbackResolved2015-11-13

Actions
Actions #1

Updated by coolo over 9 years ago

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

Actions #2

Updated by oholecek over 9 years ago

One comment only? As max. one comment?

Actions #3

Updated by coolo over 9 years ago

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

Actions #4

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.

Actions #5

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
Actions #6

Updated by coolo over 9 years ago

  • Status changed from In Progress to Resolved
  • Assignee set to coolo
Actions #7

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.

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
Actions #8

Updated by okurz almost 9 years ago

Actions #9

Updated by okurz almost 9 years ago

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

Updated by coolo almost 9 years ago

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

Updated by RBrownSUSE almost 9 years ago

  • Assignee deleted (okurz)
Actions #12

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.

Actions

Also available in: Atom PDF