Closed
Bug 969265
Opened 10 years ago
Closed 10 years ago
[Settings][meta] marionette tests and panel refactor for 1.4
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
1.4 S2 (28feb)
People
(Reporter: gasolin, Assigned: gasolin)
References
Details
(Whiteboard: [ucid:SystemPlatform30, 1.4, ft:system-platform])
marionette tests coverage and panel refactor for 1.4
Assignee | ||
Updated•10 years ago
|
Assignee | ||
Updated•10 years ago
|
Summary: [Settings][meta] marionette tests coverage and panel refactor for 1.4 → [Settings][meta] marionette tests and panel refactor for 1.4
Updated•10 years ago
|
Assignee | ||
Comment 1•10 years ago
|
||
include refactor for - do not track panel - help panel
Assignee | ||
Comment 2•10 years ago
|
||
ivan you can track Setting panel isolation tasks here
Flags: needinfo?(itsay)
Updated•10 years ago
|
Flags: needinfo?(itsay)
Whiteboard: [ft:system-platform]
Assignee | ||
Updated•10 years ago
|
Comment 3•10 years ago
|
||
Make sure your tests are passed at least 30 times continually on Travis before you set r?. You could do cherry-pick this patch https://github.com/evanxd/gaia/commit/2bd7bb516b109e1ac19ec0957888841c12ee94e3 to run your tests(not included other tests). Please don't run all tests in Gaia project for 30 times at one pull request. It take time and resource too much. If the patch in Bug 972160 is landed, it will run your tests in a pull request for 30 times automatically instead of using the 2bd7bb516b109e1ac19ec0957888841c12ee94e3 patch. Thanks.
Comment 5•10 years ago
|
||
* Review process for marionette test: Make sure your tests are passed at least 30 times continually on Travis before you set r?. Then leave the URL of 30 times pass Travis page on the review request message on bugzilla page. * How to run test for 30 times: You could do cherry-pick this patch https://github.com/evanxd/gaia/commit/2bd7bb516b109e1ac19ec0957888841c12ee94e3 to run your tests(not included other tests). You could config your test files in `tests/travis_ci/marionette_js_multi/script ` file. * Notice: Please don't run all tests in Gaia project for 30 times at one pull request. It take time and resource too much. If the patch in Bug 972160 is landed, it will run your tests in a pull request for 30 times automatically instead of using the 2bd7bb516b109e1ac19ec0957888841c12ee94e3 patch. Thanks.
Comment 6•10 years ago
|
||
Hi all, the patch for Bug Bug 972160 is landed. New rule for review process of marionette test: Please make sure the `marionette_js_stable_check` action of the patch is passed on Travis (https://travis-ci.org/mozilla-b2g/gaia/jobs/18937817), before you set r? for your patch. Thanks.
Comment 7•10 years ago
|
||
Use this one for the settings app refactoring meta bug for tracking. In v1.4, the scope is to finish the unittest cases before getting into the refactoring. The plan is to ensure the quality on settings app refactoring.
Whiteboard: [ft:system-platform] → [ucid:SystemPlatform30, 1.4, ft:system-platform]
Target Milestone: --- → 1.4 S2 (28feb)
Comment 8•10 years ago
|
||
Assign this user story/meta bug to Fred Lin for tracking on any engineering bug update/dependency.
Assignee: nobody → gasolin
Updated•10 years ago
|
Blocks: 1.4-system-platform
Assignee | ||
Comment 9•10 years ago
|
||
marionette tests done and we'll focus on refactor panels
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•