Skip to content
Snippets Groups Projects
Commit 1903b41f authored by Jean Chalard's avatar Jean Chalard Committed by Android (Google) Code Review
Browse files

Merge "Clean up tests and increase speed"

parents e2af9b7e 001884a1
No related branches found
No related tags found
No related merge requests found
...@@ -51,7 +51,7 @@ import java.util.Set; ...@@ -51,7 +51,7 @@ import java.util.Set;
@LargeTest @LargeTest
public class BinaryDictIOTests extends AndroidTestCase { public class BinaryDictIOTests extends AndroidTestCase {
private static final String TAG = BinaryDictIOTests.class.getSimpleName(); private static final String TAG = BinaryDictIOTests.class.getSimpleName();
private static final int MAX_UNIGRAMS = 1000; private static final int MAX_UNIGRAMS = 100;
private static final int UNIGRAM_FREQ = 10; private static final int UNIGRAM_FREQ = 10;
private static final int BIGRAM_FREQ = 50; private static final int BIGRAM_FREQ = 50;
private static final int TOLERANCE_OF_BIGRAM_FREQ = 5; private static final int TOLERANCE_OF_BIGRAM_FREQ = 5;
...@@ -135,11 +135,12 @@ public class BinaryDictIOTests extends AndroidTestCase { ...@@ -135,11 +135,12 @@ public class BinaryDictIOTests extends AndroidTestCase {
while (count > 0) { while (count > 0) {
final long r = Math.abs(random.nextInt()); final long r = Math.abs(random.nextInt());
if (r < 0) continue; if (r < 0) continue;
// Don't insert 0~20, but insert any other code point. // Don't insert 0~0x20, but insert any other code point.
// Code points are in the range 0~0x10FFFF. // Code points are in the range 0~0x10FFFF.
final int candidateCodePoint = (int)(20 + r % (0x10FFFF - 20)); final int candidateCodePoint = (int)(0x20 + r % (Character.MAX_CODE_POINT - 0x20));
// Code points between 0xD800 and 0xDFFF are not valid. // Code points between MIN_ and MAX_SURROGATE are not valid on their own.
if (candidateCodePoint >= 0xD800 && candidateCodePoint <= 0xDFFF) continue; if (candidateCodePoint >= Character.MIN_SURROGATE
&& candidateCodePoint <= Character.MAX_SURROGATE) continue;
builder.appendCodePoint(candidateCodePoint); builder.appendCodePoint(candidateCodePoint);
--count; --count;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment