svn commit: r1124284 - /felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FELIX2955_ShellCommandTest.java

[ma...@apache.org]

Author: marrs
Date: Wed May 18 14:33:39 2011
New Revision: 1124284

URL: http://svn.apache.org/viewvc?rev=1124284&view=rev
Log:
FELIX-2955 Small bugfix to the test, it still failed because I forgot to do a bit of cleanup halfway through the testcase.

Modified:
    felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FELIX2955_ShellCommandTest.java

Modified: felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FELIX2955_ShellCommandTest.java
URL: http://svn.apache.org/viewvc/felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FELIX2955_ShellCommandTest.java?rev=1124284&r1=1124283&r2=1124284&view=diff
==============================================================================
--- felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FELIX2955_ShellCommandTest.java (original)
+++ felix/trunk/dependencymanager/test/src/test/java/org/apache/felix/dm/test/FELIX2955_ShellCommandTest.java Wed May 18 14:33:39 2011
@@ -88,6 +88,7 @@ public class FELIX2955_ShellCommandTest 
         m.add(missing);
         e.step(4);
         e.waitForStep(5, 5000);
+        m.remove(missing);
         // now start/stop deploymentadmin, which we use here because it's a bundle that
         // publishes a service that uses the dependency manager (saving us from having to
         // create a bundle that does that on the fly)
@@ -96,7 +97,6 @@ public class FELIX2955_ShellCommandTest 
         e.step(6);
         e.waitForStep(7, 5000);
         e.ensure();
-        m.remove(missing);
         m.remove(shellClient);
         
     }