Closed Bug 1114568 Opened 6 years ago Closed 6 years ago
Add a test to check that a regular phone number should not be dialed out on the emergency dialer
QA Whiteboard: [fxosqa-auto-s7][fxosqa-auto-points=6]
Here's a first shot. The keypad in EmergencyCall shares the same components as the regular dialer so I created a BaseKeypad class to do the same in our code. Tell me what you guys think.  https://github.com/mozilla-b2g/gaia/blob/master/shared/js/dialer/keypad.js
Comment on attachment 8542550 [details] [review] Gaia PR It look all right to me. The only thing I don't really like is the switch_to_emergency_call_frame() call, but that is also already done in another test (est_lockscreen_unlock_to_emergency_call_screen.py). I don't know if we really have a policy of where we should switch contexts, but I have the feeling that we normally do it from inside the app objects. But I may be wrong.
Attachment #8542550 - Flags: review?(martijn.martijn) → review+
Hrm, never mind, the switch_to_frame() calls are all over the place in test files.
Comment on attachment 8542550 [details] [review] Gaia PR Looks good to me. Waiting for review/feedback from Gabriele before merging.
Attachment #8542550 - Flags: review?(robert.chira) → review+
Comment on attachment 8542550 [details] [review] Gaia PR Looks good to me; I like quite a bit how you managed to split the keypad class to cope both with the regular keypad as well as with the emergency one which is more limited (and I'm still waiting to fully share with the regular keypad).
Attachment #8542550 - Flags: review?(gsvelto) → review+
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.