You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@poi.apache.org by ce...@apache.org on 2018/10/26 09:32:01 UTC

svn commit: r1844874 - /poi/trunk/src/testcases/org/apache/poi/ss/util/TestDateFormatConverter.java

Author: centic
Date: Fri Oct 26 09:32:01 2018
New Revision: 1844874

URL: http://svn.apache.org/viewvc?rev=1844874&view=rev
Log:
Remove the workaround for a bug in JDK 11 which was fixed in ea+28

Modified:
    poi/trunk/src/testcases/org/apache/poi/ss/util/TestDateFormatConverter.java

Modified: poi/trunk/src/testcases/org/apache/poi/ss/util/TestDateFormatConverter.java
URL: http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/ss/util/TestDateFormatConverter.java?rev=1844874&r1=1844873&r2=1844874&view=diff
==============================================================================
--- poi/trunk/src/testcases/org/apache/poi/ss/util/TestDateFormatConverter.java (original)
+++ poi/trunk/src/testcases/org/apache/poi/ss/util/TestDateFormatConverter.java Fri Oct 26 09:32:01 2018
@@ -99,15 +99,6 @@ public final class TestDateFormatConvert
                     row.createCell(5).setCellValue(javaDateFormatPattern);
                     row.createCell(6).setCellValue(excelFormatPattern);
                 } catch (Exception e) {
-                    // this can be removed after https://bugs.openjdk.java.net/browse/JDK-8209047 is available
-                    // in JDK 11 ea > 26
-                    if(locale.toString().startsWith("my") &&
-                            e.getMessage().contains("Illegal pattern character 'B'") &&
-                                    System.getProperty("java.version").startsWith("11")) {
-                        System.out.println("DateFormat.getDateTimeInstance() fails for Malaysian Locale on JDK 11, submitted bug report to Oracle");
-                        continue;
-                    }
-
                     throw new RuntimeException(
                             "Failed for locale: " + locale + " and style " + style + "\n" +
                             "Having locales: " + Arrays.toString(DateFormat.getAvailableLocales()), e);
@@ -153,7 +144,6 @@ public final class TestDateFormatConvert
         DateFormatConverter.getPrefixForLocale(new Locale(""));
     }
 
-    @Ignore("Fails on JDK 11, submitted as ID : 9056763")
     @Test
     public void testJDK11MyLocale() {
         DateFormat.getDateTimeInstance(DateFormat.DEFAULT, DateFormat.DEFAULT, Locale.forLanguageTag("my"));



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@poi.apache.org
For additional commands, e-mail: commits-help@poi.apache.org