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

Mandy Chung mandy.chung at oracle.com
Sat Feb 4 00:14:44 UTC 2017


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