RFR (XXS) 8174862: JShell tests: new JDK-8174797 testInvalidClassPath fails on Windows

Jan Lahoda jan.lahoda at oracle.com
Mon Feb 13 21:30:19 UTC 2017


Seems OK.

Jan

On 13.2.2017 22:14, Robert Field wrote:
> A new test uses a platform-dependent file separator ("/").  The test
> doesn't need a file separator.  Fix is simply to remove it from command
> and expected resulting error.
> Second failure is just this same test inherited.
> Please review....
>
> Bug:
>
>      https://bugs.openjdk.java.net/browse/JDK-8174862
>
> hg diff --
>
> --- a/test/jdk/jshell/ToolSimpleTest.java       Mon Feb 13 12:14:23 2017
> -0800
> +++ b/test/jdk/jshell/ToolSimpleTest.java       Mon Feb 13 13:07:20 2017
> -0800
> @@ -214,8 +214,8 @@
>       @Test
>       public void testInvalidClassPath() {
>           test(
> -                a -> assertCommand(a, "/env --class-path snurge/fusal",
> -                        "|  File 'snurge/fusal' for '--class-path' is
> not found.")
> +                a -> assertCommand(a, "/env --class-path snurgefusal",
> +                        "|  File 'snurgefusal' for '--class-path' is
> not found.")
>           );
>       }
>
>
>
> Thanks,
> Robert
>


More information about the kulla-dev mailing list