New test that Import's a contact from SIM, edit it and export it again

RESOLVED FIXED

Status

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Bebe, Assigned: Bebe)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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
QA Whiteboard: [fxosqa-auto-s6]
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 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 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+
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 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+
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.