[jboss-dev-forums] [Design the new POJO MicroContainer] - MainDeployerImpl deploy/undeploy using wrong DeploymentConte

[alesj]

While fixing Module::reset for Controller::removeAlias,
I saw that this two cases don't behave the same, all though they should:

  | Deployment deployment = ...;
  | DeployerClient dc = ...;
  | 
  | dc.addDeployment(deployment);
  | dc.proocess();
  | 
  | // vs.
  | 
  | dc.deploy(deployment);
  | 
  | // same for undeploy(ment)
  | 

This was due to the fact that Deployment::name is not necessarily equal to DeploymentContext::name,
as my ModuleRemoveUnitTestCase test shows.

I've fixed it to use the right method --> getTopLevelDeploymentContext.

I guess this is the 2nd bug, which calls for another release. :-( 

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

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