Closed
Bug 968696
Opened 12 years ago
Closed 12 years ago
[settings] add marionette test for Media storage panel
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(feature-b2g:2.0)
People
(Reporter: gasolin, Assigned: gasolin)
References
Details
(Whiteboard: [p=2])
Attachments
(1 file)
Overview Description:
Add marionette test for Media storage panel
Steps to Reproduce:
1) run make test-perf APP=settings
2) run make test-integration APP=settings
Actual Results:
No such integration test for Media storage panel
Expected Results:
Write that integration test and make test pass
Reproducibility:
100%
Additional Information:
Please refer to bug 956210 for more information.
| Assignee | ||
Updated•12 years ago
|
Assignee: nobody → gasolin
| Assignee | ||
Comment 1•12 years ago
|
||
will test:
* popup dialog when tap Format SD card
* toggle 'share using usb' (will check if it works on TBPL)
* media location is disabled by default
| Assignee | ||
Comment 2•12 years ago
|
||
| Assignee | ||
Comment 3•12 years ago
|
||
Comment on attachment 8406648 [details] [review]
pull request redirect to github
travis green, try server https://tbpl.mozilla.org/?tree=Try&rev=0fb6799ede5f
Attachment #8406648 -
Flags: review?(arthur.chen)
| Assignee | ||
Comment 4•12 years ago
|
||
| Assignee | ||
Updated•12 years ago
|
Status: NEW → ASSIGNED
| Assignee | ||
Comment 5•12 years ago
|
||
integration test passed
| Assignee | ||
Comment 6•12 years ago
|
||
update to see if non-related IME bug is passed https://tbpl.mozilla.org/?tree=Try&rev=5b9dc433dea0
| Assignee | ||
Updated•12 years ago
|
Target Milestone: --- → 1.4 S6 (25apr)
| Assignee | ||
Comment 7•12 years ago
|
||
add storage display test case and passed try server https://tbpl.mozilla.org/?tree=Try&rev=c70d47984502
| Assignee | ||
Updated•12 years ago
|
Target Milestone: 1.4 S6 (25apr) → 2.0 S1 (9may)
| Assignee | ||
Updated•12 years ago
|
Whiteboard: [p=2]
Comment 8•12 years ago
|
||
Comment on attachment 8406648 [details] [review]
pull request redirect to github
Thanks for the patch. Overall it looks good to me. One of the tests seem fail.
Attachment #8406648 -
Flags: review?(arthur.chen)
| Assignee | ||
Comment 9•12 years ago
|
||
Comment on attachment 8406648 [details] [review]
pull request redirect to github
Issue addressed, please kindly review again
Attachment #8406648 -
Flags: review?(arthur.chen)
Comment 10•12 years ago
|
||
Comment on attachment 8406648 [details] [review]
pull request redirect to github
r=me with travis green, thanks!
Attachment #8406648 -
Flags: review?(arthur.chen) → review+
| Assignee | ||
Comment 11•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
feature-b2g: --- → 2.0
You need to log in
before you can comment on or make changes to this bug.
Description
•