Closed
Bug 920180
Opened 11 years ago
Closed 11 years ago
Update the gaia-ui endurance settings and sms tests for master; update manifest
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rwood, Assigned: rwood)
Details
Attachments
(1 file)
Updates to the gaia-ui endurance tests (settings and sms tests) for latest changes in master; includes addition to the gaiatest messages app; update manifest.
Assignee | ||
Comment 1•11 years ago
|
||
Attachment #809366 -
Flags: review?(bob.silverberg)
Assignee | ||
Comment 2•11 years ago
|
||
:bsilverberg, requesting a review of the addition to the gaiatest message app, thanks!
Comment 3•11 years ago
|
||
Comment on attachment 809366 [details] [review]
pull request pointer
I have added me comments to the PR.
Attachment #809366 -
Flags: review?(bob.silverberg) → review-
Assignee | ||
Updated•11 years ago
|
Attachment #809366 -
Flags: review- → review?(bob.silverberg)
Comment 4•11 years ago
|
||
Comment on attachment 809366 [details] [review]
pull request pointer
Comments in the pull request.
Attachment #809366 -
Flags: review?(bob.silverberg) → review-
Assignee | ||
Updated•11 years ago
|
Attachment #809366 -
Flags: review- → review?(bob.silverberg)
Comment 5•11 years ago
|
||
Comment on attachment 809366 [details] [review]
pull request pointer
Comments in the PR.
Attachment #809366 -
Flags: review?(bob.silverberg) → review-
Assignee | ||
Comment 6•11 years ago
|
||
Comment on attachment 809366 [details] [review]
pull request pointer
Updated the pull request
Attachment #809366 -
Flags: review- → review?(bob.silverberg)
Comment 7•11 years ago
|
||
Comment on attachment 809366 [details] [review]
pull request pointer
My last comment was just a question. The PR gets my r+ either way. For the changes to the app objects that is.
Attachment #809366 -
Flags: review?(bob.silverberg) → review+
Assignee | ||
Comment 8•11 years ago
|
||
Comment on attachment 809366 [details] [review]
pull request pointer
Thanks Bob.
Dave, would you mind doing a quick review of the rest of the changes in the pull request? Thanks!
Attachment #809366 -
Flags: review+ → review?(dave.hunt)
Comment 9•11 years ago
|
||
Comment on attachment 809366 [details] [review]
pull request pointer
One minor PEP8 issue, but you can carry my r+
Please squash your commits before merging.
Attachment #809366 -
Flags: review?(dave.hunt) → review+
Assignee | ||
Comment 10•11 years ago
|
||
Thanks Dave. Squashed and merged.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Attachment mime type: text/plain → text/x-github-pull-request
You need to log in
before you can comment on or make changes to this bug.
Description
•