Re: [14]RFR of JDK-8227289:Enable assertions for some shell to java conversion tests after JDK-8218960

2019-07-10 Thread naoto . sato

Looks good to me.

Naoto

On 7/9/19 11:53 PM, Ying Zhou wrote:

Hello,

Please review this patch for updating below tests by adding -ea/esa to 
launcher parameters.


- /test/jdk/java/util/Calendar/SupplementalJapaneseEraTestRun.java
- /test/jdk/java/util/TimeZone/TimeZoneDatePermissionCheckRun.java
- /test/jdk/java/util/TimeZone/Bug8066652Run.java
- 
/test/jdk/java/util/ResourceBundle/Control/MissingResourceCauseTestRun.java

- /test/jdk/java/util/ResourceBundle/modules/ModuleTestUtil.java
- /test/jdk/java/util/ResourceBundle/modules/layer/LayerTest.java
- /test/jdk/java/util/ResourceBundle/modules/unnamed/UnNamedTest.java
- /test/jdk/java/util/ResourceBundle/modules/visibility/VisibilityTest.java

JDK Bug: JDK-8227289 

webrev: http://cr.openjdk.java.net/~yzhou/8227289/webrev.00/

Thanks,

Daisy


[14]RFR of JDK-8227289:Enable assertions for some shell to java conversion tests after JDK-8218960

2019-07-09 Thread Ying Zhou

Hello,

Please review this patch for updating below tests by adding -ea/esa to 
launcher parameters.


- /test/jdk/java/util/Calendar/SupplementalJapaneseEraTestRun.java
- /test/jdk/java/util/TimeZone/TimeZoneDatePermissionCheckRun.java
- /test/jdk/java/util/TimeZone/Bug8066652Run.java
- 
/test/jdk/java/util/ResourceBundle/Control/MissingResourceCauseTestRun.java

- /test/jdk/java/util/ResourceBundle/modules/ModuleTestUtil.java
- /test/jdk/java/util/ResourceBundle/modules/layer/LayerTest.java
- /test/jdk/java/util/ResourceBundle/modules/unnamed/UnNamedTest.java
- /test/jdk/java/util/ResourceBundle/modules/visibility/VisibilityTest.java

JDK Bug: JDK-8227289 

webrev: http://cr.openjdk.java.net/~yzhou/8227289/webrev.00/

Thanks,

Daisy