Review Request: JDK-8173412 @Test in java/lang/annotation and java/lang/reflect/Proxy tests not run

Claes Redestad claes.redestad at oracle.com
Sat Feb 4 00:26:12 UTC 2017


+1

On 2017-02-04 01:14, Mandy Chung wrote:
> Fix @Test tests to be public.
>
> Mandy
>
> diff --git a/test/java/lang/annotation/AnnotationWithLambda.java b/test/java/lang/annotation/AnnotationWithLambda.java
> --- a/test/java/lang/annotation/AnnotationWithLambda.java
> +++ b/test/java/lang/annotation/AnnotationWithLambda.java
> @@ -43,7 +43,7 @@
>  public class AnnotationWithLambda {
>
>      @Test
> -    void testAnnotationWithLambda() {
> +    public void testAnnotationWithLambda() {
>          Method[] methods = AnnotationWithLambda.MethodsWithAnnotations.class.getDeclaredMethods();
>          for (Method method : methods) {
>              assertTrue((method.isAnnotationPresent(LambdaWithParameter.class)) &&
> diff --git a/test/java/lang/reflect/Proxy/ProxyForMethodHandle.java b/test/java/lang/reflect/Proxy/ProxyForMethodHandle.java
> --- a/test/java/lang/reflect/Proxy/ProxyForMethodHandle.java
> +++ b/test/java/lang/reflect/Proxy/ProxyForMethodHandle.java
> @@ -46,7 +46,7 @@
>       * to a dynamic module
>       */
>      @Test
> -    static void testRunnableMethodHandle() throws Exception {
> +    public static void testRunnableMethodHandle() throws Exception {
>          MethodHandles.Lookup lookup = MethodHandles.lookup();
>          MethodType mt = MethodType.methodType(void.class);
>          MethodHandle mh = lookup.findStatic(ProxyForMethodHandle.class, "runForRunnable", mt);
>


More information about the core-libs-dev mailing list