diff options
| author | Jay Shrauner <shrauner@google.com> | 2013-01-08 11:42:17 -0800 |
|---|---|---|
| committer | Jay Shrauner <shrauner@google.com> | 2013-01-08 14:20:33 -0800 |
| commit | ad29eae127034dbfd4aaf781a09ff08984b72b08 (patch) | |
| tree | 11a039eea89098c85502efe5c239999987e98b56 /tests | |
| parent | 4d3b951661c43d5c21e1d7f4cac593353aa63bf0 (diff) | |
| download | packages_apps_Contacts-ad29eae127034dbfd4aaf781a09ff08984b72b08.tar.gz packages_apps_Contacts-ad29eae127034dbfd4aaf781a09ff08984b72b08.tar.bz2 packages_apps_Contacts-ad29eae127034dbfd4aaf781a09ff08984b72b08.zip | |
Dedup NANPA numbers
Bug:7519057
Change-Id: Ia1b1e2632771f718b6e89f9d67834a7b646c45fa
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/src/com/android/contacts/ContactsUtilsTests.java | 35 |
1 files changed, 29 insertions, 6 deletions
diff --git a/tests/src/com/android/contacts/ContactsUtilsTests.java b/tests/src/com/android/contacts/ContactsUtilsTests.java index 49e9ddd45..b3910c958 100644 --- a/tests/src/com/android/contacts/ContactsUtilsTests.java +++ b/tests/src/com/android/contacts/ContactsUtilsTests.java @@ -135,6 +135,22 @@ public class ContactsUtilsTests extends AndroidTestCase { Phone.CONTENT_ITEM_TYPE, "+49 (89) 12345678", Phone.CONTENT_ITEM_TYPE, "+49 (89) 12345679"); + // test special handling of collapsing country code for NANP region only + // This is non symmetrical, because we prefer the number with the +1. + assertEquals("75", true, + MoreContactUtils.shouldCollapse + (Phone.CONTENT_ITEM_TYPE, "+1 (415) 555-1212", + Phone.CONTENT_ITEM_TYPE, "(415) 555-1212")); + assertEquals("76", false, + MoreContactUtils.shouldCollapse + (Phone.CONTENT_ITEM_TYPE, "(415) 555-1212", + Phone.CONTENT_ITEM_TYPE, "+1 (415) 555-1212")); + // Require explicit +1 country code declaration to collapse + assertEquals("77", false, + MoreContactUtils.shouldCollapse + (Phone.CONTENT_ITEM_TYPE, "1-415-555-1212", + Phone.CONTENT_ITEM_TYPE, "415-555-1212")); + // test some numbers with wait symbol and area code assertCollapses("80", true, Phone.CONTENT_ITEM_TYPE, "+49 (8092) 1234;89321", @@ -165,19 +181,26 @@ public class ContactsUtilsTests extends AndroidTestCase { Phone.CONTENT_ITEM_TYPE, "---", Phone.CONTENT_ITEM_TYPE, "---"); - assertCollapses("90", true, + assertCollapses("90", false, Phone.CONTENT_ITEM_TYPE, "1-/().", Phone.CONTENT_ITEM_TYPE, "--$%1"); - assertCollapses("91", true, - Phone.CONTENT_ITEM_TYPE, "abcdefghijklmnopqrstuvwxyz", - Phone.CONTENT_ITEM_TYPE, "22233344455566677778889999"); + // Test numbers using keypad letters. This is non-symmetrical, because we prefer + // the version with letters. + assertEquals("91", true, + MoreContactUtils.shouldCollapse + (Phone.CONTENT_ITEM_TYPE, "abcdefghijklmnopqrstuvwxyz", + Phone.CONTENT_ITEM_TYPE, "22233344455566677778889999")); + assertEquals("92", false, + MoreContactUtils.shouldCollapse + (Phone.CONTENT_ITEM_TYPE, "22233344455566677778889999", + Phone.CONTENT_ITEM_TYPE, "abcdefghijklmnopqrstuvwxyz")); - assertCollapses("92", false, + assertCollapses("93", false, Phone.CONTENT_ITEM_TYPE, "1;2", Phone.CONTENT_ITEM_TYPE, "12"); - assertCollapses("93", false, + assertCollapses("94", false, Phone.CONTENT_ITEM_TYPE, "1,2", Phone.CONTENT_ITEM_TYPE, "12"); } |