[jboss-cvs] JBossAS SVN: r74098 - in projects/metadata/trunk/src: test/java/org/jboss/test/metadata/ejb and 1 other directory.

jboss-cvs-commits at lists.jboss.org jboss-cvs-commits at lists.jboss.org
Wed Jun 4 08:40:21 EDT 2008


Author: emuckenhuber
Date: 2008-06-04 08:40:21 -0400 (Wed, 04 Jun 2008)
New Revision: 74098

Modified:
   projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/JBossEnterpriseBeanMetaData.java
   projects/metadata/trunk/src/test/java/org/jboss/test/metadata/ejb/ResolvedJndiNameUnitTestCase.java
Log:
JBMETA-42: remove exception on unknown interface 

Modified: projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/JBossEnterpriseBeanMetaData.java
===================================================================
--- projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/JBossEnterpriseBeanMetaData.java	2008-06-04 12:39:37 UTC (rev 74097)
+++ projects/metadata/trunk/src/main/java/org/jboss/metadata/ejb/jboss/JBossEnterpriseBeanMetaData.java	2008-06-04 12:40:21 UTC (rev 74098)
@@ -695,13 +695,8 @@
             ifaceType = KnownInterfaceType.LOCAL_HOME;
          else if (ebeanMD.getHome() != null && ebeanMD.getHome().equals(iface))
             ifaceType = KnownInterfaceType.REMOTE_HOME;
-      }
+      }      
       
-      // Check if it is know interface
-      if( ifaceType == KnownInterfaceType.UNKNOWN)
-         throw new IllegalArgumentException("Can't find interface " + iface + " in " + this);
-      
-      
       // Have the policy generate the actual name
       String resolvedJndiName = policy.getJndiName(ejbSummary, iface, ifaceType);
 

Modified: projects/metadata/trunk/src/test/java/org/jboss/test/metadata/ejb/ResolvedJndiNameUnitTestCase.java
===================================================================
--- projects/metadata/trunk/src/test/java/org/jboss/test/metadata/ejb/ResolvedJndiNameUnitTestCase.java	2008-06-04 12:39:37 UTC (rev 74097)
+++ projects/metadata/trunk/src/test/java/org/jboss/test/metadata/ejb/ResolvedJndiNameUnitTestCase.java	2008-06-04 12:40:21 UTC (rev 74098)
@@ -141,10 +141,14 @@
       sbeanMD.setHome("org.jboss.ifaces.HomeIF");
       sbeanMD.setRemote("org.jboss.ifaces.RemoteIF");
 
+      String resolvedJndiName = beanMD.determineResolvedJndiName(null, null);
+      assertEquals("testResolvedJndiNamesWithKnownIfacesEntity-jndi-name", resolvedJndiName);      
       String resolvedJndiNameHome = beanMD.determineResolvedJndiName("org.jboss.ifaces.HomeIF", null);
       assertEquals("testResolvedJndiNamesWithKnownIfacesEntity-jndi-name/home", resolvedJndiNameHome);      
       String resolvedJndiNameLocalHome = beanMD.determineResolvedJndiName("org.jboss.ifaces.LocalHomeIF", null);
       assertEquals("testResolvedJndiNamesWithKnownIfacesEntity-jndi-name/localHome", resolvedJndiNameLocalHome);      
+      String resolvedJndiNameIface = beanMD.determineResolvedJndiName("org.jboss.test.some.IFace", null);
+      assertEquals("testResolvedJndiNamesWithKnownIfacesEntity-jndi-name/org.jboss.test.some.IFace", resolvedJndiNameIface);   
    }
 
    /**




More information about the jboss-cvs-commits mailing list