svn commit: r1594611 - /felix/sandbox/pderop/dependencymanager-prototype/dm/test/test/ComponentTest.java

[pd...@apache.org]

Author: pderop
Date: Wed May 14 15:42:02 2014
New Revision: 1594611

URL: http://svn.apache.org/r1594611
Log:
Modified the testAddAvailableDependencyFromInitCallback() method, because the ComponentImpl.handleChanged method is not recursive anymore.
(see comments in the ComponentImpl.handleChanged method).

Modified:
    felix/sandbox/pderop/dependencymanager-prototype/dm/test/test/ComponentTest.java

Modified: felix/sandbox/pderop/dependencymanager-prototype/dm/test/test/ComponentTest.java
URL: http://svn.apache.org/viewvc/felix/sandbox/pderop/dependencymanager-prototype/dm/test/test/ComponentTest.java?rev=1594611&r1=1594610&r2=1594611&view=diff
==============================================================================
--- felix/sandbox/pderop/dependencymanager-prototype/dm/test/test/ComponentTest.java (original)
+++ felix/sandbox/pderop/dependencymanager-prototype/dm/test/test/ComponentTest.java Wed May 14 15:42:02 2014
@@ -98,29 +98,29 @@ public class ComponentTest {
 				e.step(2);
 				c.add(d);
 				d.add(new EventImpl());
-				c.add(d2);
+				c.add(d2); // not available, the start method won't be called
 			}
 			void start() {
 				System.out.println("start");
-				e.step();
+				e.step(4);
 			}
 			void stop() {
 				System.out.println("stop");
-				e.step();
+				e.step(6);
 			}
 			void destroy() {
 				System.out.println("destroy");
-				e.step(9);
+				e.step(7);
 			}
 		});
 		e.step(1);
 		c.start();
-		e.step(5);
+		e.step(3);
 		d2.add(new EventImpl());
-		e.step(7);
+		e.step(5);
 		d.remove(new EventImpl());
 		c.stop();
-		e.step(10);
+		e.step(8);
 	}
 
 	@Test
@@ -218,7 +218,7 @@ public class ComponentTest {
 		d.setRequired(true);
 		ComponentStateListener l = new ComponentStateListener() {
 			@Override
-			public void changed(ComponentState state) {
+			public void changed(Component c, ComponentState state) {
 				// make the dependency unavailable
 				d.remove(new EventImpl());
 			}