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: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to