summaryrefslogtreecommitdiff
path: root/android_icu4j/src/main/tests/android/icu/dev/test/bidi
diff options
context:
space:
mode:
authorPete Gillin <peteg@google.com>2017-08-09 14:18:54 +0100
committerPete Gillin <peteg@google.com>2017-09-26 16:41:53 +0100
commit6f1b7aa67c34422bb29fc9f1c06b95234cc197a3 (patch)
tree69c2d22278310d0feb22361f152702a3181260eb /android_icu4j/src/main/tests/android/icu/dev/test/bidi
parent039d984ce64ce860460176794e3163312a2b722f (diff)
downloadicu-6f1b7aa67c34422bb29fc9f1c06b95234cc197a3.tar.gz
Add sharding annotations to android_icu4j test classes.
Some tests were using too much heap, causing excessive garbage collection and subsequent timeouts. This change adds an annotation those specifically, so that they can be run in a separate process, using their own heaps, so avoiding the timeouts. Some of these tests were suppressed in the past. The suppressions were undone when ICU 59 landed, which most likely caused problems on low-memory devices (b/62374714, whose fix was reverted when ICU 59 landed, and perhaps b/37642683). This change (along with the AndroidTest.xml change to actually apply the sharding) fixes those problems. This was tested on fugu on an oc build. Cherry-pick notes: changes under android_icu4j were regenerated by running tools/srcgen/generate_android_icu4j.sh. Bug: 62512577 Test: cts-tradefed run cts -m CtsIcuTestCases --primary-abi-only Change-Id: I013ea26728120bbe3238a7a2dd165606ccad243d (cherry picked from commit 2e13a2bdade5cd0a635f0bd89805931a6fd710da)
Diffstat (limited to 'android_icu4j/src/main/tests/android/icu/dev/test/bidi')
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/BiDiConformanceTest.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/BidiFmwk.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidi.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidiTransform.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCharFromDirProp.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestClassOverride.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCompatibility.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestContext.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestData.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestFailureRecovery.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestInverse.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestMultipleParagraphs.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorder.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderArabicMathSymbols.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderRunsOnly.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderingMode.java2
-rw-r--r--android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestStreaming.java2
17 files changed, 34 insertions, 0 deletions
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/BiDiConformanceTest.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/BiDiConformanceTest.java
index f1952f271..2d6261e90 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/BiDiConformanceTest.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/BiDiConformanceTest.java
@@ -19,12 +19,14 @@ import android.icu.dev.test.TestUtil;
import android.icu.lang.UCharacterDirection;
import android.icu.text.Bidi;
import android.icu.text.BidiClassifier;
+import android.icu.testsharding.MainTestShard;
/**
* @author Markus W. Scherer
* BiDi conformance test, using the Unicode BidiTest.txt and BidiCharacterTest.txt files.
* Ported from ICU4C intltest/bidiconf.cpp .
*/
+@MainTestShard
public class BiDiConformanceTest extends TestFmwk {
public BiDiConformanceTest() {}
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/BidiFmwk.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/BidiFmwk.java
index b74a1bf92..9e87c13a3 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/BidiFmwk.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/BidiFmwk.java
@@ -18,6 +18,7 @@ import android.icu.lang.UCharacter;
import android.icu.text.Bidi;
import android.icu.text.BidiRun;
import android.icu.util.VersionInfo;
+import android.icu.testsharding.MainTestShard;
/**
* A base class for the Bidi test suite.
@@ -25,6 +26,7 @@ import android.icu.util.VersionInfo;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class BidiFmwk extends TestFmwk {
protected static final char[] charFromDirProp = {
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidi.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidi.java
index 55b9f9e5f..88dee912a 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidi.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidi.java
@@ -16,6 +16,7 @@ import org.junit.Test;
import android.icu.text.Bidi;
import android.icu.text.BidiRun;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for Bidi class override.
@@ -23,6 +24,7 @@ import android.icu.text.BidiRun;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestBidi extends BidiFmwk {
private static final int MAXLEN = 256;
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidiTransform.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidiTransform.java
index a2aa1a3bd..e8fe79b5f 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidiTransform.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestBidiTransform.java
@@ -13,6 +13,7 @@ import android.icu.text.Bidi;
import android.icu.text.BidiTransform;
import android.icu.text.BidiTransform.Mirroring;
import android.icu.text.BidiTransform.Order;
+import android.icu.testsharding.MainTestShard;
/**
* Verify Bidi Layout Transformations
@@ -20,6 +21,7 @@ import android.icu.text.BidiTransform.Order;
* @author Lina Kemmel
*
*/
+@MainTestShard
public class TestBidiTransform extends TestFmwk {
static final char LATN_ZERO = '\u0030';
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCharFromDirProp.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCharFromDirProp.java
index 9ea5b8e64..78aeba2e7 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCharFromDirProp.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCharFromDirProp.java
@@ -15,6 +15,7 @@ import org.junit.Test;
import android.icu.impl.Utility;
import android.icu.lang.UCharacter;
import android.icu.lang.UCharacterDirection;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for Bidi charFromDirProp
@@ -22,6 +23,7 @@ import android.icu.lang.UCharacterDirection;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestCharFromDirProp extends BidiFmwk {
/* verify that the exemplar characters have the expected bidi classes */
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestClassOverride.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestClassOverride.java
index fa248c339..890e32b6b 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestClassOverride.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestClassOverride.java
@@ -14,6 +14,7 @@ import org.junit.Test;
import android.icu.text.Bidi;
import android.icu.text.BidiClassifier;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for Bidi class override.
@@ -21,6 +22,7 @@ import android.icu.text.BidiClassifier;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestClassOverride extends BidiFmwk {
private static final int DEF = TestData.DEF;
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCompatibility.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCompatibility.java
index 0d5090a09..701fac869 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCompatibility.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestCompatibility.java
@@ -18,6 +18,7 @@ import java.text.AttributedString;
import org.junit.Test;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for java.text.Bidi compatibility
@@ -25,6 +26,7 @@ import android.icu.text.Bidi;
* @author Matitiahu Allouche
*/
+@MainTestShard
public class TestCompatibility extends BidiFmwk {
void compareBidi(Bidi bidi, java.text.Bidi jbidi)
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestContext.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestContext.java
index cd0d61dda..ceb438c57 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestContext.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestContext.java
@@ -13,6 +13,7 @@ package android.icu.dev.test.bidi;
import org.junit.Test;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for doing transformations in context
@@ -20,6 +21,7 @@ import android.icu.text.Bidi;
* @author Matitiahu Allouche
*/
+@MainTestShard
public class TestContext extends BidiFmwk {
private class ContextCase {
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestData.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestData.java
index 00fa42999..c444dc784 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestData.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestData.java
@@ -12,6 +12,7 @@ package android.icu.dev.test.bidi;
import android.icu.lang.UCharacterDirection;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
@@ -20,6 +21,7 @@ import android.icu.text.Bidi;
* Ported from C by Lina Kemmel, Matitiahu Allouche
*
*/
+@MainTestShard
public class TestData {
protected static final int L = UCharacterDirection.LEFT_TO_RIGHT;
protected static final int R = UCharacterDirection.RIGHT_TO_LEFT;
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestFailureRecovery.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestFailureRecovery.java
index df1adb2fe..218035823 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestFailureRecovery.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestFailureRecovery.java
@@ -13,6 +13,7 @@ package android.icu.dev.test.bidi;
import org.junit.Test;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for Bidi failure recovery
@@ -20,6 +21,7 @@ import android.icu.text.Bidi;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestFailureRecovery extends BidiFmwk {
@Test
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestInverse.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestInverse.java
index b8a6fa10a..a872deb5b 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestInverse.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestInverse.java
@@ -16,6 +16,7 @@ import org.junit.Test;
import android.icu.impl.Utility;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for the basic "inverse" Bidi mode.
@@ -23,6 +24,7 @@ import android.icu.text.Bidi;
* ported from C by Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestInverse extends BidiFmwk {
private int countRoundtrips = 0;
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestMultipleParagraphs.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestMultipleParagraphs.java
index b1c9142f3..12d555f8e 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestMultipleParagraphs.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestMultipleParagraphs.java
@@ -17,6 +17,7 @@ import org.junit.Test;
import android.icu.impl.Utility;
import android.icu.text.Bidi;
import android.icu.text.BidiRun;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for Bidi multiple paragraphs
@@ -24,6 +25,7 @@ import android.icu.text.BidiRun;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestMultipleParagraphs extends BidiFmwk {
private static final String text =
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorder.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorder.java
index d0a6f6689..c7d04a841 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorder.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorder.java
@@ -13,6 +13,7 @@ package android.icu.dev.test.bidi;
import org.junit.Test;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
@@ -21,6 +22,7 @@ import android.icu.text.Bidi;
* ported from C by Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestReorder extends BidiFmwk {
private static final String[] logicalOrder = {
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderArabicMathSymbols.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderArabicMathSymbols.java
index e14c80041..622839f09 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderArabicMathSymbols.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderArabicMathSymbols.java
@@ -12,6 +12,7 @@ package android.icu.dev.test.bidi;
import org.junit.Test;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
@@ -20,6 +21,7 @@ import android.icu.text.Bidi;
* Ported from C by Ramy Said
*/
+@MainTestShard
public class TestReorderArabicMathSymbols extends BidiFmwk {
private static final String[] logicalOrder = {
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderRunsOnly.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderRunsOnly.java
index 1b1450eae..cc774af71 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderRunsOnly.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderRunsOnly.java
@@ -13,6 +13,7 @@ package android.icu.dev.test.bidi;
import org.junit.Test;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for variants to the UBA.
@@ -20,6 +21,7 @@ import android.icu.text.Bidi;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestReorderRunsOnly extends BidiFmwk {
static class TestCase {
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderingMode.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderingMode.java
index 954594657..d70c9c4a7 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderingMode.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestReorderingMode.java
@@ -15,6 +15,7 @@ import java.util.Arrays;
import org.junit.Test;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for variants to the UBA.
@@ -22,6 +23,7 @@ import android.icu.text.Bidi;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestReorderingMode extends BidiFmwk {
static final String[] textIn = {
diff --git a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestStreaming.java b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestStreaming.java
index 18dc2c991..cf7bb254d 100644
--- a/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestStreaming.java
+++ b/android_icu4j/src/main/tests/android/icu/dev/test/bidi/TestStreaming.java
@@ -14,6 +14,7 @@ import org.junit.Test;
import android.icu.impl.Utility;
import android.icu.text.Bidi;
+import android.icu.testsharding.MainTestShard;
/**
* Regression test for streaming mode
@@ -21,6 +22,7 @@ import android.icu.text.Bidi;
* @author Lina Kemmel, Matitiahu Allouche
*/
+@MainTestShard
public class TestStreaming extends BidiFmwk {
static final int MAXPORTIONS = 10;