diff options
author | 2014-09-30 12:49:25 +0900 | |
---|---|---|
committer | 2014-09-30 12:49:25 +0900 | |
commit | 0c2cdcc7350e8dc0e97f0aa4f343984f3cf39f72 (patch) | |
tree | 07effa27dd475d277993ec332581f57ef721d6d5 | |
parent | a69c170a3f934dbeec9e1ea5e25c7b737b299c24 (diff) | |
download | latinime-0c2cdcc7350e8dc0e97f0aa4f343984f3cf39f72.tar.gz latinime-0c2cdcc7350e8dc0e97f0aa4f343984f3cf39f72.tar.xz latinime-0c2cdcc7350e8dc0e97f0aa4f343984f3cf39f72.zip |
Fix bug of testCodePointTable.
Bug:17097992
Change-Id: Ia26bfceb008f52e2c3b760159c5129b7eca5d6d6
-rw-r--r-- | tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoder.java | 5 | ||||
-rw-r--r-- | tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoderTests.java | 10 |
2 files changed, 9 insertions, 6 deletions
diff --git a/tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoder.java b/tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoder.java index 012fd811c..c47190190 100644 --- a/tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoder.java +++ b/tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoder.java @@ -100,7 +100,10 @@ public class Ver2DictEncoder implements DictEncoder { Collections.sort(codePointOccurrenceArray, new Comparator<Entry<Integer, Integer>>() { @Override public int compare(final Entry<Integer, Integer> a, final Entry<Integer, Integer> b) { - return b.getValue().compareTo(a.getValue()); + if (a.getValue() != b.getValue()) { + return b.getValue().compareTo(a.getValue()); + } + return b.getKey().compareTo(a.getKey()); } }); int currentCodePointTableIndex = FormatSpec.MINIMAL_ONE_BYTE_CHARACTER_VALUE; diff --git a/tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoderTests.java b/tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoderTests.java index 7c0b92dc3..9104c2fcb 100644 --- a/tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoderTests.java +++ b/tests/src/com/android/inputmethod/latin/makedict/Ver2DictEncoderTests.java @@ -42,8 +42,8 @@ public class Ver2DictEncoderTests extends AndroidTestCase { final List<String> words = Arrays.asList(wordSource); final String correctCodePointTable = "eotdsanirfg bclwup"; final String correctCodePointOccurrenceArrayString = - "10141164111411531003110297210521142103111911171108198199132111211021"; - final String correctCodePointExpectedMapString = "323433363538373940494147454644424348"; + "11641114101411531003114211021052972119111711121108110311021991981321"; + final String correctCodePointExpectedMapString = "343332363540383937464549484744414243"; final String dictName = "codePointTableTest"; final String dictVersion = Long.toString(System.currentTimeMillis()); @@ -60,8 +60,8 @@ public class Ver2DictEncoderTests extends AndroidTestCase { codePointOccurrenceArrayString.append(entry.getKey()); codePointOccurrenceArrayString.append(entry.getValue()); } - assertEquals(codePointOccurrenceArrayString.toString(), - correctCodePointOccurrenceArrayString); + assertEquals(correctCodePointOccurrenceArrayString, + codePointOccurrenceArrayString.toString()); // Check if mCodePointToOneByteCodeMap is correct final StringBuilder codePointExpectedMapString = new StringBuilder(); @@ -69,7 +69,7 @@ public class Ver2DictEncoderTests extends AndroidTestCase { codePointExpectedMapString.append(codePointTable.mCodePointToOneByteCodeMap.get( correctCodePointTable.codePointAt(i))); } - assertEquals(codePointExpectedMapString.toString(), correctCodePointExpectedMapString); + assertEquals(correctCodePointExpectedMapString, codePointExpectedMapString.toString()); } /** |