[Bug 52049] Update telepathy-gabble-tests to show pass/fail tests status

bugzilla-daemon at freedesktop.org bugzilla-daemon at freedesktop.org
Tue Jul 31 00:30:44 CEST 2012


https://bugs.freedesktop.org/show_bug.cgi?id=52049

--- Comment #8 from Luis Araujo <luis.araujo at collabora.co.uk> 2012-07-30 22:30:44 UTC ---
(In reply to comment #7)
> Is there any reason not to use "PASS: $i", "SKIP: $i", "FAIL: $i" like Automake
> does? (Or "FAIL: $i ($e)" would be OK if you'd prefer to have the status
> included.)
> 

No special reason, this will work just fine.

> I would much prefer the pass/fail/skip to be at the beginning of the line like
> that: "^FAIL" is an easier thing to grep for than "^Result.*fail", and much
> less likely to appear during normal operation as a result of a debug message or
> something. If you're going to screen-scrape test results like this, make them
> as unambiguous as possible.
> 

Agreed, I will submit new patch with these changes.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
You are the assignee for the bug.



More information about the telepathy-bugs mailing list