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