Closed
Bug 868290
Opened 12 years ago
Closed 12 years ago
Need to recognize gaia-unit as something other than a generic "U"
Categories
(Tree Management Graveyard :: TBPL, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: philor, Assigned: ewong)
References
Details
Attachments
(1 file, 1 obsolete file)
|
1.80 KB,
patch
|
emorley
:
review+
|
Details | Diff | Splinter Review |
"b2g_ubuntu32_vm cedar opt test gaia-unit"
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → ewong
Status: NEW → ASSIGNED
| Assignee | ||
Comment 1•12 years ago
|
||
Attachment #744977 -
Flags: review?(emorley)
Comment 2•12 years ago
|
||
Comment on attachment 744977 [details] [diff] [review]
Add Gaia-unit to tbpl. (v1)
Thank you for the patch - mostly looks good, just need to:
* Check with how these tests differ from the Gaia UI smoketests, and as such how we should name both (and what symbols to use). Asked in bug 865379 comment 19.
* Remove the "Opt" from the test description, since opt/debug isn't put there - as the symbols/descriptions will be used on any row on TBPL (ie need to be generic).
Attachment #744977 -
Flags: review?(emorley)
Comment 3•12 years ago
|
||
Comment on attachment 744977 [details] [diff] [review]
Add Gaia-unit to tbpl. (v1)
I think best bet here is to add the unit tests as:
"Gaia Unit Test" : "G"
...and rename the UI tests from "G" to "Gu" - since that way around seems more intuitive.
| Assignee | ||
Comment 4•12 years ago
|
||
Attachment #744977 -
Attachment is obsolete: true
Attachment #746721 -
Flags: review?(emorley)
Comment 5•12 years ago
|
||
Comment on attachment 746721 [details] [diff] [review]
Add Gaia-unit to tbpl. (v2)
Looks good, thank you :-)
Attachment #746721 -
Flags: review?(emorley) → review+
Comment 7•12 years ago
|
||
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: Webtools → Tree Management
Updated•10 years ago
|
Product: Tree Management → Tree Management Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•