]> granicus.if.org Git - icu/commitdiff
ICU-13372 Ignore TestDataDrivenJDK instead of Pass when using a different JDK version.
authorShane Carr <shane@unicode.org>
Thu, 5 Oct 2017 04:27:34 +0000 (04:27 +0000)
committerShane Carr <shane@unicode.org>
Thu, 5 Oct 2017 04:27:34 +0000 (04:27 +0000)
X-SVN-Rev: 40564

icu4j/main/tests/core/src/com/ibm/icu/dev/test/format/NumberFormatDataDrivenTest.java

index 7fcafc290c9e53d54fa11699286da1e4783749d6..f0312bef172666d47ea4e4d447416db8e07d4075 100644 (file)
@@ -752,15 +752,17 @@ public class NumberFormatDataDrivenTest {
   @Test
   public void TestDataDrivenJDK() {
     // Android implements java.text.DecimalFormat with ICU4J (ticket #13322).
-    if (TestUtil.getJavaVendor() == TestUtil.JavaVendor.Android) return;
+    // Oracle/OpenJDK 9's behavior is not exactly same with Oracle/OpenJDK 8.
+    // Some test cases failed on 8 work well, while some other test cases
+    // fail on 9, but worked on 8. Skip this test case if Java version is not 8.
+    org.junit.Assume.assumeTrue(
+            TestUtil.getJavaVendor() != TestUtil.JavaVendor.Android
+            && TestUtil.getJavaVersion() < 9);
 
     if (TestUtil.getJavaVersion() == 8) {
-        DataDrivenNumberFormatTestUtility.runFormatSuiteIncludingKnownFailures(
-                "numberformattestspecification.txt", JDK);
+      DataDrivenNumberFormatTestUtility.runFormatSuiteIncludingKnownFailures(
+              "numberformattestspecification.txt", JDK);
     } else {
-        // Oracle/OpenJDK 9's behavior is not exactly same with Oracle/OpenJDK 8.
-        // Some test cases failed on 8 work well, while some other test cases
-        // fail on 9, but worked on 8. Skip this test case if Java version is not 8.
         return;
     }
   }