Author: jawi
Date: Wed Oct 23 07:18:28 2013
New Revision: 1534935
URL: http://svn.apache.org/r1534935
Log:
Once again a fix to make this itest behave properly with the other itests
without breaking them.
Modified:
ace/trunk/org.apache.ace.agent.itest/src/org/apache/ace/agent/itest/AgentExtensionTest.java
Modified:
ace/trunk/org.apache.ace.agent.itest/src/org/apache/ace/agent/itest/AgentExtensionTest.java
URL:
http://svn.apache.org/viewvc/ace/trunk/org.apache.ace.agent.itest/src/org/apache/ace/agent/itest/AgentExtensionTest.java?rev=1534935&r1=1534934&r2=1534935&view=diff
==============================================================================
---
ace/trunk/org.apache.ace.agent.itest/src/org/apache/ace/agent/itest/AgentExtensionTest.java
(original)
+++
ace/trunk/org.apache.ace.agent.itest/src/org/apache/ace/agent/itest/AgentExtensionTest.java
Wed Oct 23 07:18:28 2013
@@ -40,47 +40,45 @@ import org.osgi.framework.ServiceRegistr
public class AgentExtensionTest extends BaseAgentTest {
public void testLifecycle() throws Exception {
- try {
- AgentControl agentControl = getService(AgentControl.class);
- assertNotNull(agentControl);
+ AgentControl agentControl = getService(AgentControl.class);
+ assertNotNull(agentControl);
- getAgentBundle().stop();
- System.setProperty(AgentConstants.CONFIG_IDENTIFICATION_DISABLED,
"true");
- System.setProperty(AgentConstants.CONFIG_DISCOVERY_DISABLED,
"true");
- System.setProperty(AgentConstants.CONFIG_CONNECTION_DISABLED,
"true");
- getAgentBundle().start();
+ getAgentBundle().stop();
+ System.setProperty(AgentConstants.CONFIG_IDENTIFICATION_DISABLED,
"true");
+ System.setProperty(AgentConstants.CONFIG_DISCOVERY_DISABLED, "true");
+ System.setProperty(AgentConstants.CONFIG_CONNECTION_DISABLED, "true");
+ getAgentBundle().start();
- assertNull(locateService(AgentControl.class));
+ assertNull(locateService(AgentControl.class));
- ServiceRegistration idreg1 = registerIdentification("TEST1", 1);
- assertNull(locateService(AgentControl.class));
- ServiceRegistration direg1 = registerDiscovery(new
URL("http://test1"), 1);
- assertNull(locateService(AgentControl.class));
- ServiceRegistration coreg1 = registerConnectionHandler();
- assertNotNull(locateService(AgentControl.class));
+ ServiceRegistration idreg1 = registerIdentification("TEST1", 1);
+ assertNull(locateService(AgentControl.class));
+ ServiceRegistration direg1 = registerDiscovery(new
URL("http://test1"), 1);
+ assertNull(locateService(AgentControl.class));
+ ServiceRegistration coreg1 = registerConnectionHandler();
+ assertNotNull(locateService(AgentControl.class));
- assertEquals("TEST1",
locateService(AgentControl.class).getAgentId());
+ assertEquals("TEST1", locateService(AgentControl.class).getAgentId());
- ServiceRegistration idreg2 = registerIdentification("TEST2", 2);
+ ServiceRegistration idreg2 = registerIdentification("TEST2", 2);
- assertEquals("TEST2",
locateService(AgentControl.class).getAgentId());
+ assertEquals("TEST2", locateService(AgentControl.class).getAgentId());
- idreg2.unregister();
+ idreg2.unregister();
- assertEquals("TEST1",
locateService(AgentControl.class).getAgentId());
+ assertEquals("TEST1", locateService(AgentControl.class).getAgentId());
- idreg1.unregister();
+ idreg1.unregister();
- assertNull(locateService(AgentControl.class));
+ assertNull(locateService(AgentControl.class));
- direg1.unregister();
- coreg1.unregister();
- }
- finally {
-
System.clearProperty(AgentConstants.CONFIG_IDENTIFICATION_DISABLED);
- System.clearProperty(AgentConstants.CONFIG_DISCOVERY_DISABLED);
- System.clearProperty(AgentConstants.CONFIG_CONNECTION_DISABLED);
- }
+ direg1.unregister();
+ coreg1.unregister();
+ }
+
+ @Override
+ protected void doTearDown() throws Exception {
+ resetAgentBundleState();
}
private ServiceRegistration registerIdentification(final String id, final
int rank) {