felix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From rickh...@apache.org
Subject svn commit: r1027251 - in /felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation: bnd.bnd src/org/apache/felix/test/framework/bootdelegation/BootDelegationTest.java src/org/apache/felix/test/framework/bootdelegation/HelloImpl.java
Date Mon, 25 Oct 2010 20:06:59 GMT
Author: rickhall
Date: Mon Oct 25 20:06:59 2010
New Revision: 1027251

URL: http://svn.apache.org/viewvc?rev=1027251&view=rev
Log:
Add reflective test case.

Added:
    felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/HelloImpl.java
Modified:
    felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/bnd.bnd
    felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/BootDelegationTest.java

Modified: felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/bnd.bnd
URL: http://svn.apache.org/viewvc/felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/bnd.bnd?rev=1027251&r1=1027250&r2=1027251&view=diff
==============================================================================
--- felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/bnd.bnd
(original)
+++ felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/bnd.bnd
Mon Oct 25 20:06:59 2010
@@ -1,4 +1,4 @@
--runvm = -Xdebug, "-Xrunjdwp:transport=dt_socket,address=8001,server=y,suspend=n"
+-runvm = -Xdebug, "-Xrunjdwp:transport=dt_socket,address=8001,server=y,suspend=y"
 
 # Override runpath to include content on class path
 -runpath                = \

Modified: felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/BootDelegationTest.java
URL: http://svn.apache.org/viewvc/felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/BootDelegationTest.java?rev=1027251&r1=1027250&r2=1027251&view=diff
==============================================================================
--- felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/BootDelegationTest.java
(original)
+++ felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/BootDelegationTest.java
Mon Oct 25 20:06:59 2010
@@ -18,6 +18,7 @@
  */
 package org.apache.felix.test.framework.bootdelegation;
 
+import java.lang.reflect.Constructor;
 import java.lang.reflect.InvocationHandler;
 import java.lang.reflect.Method;
 import java.lang.reflect.Proxy;
@@ -494,51 +495,25 @@ public class BootDelegationTest extends 
         }
     }
 
-    public void testImplicitBootDelegationWithProxies()
+    public void testImplicitBootDelegationWithReflection()
     {
-        // Try to trigger implicit boot delegation under these conditions:
-        //    - Instigating stack:
-        //      - this bundle class,
-        //      - external class.
-        //    - Class load requested via:
-        //      - Bundle.loadClass().
-        // Implicit boot delegation should NOT occur.
-        InvocationHandler ih = new InvocationHandler() {
-            public Object invoke(Object o, Method method, Object[] os) throws Throwable
-            {
-                // No matter what method is thrown, just print hello.
-                System.out.println("1: Hello from proxied object!");
-                return null;
-            }
-        };
-        Hello hello = (Hello) Proxy.newProxyInstance(
-            this.getClass().getClassLoader(),
-            new Class[] { Hello.class },
-            ih);
-        hello.sayHello();
-
-        // Try to trigger implicit boot delegation under these conditions:
-        //    - Instigating stack:
-        //      - this bundle class,
-        //      - external class.
-        //    - Class load requested via:
-        //      - Bundle.loadClass().
-        // Implicit boot delegation should NOT occur.
-        ih = new InvocationHandler() {
-            public Object invoke(Object o, Method method, Object[] os) throws Throwable
+        // Try to use reflection numerous times (e.g., at least 15) to trigger
+        // internal JVM optimization which causes boot delegation for sun.reflect
+        // classes.
+        // See: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=6265952
+        try
+        {
+            for (int i = 0; i < 20; i++)
             {
-                // No matter what method is thrown, just print hello.
-                System.out.println("2: Hello from proxied object!");
-                return null;
+                Constructor ctor = HelloImpl.class.getConstructor((Class[]) null);
+                Object o = ctor.newInstance((Object[]) null);
+                Method m = o.getClass().getDeclaredMethod("sayHello", (Class[]) null);
+                m.invoke(o, (Object[]) null);
             }
-        };
-        hello = (Hello) Proxy.newProxyInstance(
-            this.getClass().getClassLoader(),
-            new Class[] { Hello.class },
-            ih);
-        for (int i = 0; i < 20; i++)
+        }
+        catch (Exception ex)
         {
-            hello.sayHello();
+            fail("Exception while using reflection: " + ex);
         }
     }
 }
\ No newline at end of file

Added: felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/HelloImpl.java
URL: http://svn.apache.org/viewvc/felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/HelloImpl.java?rev=1027251&view=auto
==============================================================================
--- felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/HelloImpl.java
(added)
+++ felix/sandbox/rickhall/bnd-test/org.apache.felix.test.framework.bootdelegation/src/org/apache/felix/test/framework/bootdelegation/HelloImpl.java
Mon Oct 25 20:06:59 2010
@@ -0,0 +1,27 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ * 
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+package org.apache.felix.test.framework.bootdelegation;
+
+public class HelloImpl implements Hello
+{
+    public void sayHello()
+    {
+        System.out.println("+++ HelloImpl says 'Hello'");
+    }
+}
\ No newline at end of file



Mime
View raw message