[jboss-cvs] JBossAS SVN: r82921 - projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test.

[jboss-cvs-commits at lists.jboss.org]

Author: kabir.khan at jboss.com
Date: 2009-01-15 13:23:31 -0500 (Thu, 15 Jan 2009)
New Revision: 82921

Modified:
   projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test/ClassPoolTest.java
Log:
[JBAOP-666] Fix bug, hard reference to loader should be kept

Modified: projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test/ClassPoolTest.java
===================================================================
--- projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test/ClassPoolTest.java	2009-01-15 17:29:45 UTC (rev 82920)
+++ projects/aop/trunk/asintegration-core/src/test/java/org/jboss/test/aop/classpool/test/ClassPoolTest.java	2009-01-15 18:23:31 UTC (rev 82921)
@@ -103,7 +103,9 @@
       }
       //Once all the classes in the loader have been loaded, it seems to clear the URLs. Work around this 
       //by recreating the loader since we need the urls in the URLClassLoaderIsLocalResourcePlugin
-      loaders.add(new URLClassLoader(urls));
+      loader = new URLClassLoader(urls);
+      //Add hard reference to loader
+      loaders.add(loader);
       return new DelegatingClassPool(domain, loader, ClassPool.getDefault(), AOPClassPoolRepository.getInstance());
    }