Evan Tseng [:evanxd] Reporter
	Description • 10 years ago

      No description provided.

	Evan Tseng [:evanxd] Reporter
	Comment 1 • 10 years ago

We should just disable the lockscreen and ftu by default. Because marionette tests of each app should just take care of itself.

	Evan Tseng [:evanxd] Reporter
	Comment 3 • 10 years ago

And it also saves time of running tests.

	Evan Tseng [:evanxd] Reporter
	Comment 4 • 10 years ago

How do you guys think about this?

	Evan Tseng [:evanxd] Reporter
	Comment 5 • 10 years ago

Many tests just doesn't need lockscreen and ftu to test their apps. So we could just disable lockscreen and ftu by default to instead of disabling them manually in each test. For example, the below tests just disable them manually in gaia code base.

1. https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/dialer/test/marionette/lib/dialer.js#L17
2. https://github.com/mozilla-b2g/gaia/blob/master/apps/communications/dialer/test/performance/call_log_rendering_test.js#L15
3. https://github.com/mozilla-b2g/gaia/blob/master/apps/findmydevice/test/marionette/findmydevice_test.js#L13
4. https://github.com/mozilla-b2g/gaia/blob/master/apps/homescreen/test/marionette/active_icons_test.js#L9
5. https://github.com/mozilla-b2g/gaia/blob/master/apps/music/test/marionette/Player_test.js:#L24
6. https://github.com/mozilla-b2g/gaia/blob/master/apps/settings/test/performance/overfill_ratio_test.js#L13
7. https://github.com/mozilla-b2g/gaia/blob/master/apps/settings/test/performance/rendering_wifi_list_test.js#L16
8. https://github.com/mozilla-b2g/gaia/blob/master/apps/sms/test/marionette/phone_number_service_test.js#L10
9. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/activity_callee_alert_test.js#L12
10. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/app_close_test.js#L9
11. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/camera_on_call_test.js#L13
12. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/desktop_notification_test.js#L8
13. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/edges_gesture_test.js#L21
14. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/fullscreen_app_alert_test.js#L13
15. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/fullscreen_statusbar_test.js#L15
16. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/homescreen_navigation_test.js#L18
17. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/install_ime_test.js#L11
18. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/lockscreen_disabled_test.js#L18
19. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/media_playback_test.js#L17
20. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/net_error_test.js#L7
21. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/notification_events_test.js#L14
22. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/notification_get_test.js#L12
23. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/notification_launch_app.js#L11
24. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/notification_resend_test.js#L14
25. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/sim_dialog_test.js#L16
26. https://github.com/mozilla-b2g/gaia/blob/master/apps/system/test/marionette/lib/rocketbar.js#L28
27. https://github.com/mozilla-b2g/gaia/blob/master/dev_apps/home2/test/marionette/lib/home2.js#L19
28. https://github.com/mozilla-b2g/gaia/blob/master/tests/performance/startup_events_test.js#L41
29. https://github.com/mozilla-b2g/gaia/blob/master/tests/performance/startup_test.js#L20

	Evan Tseng [:evanxd] Reporter
	Comment 6 • 10 years ago

And we also need to fix some tests[1], if we decide to disable ftu and lockscreen by default. We could just enable the ftu or lockscreen manually in the tests to fix the them.

[1] https://travis-ci.org/mozilla-b2g/gaia/jobs/26366598#L2981

	Kevin Grandon :kgrandon
	Comment 7 • 10 years ago

Sounds good to me! These tests have started timing out a lot on travis (50+ mins), so saving some time would be nice.

	Evan Tseng [:evanxd] Reporter
	Comment 8 • 10 years ago

Hi Kevin,
So let's do this.
Thanks.

	James Lal [:lightsofapollo]
	Comment 9 • 10 years ago

wfm too

	Firefox Bug Husbandry Bot
	Comment 10 • 7 years ago

Bulk closed as part of https://bugzilla.mozilla.org/show_bug.cgi?id=1422750