diff options
author | 2012-07-06 12:34:41 +0900 | |
---|---|---|
committer | 2012-07-09 15:06:16 +0900 | |
commit | d579f1aefc8d02254db297ffd6d8f9dbdcab0637 (patch) | |
tree | 5155aa54e99b83d9fae09e74a03b678576b3316a /tests/src | |
parent | 347a80f793c2b3e132b48d87918e331352b142f4 (diff) | |
download | latinime-d579f1aefc8d02254db297ffd6d8f9dbdcab0637.tar.gz latinime-d579f1aefc8d02254db297ffd6d8f9dbdcab0637.tar.xz latinime-d579f1aefc8d02254db297ffd6d8f9dbdcab0637.zip |
Consolidate methods that do the same thing (A74)
Change-Id: I6e7c3aba2b40ecefa483c9680978bc806368e96e
Diffstat (limited to 'tests/src')
-rw-r--r-- | tests/src/com/android/inputmethod/latin/RichInputConnectionTests.java | 31 |
1 files changed, 13 insertions, 18 deletions
diff --git a/tests/src/com/android/inputmethod/latin/RichInputConnectionTests.java b/tests/src/com/android/inputmethod/latin/RichInputConnectionTests.java index 7bd7b0e5a..ad9937940 100644 --- a/tests/src/com/android/inputmethod/latin/RichInputConnectionTests.java +++ b/tests/src/com/android/inputmethod/latin/RichInputConnectionTests.java @@ -102,31 +102,26 @@ public class RichInputConnectionTests extends AndroidTestCase { */ public void testGetPreviousWord() { // If one of the following cases breaks, the bigram suggestions won't work. - assertEquals(RichInputConnection.getPreviousWord("abc def", sSeparators), "abc"); - assertNull(RichInputConnection.getPreviousWord("abc", sSeparators)); - assertNull(RichInputConnection.getPreviousWord("abc. def", sSeparators)); + assertEquals(RichInputConnection.getNthPreviousWord("abc def", sSeparators, 2), "abc"); + assertNull(RichInputConnection.getNthPreviousWord("abc", sSeparators, 2)); + assertNull(RichInputConnection.getNthPreviousWord("abc. def", sSeparators, 2)); // The following tests reflect the current behavior of the function - // RichInputConnection#getPreviousWord. + // RichInputConnection#getNthPreviousWord. // TODO: However at this time, the code does never go // into such a path, so it should be safe to change the behavior of // this function if needed - especially since it does not seem very // logical. These tests are just there to catch any unintentional // changes in the behavior of the RichInputConnection#getPreviousWord method. - assertEquals(RichInputConnection.getPreviousWord("abc def ", sSeparators), "abc"); - assertEquals(RichInputConnection.getPreviousWord("abc def.", sSeparators), "abc"); - assertEquals(RichInputConnection.getPreviousWord("abc def .", sSeparators), "def"); - assertNull(RichInputConnection.getPreviousWord("abc ", sSeparators)); - } - - /** - * Test for getting the word before the cursor (for bigram) - */ - public void testGetThisWord() { - assertEquals(RichInputConnection.getThisWord("abc def", sSeparators), "def"); - assertEquals(RichInputConnection.getThisWord("abc def ", sSeparators), "def"); - assertNull(RichInputConnection.getThisWord("abc def.", sSeparators)); - assertNull(RichInputConnection.getThisWord("abc def .", sSeparators)); + assertEquals(RichInputConnection.getNthPreviousWord("abc def ", sSeparators, 2), "abc"); + assertEquals(RichInputConnection.getNthPreviousWord("abc def.", sSeparators, 2), "abc"); + assertEquals(RichInputConnection.getNthPreviousWord("abc def .", sSeparators, 2), "def"); + assertNull(RichInputConnection.getNthPreviousWord("abc ", sSeparators, 2)); + + assertEquals(RichInputConnection.getNthPreviousWord("abc def", sSeparators, 1), "def"); + assertEquals(RichInputConnection.getNthPreviousWord("abc def ", sSeparators, 1), "def"); + assertNull(RichInputConnection.getNthPreviousWord("abc def.", sSeparators, 1)); + assertNull(RichInputConnection.getNthPreviousWord("abc def .", sSeparators, 1)); } /** |