[jboss-dev-forums] [Design of POJO Server] - Re: Regression in Smoke Test - WebIntegrationUnitTestCase

[ALRubinger]

This patch brings the WebIntegrationUnitTestCase back in line:

Index: server/src/main/org/jboss/deployment/MappedReferenceMetaDataResolverDeployer.java
  | ===================================================================
  | --- server/src/main/org/jboss/deployment/MappedReferenceMetaDataResolverDeployer.java	(revision 75930)
  | +++ server/src/main/org/jboss/deployment/MappedReferenceMetaDataResolverDeployer.java	(working copy)
  | @@ -390,6 +390,8 @@
  |                 {
  |                    // Add ejb/vfsPath at iface
  |                    String remote = sbean.getRemote();
  | +                  String remoteJndiName = sbean.determineJndiName();
  | +                  cdmd.addJndiName(remoteJndiName);
  |                    String ifacePath = prefix + "@" + remote;
  |                    if(endpointAlternateMap.containsKey(ifacePath))
  |                       log.debug(ejbName+" duplicates remote: "+remote+", existing: "+endpointAlternateMap.get(ifacePath));
  | @@ -451,6 +453,8 @@
  |                 {
  |                    // Add ejb/vfsPath at iface
  |                    String local = sbean.getLocal();
  | +                  String localJndiName = sbean.determineLocalJndiName();
  | +                  cdmd.addJndiName(localJndiName);
  |                    String ifacePath = prefix + "@" + local;
  |                    if(endpointAlternateMap.containsKey(ifacePath))
  |                       log.debug(ejbName+" duplicates local: "+local+", existing: "+endpointAlternateMap.get(ifacePath));
  | @@ -537,8 +541,8 @@
  |                       }
  |                       // Add ejb/iface
  |                       ifacePath = "ejb@" + remote;
  | -                     if(endpointAlternateMap.containsKey(ifacePath))
  | -                        log.debug(ejbName+" duplicates business-remote: "+remote+", existing: "+endpointAlternateMap.get(ifacePath));
  | +                     if(endpointMap.containsKey(ifacePath))
  | +                        log.debug(ejbName+" duplicates business-remote: "+remote+", existing: "+endpointMap.get(ifacePath));
  |                       else
  |                       {
  |                          endpointAlternateMap.put(ifacePath, ejbCompID);
  | @@ -946,7 +950,7 @@
  |                 // Determine the jndi name for the reference interface
  |                 String iface = getInterface(ref);
  |                 //LegacyEjb3JndiPolicy policy = new LegacyEjb3JndiPolicy();
  | -               String containerJndiName = target.getBeanMetaData().determineResolvedJndiName(iface);
  | +               String containerJndiName = target.getBeanMetaData().determineJndiName();
  |                 if(containerJndiName != null)
  |                    ref.setResolvedJndiName(containerJndiName);
  |              }

I'm testing against some of the other testsuites locally now to see what impact this has elsewhere.

S,
ALR

View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4164904#4164904

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4164904