summaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorYorke Lee <yorkelee@google.com>2012-11-07 15:11:44 -0800
committerYorke Lee <yorkelee@google.com>2012-11-07 15:50:04 -0800
commit1be0178a11b1cc3b06867b14446e1e041e97a82c (patch)
tree90d972b1c98ce31fee597ba57461741266c0149f /tests
parent90a85b2201a3e1132124827714ee34e00091036f (diff)
Add smart dialling capabilities to dialer
Layout changes to dialpad fragment to make space for smart dial suggestions. This feature does not appear in landscape mode. SmartDialTextView automatically resizes text to fit within the bounds of the view. SmartDialAdapter highlights certain portions of the matching name based on match positions provided by SmartDialNameMatcher. SmartDialLoaderTask is an AsyncTask that caches all contacts with a phone number into memory, or matches all contact names against a provided query to return a list of matches (maximum of 3). SmartDialNameMatcher contains utility functions to remove accents from accented characters and normalize a phone number. It also contains the matching logic that determines if a contact's display name matches a numeric query. Added some tests for SmartDialNameMatcher Bug: 6977981 Change-Id: I43e1a70d8d0d46e02fc67fe1caaec9a1769124f5
Diffstat (limited to 'tests')
-rw-r--r--tests/src/com/android/dialer/SmartDialNameMatcherTest.java124
1 files changed, 124 insertions, 0 deletions
diff --git a/tests/src/com/android/dialer/SmartDialNameMatcherTest.java b/tests/src/com/android/dialer/SmartDialNameMatcherTest.java
new file mode 100644
index 000000000..babae55fa
--- /dev/null
+++ b/tests/src/com/android/dialer/SmartDialNameMatcherTest.java
@@ -0,0 +1,124 @@
+/*
+ * Copyright (C) 2012 The Android Open Source Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.android.dialer.dialpad;
+
+import android.test.suitebuilder.annotation.SmallTest;
+import android.test.suitebuilder.annotation.Suppress;
+import android.util.Log;
+
+import com.android.dialer.dialpad.SmartDialNameMatcher;
+
+import java.text.Normalizer;
+import java.util.ArrayList;
+
+import junit.framework.TestCase;
+
+@SmallTest
+public class SmartDialNameMatcherTest extends TestCase {
+ private static final String TAG = "SmartDialNameMatcherTest";
+
+ public void testMatches() {
+ // Test to ensure that all alphabetic characters are covered
+ checkMatches("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz",
+ "22233344455566677778889999" + "22233344455566677778889999", true, 0, 26 * 2);
+ // Should fail because of a mistyped 2 instead of 9 in the second last character
+ checkMatches("ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz",
+ "22233344455566677778889999" + "22233344455566677778889929", false, 0, 0);
+
+ // Basic name test
+ checkMatches("joe", "5", true, 0, 1);
+ checkMatches("joe", "56", true, 0, 2);
+ checkMatches("joe", "563", true, 0, 3);
+
+ // Matches only word boundary.
+ checkMatches("joe", "63", false, 0, 0);
+ checkMatches("joe oe", "63", true, 4, 6);
+
+ // Test for a match across word boundaries
+ checkMatches("joe oe", "56363", true, 0, 6);
+ }
+
+ public void testMatches_repeatedLetters() {
+ checkMatches("aaaaaaaaaa", "2222222222", true, 0, 10);
+ // Fails because of one extra 2
+ checkMatches("aaaaaaaaaa", "22222222222", false, 0, 0);
+ checkMatches("zzzzzzzzzz zzzzzzzzzz", "99999999999999999999", true, 0, 21);
+ }
+
+ public void testMatches_repeatedSpaces() {
+ checkMatches("William J Smith", "9455426576", true, 0, 17);
+ checkMatches("William J Smith", "576", true, 12, 17);
+ // Fails because we start at non-word boundary
+ checkMatches("William J Smith", "6576", false, 0, 0);
+ }
+
+ // TODO: Do we want to make these pass anymore?
+ @Suppress
+ public void testMatches_repeatedSeparators() {
+ // Simple match for single token
+ checkMatches("John,,,,,Doe", "5646", true, 0, 4);
+ // Match across tokens
+ checkMatches("John,,,,,Doe", "56463", true, 0, 10);
+ // Match token after chain of separators
+ checkMatches("John,,,,,Doe", "363", true, 9, 12);
+ }
+
+ public void testMatches_umlaut() {
+ checkMatches("ÄÖÜäöü", "268268", true, 0, 6);
+ }
+ // TODO: Great if it was treated as "s" or "ss. Figure out if possible without prefix trie?
+ @Suppress
+ public void testMatches_germanSharpS() {
+ checkMatches("ß", "s", true, 0, 1);
+ checkMatches("ß", "ss", true, 0, 1);
+ }
+
+ // TODO: Add this and make it work
+ @Suppress
+ public void testMatches_greek() {
+ // http://en.wikipedia.org/wiki/Greek_alphabet
+ fail("Greek letters aren't supported yet.");
+ }
+
+ // TODO: Add this and make it work
+ @Suppress
+ public void testMatches_cyrillic() {
+ // http://en.wikipedia.org/wiki/Cyrillic_script
+ fail("Cyrillic letters aren't supported yet.");
+ }
+
+ private void checkMatches(String displayName, String query, boolean expectedMatches,
+ int expectedMatchStart, int expectedMatchEnd) {
+ final SmartDialNameMatcher matcher = new SmartDialNameMatcher(query);
+ final ArrayList<SmartDialMatchPosition> matchPositions =
+ new ArrayList<SmartDialMatchPosition>();
+ displayName = SmartDialNameMatcher.stripDiacritics(displayName);
+ final boolean matches = matcher.matchesCombination(
+ displayName, query, matchPositions);
+ Log.d(TAG, "query=" + query + " text=" + displayName
+ + " nfd=" + Normalizer.normalize(displayName, Normalizer.Form.NFD)
+ + " nfc=" + Normalizer.normalize(displayName, Normalizer.Form.NFC)
+ + " nfkd=" + Normalizer.normalize(displayName, Normalizer.Form.NFKD)
+ + " nfkc=" + Normalizer.normalize(displayName, Normalizer.Form.NFKC)
+ + " matches=" + matches);
+ assertEquals("matches", expectedMatches, matches);
+ if (matches) {
+ assertEquals("start", expectedMatchStart, matchPositions.get(0).start);
+ assertEquals("end", expectedMatchEnd, matchPositions.get(0).end);
+ }
+ }
+}