You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by is...@apache.org on 2007/02/21 23:09:19 UTC

svn commit: r510263 - in /incubator/tuscany/sandbox/isilval/notification/local/src: main/java/org/apache/tuscany/notification/local/ test/java/org/apache/tuscany/notification/local/ test/resources/

Author: isilval
Date: Wed Feb 21 14:09:18 2007
New Revision: 510263

URL: http://svn.apache.org/viewvc?view=rev&rev=510263
Log:
More bug fixes

Modified:
    incubator/tuscany/sandbox/isilval/notification/local/src/main/java/org/apache/tuscany/notification/local/LocalNotificationComponent.java
    incubator/tuscany/sandbox/isilval/notification/local/src/test/java/org/apache/tuscany/notification/local/LocalNotificationITestCase.java
    incubator/tuscany/sandbox/isilval/notification/local/src/test/resources/itest.scdl

Modified: incubator/tuscany/sandbox/isilval/notification/local/src/main/java/org/apache/tuscany/notification/local/LocalNotificationComponent.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/sandbox/isilval/notification/local/src/main/java/org/apache/tuscany/notification/local/LocalNotificationComponent.java?view=diff&rev=510263&r1=510262&r2=510263
==============================================================================
--- incubator/tuscany/sandbox/isilval/notification/local/src/main/java/org/apache/tuscany/notification/local/LocalNotificationComponent.java (original)
+++ incubator/tuscany/sandbox/isilval/notification/local/src/main/java/org/apache/tuscany/notification/local/LocalNotificationComponent.java Wed Feb 21 14:09:18 2007
@@ -99,17 +99,23 @@
     class LocalNotificationTargetInvoker extends TargetInvokerExtension {
         
         private List<Interceptor> subscriberInterceptors;
+        private Operation operation;
         
         public LocalNotificationTargetInvoker(Operation operation, WorkContext wc, ExecutionMonitor mon) {
             super(wc, mon);
-            subscriberInterceptors = new ArrayList<Interceptor>();
-            for (Wire wire : wires) {
-                InvocationChain chain = wire.getInvocationChains().get(operation);
-                subscriberInterceptors.add(chain.getHeadInterceptor());
-            }
+            this.subscriberInterceptors = null;
+            this.operation = operation;
         }
 
         public Message invoke(Message msg) throws InvocationRuntimeException {
+            if (subscriberInterceptors == null) {
+                subscriberInterceptors = new ArrayList<Interceptor>();
+                for (Wire wire : wires) {
+                    InvocationChain chain = wire.getInvocationChains().get(operation);
+                    subscriberInterceptors.add(chain.getHeadInterceptor());
+                }
+            }
+            
             // REVIEW Should this be done in separate thread(s)?
             // REVIEW Should separate copies of message be used?
             for (Interceptor subscriberInterceptor : subscriberInterceptors) {

Modified: incubator/tuscany/sandbox/isilval/notification/local/src/test/java/org/apache/tuscany/notification/local/LocalNotificationITestCase.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/sandbox/isilval/notification/local/src/test/java/org/apache/tuscany/notification/local/LocalNotificationITestCase.java?view=diff&rev=510263&r1=510262&r2=510263
==============================================================================
--- incubator/tuscany/sandbox/isilval/notification/local/src/test/java/org/apache/tuscany/notification/local/LocalNotificationITestCase.java (original)
+++ incubator/tuscany/sandbox/isilval/notification/local/src/test/java/org/apache/tuscany/notification/local/LocalNotificationITestCase.java Wed Feb 21 14:09:18 2007
@@ -25,7 +25,7 @@
 public class LocalNotificationITestCase extends TestCase {
     
     @Reference
-    TestCaseProducer producer;
+    protected TestCaseProducer producer;
     
     public void testTrafficNotification() throws Exception {
         try {

Modified: incubator/tuscany/sandbox/isilval/notification/local/src/test/resources/itest.scdl
URL: http://svn.apache.org/viewvc/incubator/tuscany/sandbox/isilval/notification/local/src/test/resources/itest.scdl?view=diff&rev=510263&r1=510262&r2=510263
==============================================================================
--- incubator/tuscany/sandbox/isilval/notification/local/src/test/resources/itest.scdl (original)
+++ incubator/tuscany/sandbox/isilval/notification/local/src/test/resources/itest.scdl Wed Feb 21 14:09:18 2007
@@ -25,7 +25,7 @@
              scdlResource="META-INF/sca/TrafficAdvisoryNotificationLocal.scdl"/>
 
     <component name="testMyService">
-        <tuscany:junit class="LocalNotificationITestCase"/>
+        <tuscany:junit class="org.apache.tuscany.notification.local.LocalNotificationITestCase"/>
         <reference name="producer">TrafficAdvisoryProducer</reference>
     </component>
 



---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org