Closed
Bug 1109587
Opened 10 years ago
Closed 10 years ago
New test that Import's a contact from SIM, edit it and export it again
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Bebe, Assigned: Bebe)
Details
Attachments
(1 file)
https://moztrap.mozilla.org/manage/case/14115/
There should be at least one contact in the SIM card
Import a contact with Name and Phone number from the SIM card
Once in the contact list edit it adding Last Name
When it is saved, export it to SIM card
There should not be created a new entry for the contact in the SIM card. If we remove the contact from the device and import from SIM, only one contact should appear with Name, Last Name and Phone Number
Assignee | ||
Updated•10 years ago
|
QA Whiteboard: [fxosqa-auto-s6]
Assignee | ||
Comment 1•10 years ago
|
||
Dave can you take a look over the changes in the atom?
Attachment #8536555 -
Flags: review?(viorela.ioia)
Attachment #8536555 -
Flags: review?(jlorenzo)
Attachment #8536555 -
Flags: feedback?(dave.hunt)
Comment 2•10 years ago
|
||
Comment on attachment 8536555 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/26784
This PR looks great. I have one issue with my SIM card though. I can't store more than 14 characters for a contact's name. So "gaiaXXXXXX test" becomes "gaiaXXXXXX tes".
I don't know if it's also the case on AT&T/T-Mobile's SIM cards, but I think it's better if we shorten the name of the contact.
Attachment #8536555 -
Flags: review?(jlorenzo) → review-
Comment 3•10 years ago
|
||
Comment on attachment 8536555 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/26784
The atom changes look fine.
Attachment #8536555 -
Flags: feedback?(dave.hunt) → feedback+
Assignee | ||
Comment 4•10 years ago
|
||
Comment on attachment 8536555 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/26784
Updated
Attachment #8536555 -
Flags: review- → review?(jlorenzo)
Comment 5•10 years ago
|
||
Comment on attachment 8536555 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/26784
Looks and runs fine to me. Once it's excluded from the desktop run, it'll be ready to be merged.
Attachment #8536555 -
Flags: review?(jlorenzo) → review+
Comment 6•10 years ago
|
||
Comment on attachment 8536555 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/26784
Merged in: https://github.com/mozilla-b2g/gaia/commit/ea6e668ecad6018f76d3628c4a13aa829dfd09df
Attachment #8536555 -
Flags: review?(viorela.ioia) → review+
Updated•10 years ago
|
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
•