felix-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pde...@apache.org
Subject svn commit: r1629078 - /felix/sandbox/pderop/dependencymanager-prototype/org.apache.felix.dependencymanager/test/test/ServiceRaceTest.java
Date Thu, 02 Oct 2014 21:52:58 GMT
Author: pderop
Date: Thu Oct  2 21:52:57 2014
New Revision: 1629078

URL: http://svn.apache.org/r1629078
Log:
Fixed a bug in the test: The client service dependencies must be added after the client has
been started.

Modified:
    felix/sandbox/pderop/dependencymanager-prototype/org.apache.felix.dependencymanager/test/test/ServiceRaceTest.java

Modified: felix/sandbox/pderop/dependencymanager-prototype/org.apache.felix.dependencymanager/test/test/ServiceRaceTest.java
URL: http://svn.apache.org/viewvc/felix/sandbox/pderop/dependencymanager-prototype/org.apache.felix.dependencymanager/test/test/ServiceRaceTest.java?rev=1629078&r1=1629077&r2=1629078&view=diff
==============================================================================
--- felix/sandbox/pderop/dependencymanager-prototype/org.apache.felix.dependencymanager/test/test/ServiceRaceTest.java
(original)
+++ felix/sandbox/pderop/dependencymanager-prototype/org.apache.felix.dependencymanager/test/test/ServiceRaceTest.java
Thu Oct  2 21:52:57 2014
@@ -130,21 +130,25 @@ public class ServiceRaceTest extends Tes
         };
         client.add(listener);
 
-        // Activate the client service dependencies concurrently.
-        for (int i = 0; i < DEPENDENCIES; i++) {
-            final DependencyImpl dep = dependencies[i];
-            final Event added = new EventImpl(i);
-            m_threadpool.execute(new Runnable() {
-                public void run() {
-                    dep.add(added);
-                }
-            });
-        }
 
         // Start the client (concurrently)
         m_threadpool.execute(new Runnable() {
             public void run() {
                 client.start();
+                
+                // Activate the client service dependencies concurrently.
+                // We *must* do this after having started the component (in a reality, the
dependencies can be 
+                // injected only one the tracker has been opened ...
+                for (int i = 0; i < DEPENDENCIES; i++) {
+                    final DependencyImpl dep = dependencies[i];
+                    final Event added = new EventImpl(i);
+                    m_threadpool.execute(new Runnable() {
+                        public void run() {
+                            dep.add(added);
+                        }
+                    });
+                }
+
             }
         });
 



Mime
View raw message