From 2522a4a477d366a1e69c07c0a5b7c8a0f43118f6 Mon Sep 17 00:00:00 2001 From: Keisuke Kuroyanagi Date: Fri, 3 Oct 2014 19:53:23 +0900 Subject: [PATCH] Fix: NPE in unit test. Change-Id: I769b43f2afa1151d5c0d9c408018c3932be377dc --- .../latin/BinaryDictionaryTests.java | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/tests/src/com/android/inputmethod/latin/BinaryDictionaryTests.java b/tests/src/com/android/inputmethod/latin/BinaryDictionaryTests.java index 9c7792cf2..b15987fd7 100644 --- a/tests/src/com/android/inputmethod/latin/BinaryDictionaryTests.java +++ b/tests/src/com/android/inputmethod/latin/BinaryDictionaryTests.java @@ -1237,15 +1237,17 @@ public class BinaryDictionaryTests extends AndroidTestCase { wordSet.remove(word0); final HashSet bigramWord1s = bigrams.get(word0); // TODO: Support ngram. - for (final WeightedString bigramTarget : wordProperty.getBigrams()) { - final String word1 = bigramTarget.mWord; - assertTrue(bigramWord1s.contains(word1)); - final Pair bigram = new Pair<>(word0, word1); - if (canCheckBigramProbability(formatVersion)) { - final int bigramProbability = bigramProbabilitiesToCheckLater.get(bigram); - assertEquals(bigramProbability, bigramTarget.getProbability()); + if (wordProperty.mHasNgrams) { + for (final WeightedString bigramTarget : wordProperty.getBigrams()) { + final String word1 = bigramTarget.mWord; + assertTrue(bigramWord1s.contains(word1)); + final Pair bigram = new Pair<>(word0, word1); + if (canCheckBigramProbability(formatVersion)) { + final int bigramProbability = bigramProbabilitiesToCheckLater.get(bigram); + assertEquals(bigramProbability, bigramTarget.getProbability()); + } + bigramSet.remove(bigram); } - bigramSet.remove(bigram); } token = result.mNextToken; } while (token != 0);