Bug 1159191 - rlPhaseEnd truncates phase names containing ':' when reporting results back to Beaker
Summary: rlPhaseEnd truncates phase names containing ':' when reporting results back t...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: beakerlib
Version: rawhide
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
Assignee: Dalibor Pospíšil
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 1146918 (view as bug list)
Depends On:
Blocks: Fedora-beakerlib-1.10 1136362
TreeView+ depends on / blocked
 
Reported: 2014-10-31 07:53 UTC by Karel Srot
Modified: 2014-12-12 14:51 UTC (History)
8 users (show)

Fixed In Version: beakerlib-1.10-1
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-12-12 14:48:00 UTC
Type: Bug
Embargoed:

Attachments (Terms of Use)
patch (1.94 KB, patch)
2014-11-19 03:55 UTC, Dan Callaghan 		no flags Details | Diff
View All

Description Karel Srot 2014-10-31 07:53:49 UTC 
Description of problem:

I have a test with test phases:

rlPhaseStartTest "Scenario: foo"
and
rlPhaseStartTest "Scenario: bar"

tcms-results in that case warns about duplicate phase names.

 WARNING  Duplicate phase name Scenario in /CoreOS/...

It seems that it cuts-off all the text starting the ':' character.

Version-Release number of selected component (if applicable):
qa-tools-workstation-3.3-22.x86_64
Comment 1 Karel Srot 2014-10-31 09:03:33 UTC 
I just realized that the beaker is truncating phase names, not tcms-results.
Comment 2 Karel Srot 2014-10-31 09:06:09 UTC 
Reopening against the Beaker component.

If the test phase contain ':' character the phase name gets truncated.
Comment 3 Dan Callaghan 2014-11-19 03:54:34 UTC 
This is actually a bug in beakerlib's handling of phase names. Patch to come.
Comment 4 Dan Callaghan 2014-11-19 03:55:12 UTC 
Created attachment 958863 [details]
patch
Comment 5 Dan Callaghan 2014-11-20 07:41:20 UTC 
*** Bug 1146918 has been marked as a duplicate of this bug. ***
Comment 6 Dalibor Pospíšil 2014-11-20 12:24:51 UTC 
Thanks for patch. I will include it in the next release.
Comment 7 Dalibor Pospíšil 2014-11-20 16:50:03 UTC 
fixed by https://git.fedorahosted.org/cgit/beakerlib.git/commit/?id=2ac7f25fd77775949216d129178b4b3204b734d4
Note You need to log in before you can comment on or make changes to this bug.