JDK 14 RFR of JDK-8233940: Preview API tests for String methods should use ${jdk.version} as -source arg

Jan Lahoda jan.lahoda at oracle.com
Tue Nov 12 17:50:26 UTC 2019


Looks good to me.

Jan

On 12. 11. 19 7:13, Joe Darcy wrote:
> Hello,
> 
> To remove the need to modify tests when the JDK version is updated, the 
> tests of the preview API in String should use "${jdk.version}" as an 
> argument to -source rather than a fixed version string:
> 
>      JDK-8233940: Preview API tests for String methods should use 
> ${jdk.version} as -source arg
>      http://cr.openjdk.java.net/~darcy/8233940.0/
> 
> The new TEST.properties file only affects tests in the java/lang/String 
> directory. Some additional work is needed before jtreg "smart actions" 
> can be enabled throughout the tests in the "jdk" directory as they are 
> already in "langtools".
> 
> Patch below.
> 
> Thanks,
> 
> -Joe
> 
> --- old/test/jdk/java/lang/String/Formatted.java    2019-11-11 
> 22:09:42.692005000 -0800
> +++ new/test/jdk/java/lang/String/Formatted.java    2019-11-11 
> 22:09:42.476005000 -0800
> @@ -25,7 +25,7 @@
>    * @test
>    * bug 8203444
>    * @summary Unit tests for instance versions of String#format
> - * @compile --enable-preview -source 14 Formatted.java
> + * @compile --enable-preview -source ${jdk.version} Formatted.java
>    * @run main/othervm --enable-preview Formatted
>    */
> 
> --- old/test/jdk/java/lang/String/StripIndent.java    2019-11-11 
> 22:09:43.376005000 -0800
> +++ new/test/jdk/java/lang/String/StripIndent.java    2019-11-11 
> 22:09:43.120005000 -0800
> @@ -25,7 +25,7 @@
>    * @test
>    * @bug 8223775
>    * @summary This exercises String#stripIndent patterns and limits.
> - * @compile --enable-preview -source 14 StripIndent.java
> + * @compile --enable-preview -source ${jdk.version} StripIndent.java
>    * @run main/othervm --enable-preview StripIndent
>    */
> 
> --- old/test/jdk/java/lang/String/TranslateEscapes.java  2019-11-11 
> 22:09:44.020005000 -0800
> +++ new/test/jdk/java/lang/String/TranslateEscapes.java  2019-11-11 
> 22:09:43.776005000 -0800
> @@ -25,7 +25,7 @@
>    * @test
>    * @bug 8223780
>    * @summary This exercises String#translateEscapes patterns and limits.
> - * @compile --enable-preview -source 14 TranslateEscapes.java
> + * @compile --enable-preview -source ${jdk.version} TranslateEscapes.java
>    * @run main/othervm --enable-preview TranslateEscapes
>    */
> 
> --- /dev/null    2019-09-13 10:19:35.768000000 -0700
> +++ new/test/jdk/java/lang/String/TEST.properties    2019-11-11 
> 22:09:44.396005000 -0800
> @@ -0,0 +1 @@
> +allowSmartActionArgs=true
> 


More information about the core-libs-dev mailing list