Closed
Bug 1026778
Opened 10 years ago
Closed 10 years ago
Add test for smart collection pinning
Categories
(Firefox OS Graveyard :: Gaia::Everything.me, defect)
Tracking
(b2g-v2.0 fixed, b2g-v2.1 fixed)
RESOLVED
FIXED
People
(Reporter: kgrandon, Assigned: kgrandon)
References
Details
(Whiteboard: [p=3],[systemsfe])
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
amirn
:
review+
ranbena
:
review+
bajaj
:
approval-gaia-v2.0+
|
Details |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
Work in progress...
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8441755 [details]
Github pull request
Ran, Amir - wondering if you guys could give this one a review. It's a test and not part of the build, so we don't need to be 100% accurate and syntax correct. This is more-so to inform you guys of what I'm doing and answer any questions you might have about the integration testing framework.
I think we're in a unique position where we can have really well tested functionality. Let me know what you think!
Attachment #8441755 -
Flags: review?(ran)
Attachment #8441755 -
Flags: review?(amirn)
Comment 3•10 years ago
|
||
Comment on attachment 8441755 [details]
Github pull request
looks great. thanks!
Attachment #8441755 -
Flags: review?(amirn) → review+
Comment 4•10 years ago
|
||
Comment on attachment 8441755 [details]
Github pull request
Code looks good. Didn't run it.
Attachment #8441755 -
Flags: review?(ran) → review+
Updated•10 years ago
|
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking-]
Assignee | ||
Comment 5•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•10 years ago
|
||
Comment on attachment 8441755 [details]
Github pull request
This is needed for the vertical homescreen. We've done our best effort at testing this and believe it is safe for uplift.
Attachment #8441755 -
Flags: approval-gaia-v2.0?(bbajaj)
Updated•10 years ago
|
Attachment #8441755 -
Flags: approval-gaia-v2.0?(bbajaj) → approval-gaia-v2.0+
Assignee | ||
Comment 7•10 years ago
|
||
status-b2g-v2.0:
--- → fixed
status-b2g-v2.1:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•