Closed
Bug 868290
Opened 11 years ago
Closed 11 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•11 years ago
|
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #744977 -
Flags: review?(emorley)
Comment 2•11 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•11 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•11 years ago
|
||
Attachment #744977 -
Attachment is obsolete: true
Attachment #746721 -
Flags: review?(emorley)
Comment 5•11 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•11 years ago
|
||
In production :-)
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
Product: Webtools → Tree Management
Updated•9 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
•