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

[Claes Redestad]

+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);