[jboss-dev-forums] [Design of EJB 3.0] - Re: EJBTHREE-1454 Encapsulate Container infomation in TO/VO

[jaikiran]

"ALRubinger" wrote : * The patch has a series of errors in the Proxy SessionContainerTestCase, references to container.getName().  How were we handling this in InvokableContext - is "getName" valid, or should these references be using "getGuid" or "getDispatcherRegistrationName"?
  | 

Andrew,

Are you sure the patch got applied correctly? On my local setup, the SessionContainerTestCase does not have reference to getName. Instead it has been replaced with getDispatcherRegistrationName. All the unit tests pass in my workspace. I rechecked the attached patch and here's an extract:

  | Index: proxy/src/test/java/org/jboss/ejb3/test/proxy/common/container/unit/SessionContainerTestCase.java
  | ===================================================================
  | --- proxy/src/test/java/org/jboss/ejb3/test/proxy/common/container/unit/SessionContainerTestCase.java	(revision 78500)
  | +++ proxy/src/test/java/org/jboss/ejb3/test/proxy/common/container/unit/SessionContainerTestCase.java	(working copy)
  | @@ -111,16 +111,16 @@
  |  
  |        if (sessionContainer != null)
  |        {
  | -         logger.info("Unbinding: " + sessionContainer.getName());
  | +         logger.info("Unbinding: " + sessionContainer.getDispatcherRegistrationName());
  |           try
  |           {
  | -            Ejb3RegistrarLocator.locateRegistrar().unbind(sessionContainer.getName());
  | +            Ejb3RegistrarLocator.locateRegistrar().unbind(sessionContainer.getDispatcherRegistrationName());
  |           }
  |           catch (NotBoundException nbe)
  |           {
  |              // we are ok with this exception, which indicates that the test case had 
  |              // already unbound the ejb related bindings.
  | -            logger.debug(sessionContainer.getName() + " was already unbound");
  | +            logger.debug(sessionContainer.getDispatcherRegistrationName() + " was already unbound");
  |  
  |           }
  |        }
  | 


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

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