RFR: 8202810: Remove class-for-name test

mandy chung mandy.chung at oracle.com
Tue May 8 20:51:37 UTC 2018


+1

I checked and this is the only test needed update.

Mandy

On 5/8/18 1:45 PM, Claes Redestad wrote:
> Hi,
>
> please review the removal of this test that explicitly attempts to run 
> the now removed jlink --class-for-name plugin:
>
> diff -r 7f05a1e6a02a test/jdk/tools/jlink/JLinkPluginsTest.java
> --- a/test/jdk/tools/jlink/JLinkPluginsTest.java    Tue May 08 
> 15:42:19 2018 +0200
> +++ b/test/jdk/tools/jlink/JLinkPluginsTest.java    Tue May 08 
> 22:46:14 2018 +0200
> @@ -77,14 +77,6 @@
>              helper.checkImage(imageDir, moduleName, res, null);
>          }
>          {
> -            // Optimize Class.forName
> -            String[] userOptions = {"--class-for-name"};
> -            String moduleName = "classforname";
> -            helper.generateDefaultJModule(moduleName, "composite2");
> -            Path imageDir = helper.generateDefaultImage(userOptions, 
> moduleName).assertSuccess();
> -            helper.checkImage(imageDir, moduleName, null, null);
> -        }
> -        {
>              // disable generate jli classes - JDK-8160063
>              String[] userOptions = {"--disable-plugin", 
> "generate-jli-classes"};
>              String moduleName = "jlidisabled";
>
> /Claes



More information about the jigsaw-dev mailing list