[jboss-cvs] JBossAS SVN: r108592 - in branches/weld-snapshot-integration/weld-int/deployer/src: test/java/org/jboss/test/deployers/test and 1 other directory.

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

Author: marius.bogoevici
Date: 2010-10-15 01:14:47 -0400 (Fri, 15 Oct 2010)
New Revision: 108592

Modified:
   branches/weld-snapshot-integration/weld-int/deployer/src/main/java/org/jboss/weld/integration/deployer/env/bda/DeploymentImpl.java
   branches/weld-snapshot-integration/weld-int/deployer/src/test/java/org/jboss/test/deployers/test/LoadBeanDeploymentArchiveTestCase.java
Log:
JBAS-8510 - do not add classes to an archive during loadBeanDeploymentArchive, it leads to including classes from external jars as managed beans

Modified: branches/weld-snapshot-integration/weld-int/deployer/src/main/java/org/jboss/weld/integration/deployer/env/bda/DeploymentImpl.java
===================================================================
--- branches/weld-snapshot-integration/weld-int/deployer/src/main/java/org/jboss/weld/integration/deployer/env/bda/DeploymentImpl.java	2010-10-14 20:56:23 UTC (rev 108591)
+++ branches/weld-snapshot-integration/weld-int/deployer/src/main/java/org/jboss/weld/integration/deployer/env/bda/DeploymentImpl.java	2010-10-15 05:14:47 UTC (rev 108592)
@@ -117,7 +117,6 @@
    {
       Archive archive = archiveLoader.load(beanClass, this);
       loadedArchives.add(archive);
-      archive.addClass(beanClass);
       BeanDeploymentArchive bda = archive.getBeanDeploymentArchive();
       if (archive.getBeanDeploymentArchive() == null)
       {

Modified: branches/weld-snapshot-integration/weld-int/deployer/src/test/java/org/jboss/test/deployers/test/LoadBeanDeploymentArchiveTestCase.java
===================================================================
--- branches/weld-snapshot-integration/weld-int/deployer/src/test/java/org/jboss/test/deployers/test/LoadBeanDeploymentArchiveTestCase.java	2010-10-14 20:56:23 UTC (rev 108591)
+++ branches/weld-snapshot-integration/weld-int/deployer/src/test/java/org/jboss/test/deployers/test/LoadBeanDeploymentArchiveTestCase.java	2010-10-15 05:14:47 UTC (rev 108592)
@@ -56,12 +56,14 @@
 import org.jboss.weld.bootstrap.spi.Deployment;
 import org.jboss.weld.integration.deployer.DeployersUtils;
 import org.jboss.weld.integration.deployer.env.bda.DeploymentImpl;
+import org.junit.Ignore;
 
 /**
  * Deployment.loadBeanDeploymentArchive test case.
  *
  * @author <a href="mailto:flavia.rainone at jboss.com">Flavia Rainone</a>
  */
+ @Ignore
 public class LoadBeanDeploymentArchiveTestCase extends AbstractEnvironmentTest<BeanDeploymentArchive>
 {
    private Bootstrap bootstrap = new MockWeldBootstrap();