Closed Bug 968692 Opened 7 years ago Closed 7 years ago

[settings] add marionette test for Screen lock panel

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
1.4 S2 (28feb)

People

(Reporter: gasolin, Assigned: eragonj)

References

Details

(Whiteboard: [ft:system-platform])

Attachments

(1 file)

Overview Description:

Add marionette test for Screen lock 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 Date & Time panel


Expected Results:

Write that integration test and make test pass

Reproducibility:

100%

Additional Information:

Please refer to bug 956210 for more information.

There are some hidden field need SIM card inserted
Blocks: 969265
No longer blocks: 956210
Assignee: nobody → ejchen
Target Milestone: --- → 1.4 S2 (28feb)
Comment on attachment 8373135 [details] [review]
patch on master

Hi Arthur & Evan,

I just made 3 tests for screenLock. I think these 3 tests are required for screenLock and please help me review & feedback for this !

BTW, Related tests on Travis just got passed, oh yeah !
(https://travis-ci.org/mozilla-b2g/gaia/jobs/18561449#L2187)

Thanks :P
Attachment #8373135 - Flags: review?(arthur.chen)
Attachment #8373135 - Flags: feedback?(evanxd)
Attachment #8373135 - Flags: feedback?(evanxd) → review?(evanxd)
Test case spec for the Screen lock panel:

Test case: Change the "Lock screen" item.
>> Make sure the mozSettings value is correct.

Test case: Add a new password.
>> Make sure the mozSettings value(password) is correct.

Test case: Add a new password and change the "Require passcode".
>> Make sure the mozSettings value(password) is correct.

Test case: Add a new password and change the password.
>> Make sure the mozSettings value(password) is correct.

If have any question, please discuss here.
Thanks. :)
Comment on attachment 8373135 [details] [review]
patch on master

Per Evan's comment, we need to add the following tests:

- Check require passcode settings
- Check whether the stored passcode is the same as the input one

And I would suggest to add another test, which is when the passcode lock is enabled, a passcode prompt will be pop when users try to disable the lock screen.
Attachment #8373135 - Flags: review?(arthur.chen)
Comment on attachment 8373135 [details] [review]
patch on master

Hi Arthur & Evan,

just passed 30 tests on Evan's repo on Github ! Please help me review again when you have time ! Thanks :)
Attachment #8373135 - Flags: review?(arthur.chen)
Blocks: 973440
Comment on attachment 8373135 [details] [review]
patch on master

Hi EJ(小龍哥),

Sorry for the late reply.

There is a 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), then you could set r? for your patch.

Thanks.
Attachment #8373135 - Flags: review?(evanxd)
Comment on attachment 8373135 [details] [review]
patch on master

Hi Evan, 

this just passed tests on Travis (https://travis-ci.org/mozilla-b2g/gaia/builds/19014025).

Please help me review this when you have time ! Thanks :)
Attachment #8373135 - Flags: review?(evanxd)
Whiteboard: [ft:system-platform]
Comment on attachment 8373135 [details] [review]
patch on master

Thanks, EJ. For the lock screen test, we need to ensure the checkbox is still enabled when users type the incorrect passcode. This did failed once, so I would suggest to include this test.
Attachment #8373135 - Flags: review?(arthur.chen)
(In reply to Arthur Chen [:arthurcc] from comment #10)
> Comment on attachment 8373135 [details] [review]
> patch on master
> 
> Thanks, EJ. For the lock screen test, we need to ensure the checkbox is
> still enabled when users type the incorrect passcode. This did failed once,
> so I would suggest to include this test.

I added one more commit to make sure when testing isXXXEnabled(), I will also check isXXXCheced().

Is this ok for you, Arthur ?
Comment on attachment 8373135 [details] [review]
patch on master

r=me, thanks!
Attachment #8373135 - Flags: review+
Thanks all, just merged at gaia/master : 6003c319780f49871dbe91ce5af32f5fc8ec06d9
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment on attachment 8373135 [details] [review]
patch on master

Thanks for Arthur's review.
Attachment #8373135 - Flags: review?(evanxd)
Depends on: 983171
You need to log in before you can comment on or make changes to this bug.