956210 - [Settings][meta] marionette tests and panel refactor for 2.0

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Settings, defect)

Description

[[e-mail]]

We need have more marionette UI test to help make sure the rewrite of settings not break any obvious things.

The plan might be

1. write Settings 2 that can pass current integration test
2. write more Settings integration test
3. Make settings 2 pass more integration test

Comment 1

[Kevin Grandon :kgrandon]

Is there a bug/info for the rewrite? We may also want to enhance our desktop shims so we can test additional functionality. Adding mobile/bluetooth tests for example should be possible if we enhance the shims where needed.

https://github.com/mozilla-b2g/gaia/tree/master/tools/extensions/desktop-helper/content/data/lib

Comment 2

[[e-mail]]

Arthur has a early prototype for the rewrite based on your discussion (with kaze as well).
The detail is not confirmed yet but more integration test takes no harm :)

https://github.com/crh0716/gaia/tree/settings2

Comment 3

[[e-mail]]

There's 14 uitests in gaia/tests/python/gaia-ui-tests/gaiatest/tests/functional/settings already covered bluetooth, wifi, display (wallpaper), do not track...,

so please check if js integration test cover the python version as well.

Comment 4

[[e-mail]]

We're taking new approach which is backward compatible to current panels 
https://github.com/crh0716/gaia/tree/settings2_iterative

Comment 5

[[e-mail]]

We'll remove bugs that categorized in after 1.4 bug 968694 and for 1.4 bug 969265

Comment 6

[[e-mail]]

typo 968694, should be bug 969264

Comment 7

[[e-mail]]

We've some tech discussion about 1.5 Goal for settings. We decide to start from **improving settings app load time**.  Will start investigate how to separate this goal to workable tasks.

For the record, in datazilla current settings app load time is 1728.7ms
https://datazilla.mozilla.org/?start=1396340543&stop=1396945810&product=B2G&repository=master&test=settings&page=cold_load_time&project=b2g

Comment 8

[[e-mail]]

Evan, could you help addressing marionette js test required fields under bug 956210?

Comment 9

[Evan Tseng [:evanxd]]

Hi Fred,

I will check on that in next sprint. :)

Comment 10

[Evan Tseng [:evanxd]]

Hi Fred,

For the test case planning things, based on the experience in v1.4, I think we could let reviewer and developers to figure out this for each panel. It might be easier for us.

Comment 11

[[e-mail]]

thanks Evan, that's do it :)

Comment 12

[[e-mail]]

propagate rest works to bug 969264 (refactor and test works for 2.1)