You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by gn...@apache.org on 2008/11/03 12:56:18 UTC

svn commit: r710038 - in /servicemix/components/engines/servicemix-eip/trunk/src: main/java/org/apache/servicemix/eip/support/AbstractAggregator.java test/java/org/apache/servicemix/eip/RecipientListAggregatorTest.java

Author: gnodet
Date: Mon Nov  3 03:56:18 2008
New Revision: 710038

URL: http://svn.apache.org/viewvc?rev=710038&view=rev
Log:
SM-1567: fix typo (patch provided by Ron Gavlin)

Modified:
    servicemix/components/engines/servicemix-eip/trunk/src/main/java/org/apache/servicemix/eip/support/AbstractAggregator.java
    servicemix/components/engines/servicemix-eip/trunk/src/test/java/org/apache/servicemix/eip/RecipientListAggregatorTest.java

Modified: servicemix/components/engines/servicemix-eip/trunk/src/main/java/org/apache/servicemix/eip/support/AbstractAggregator.java
URL: http://svn.apache.org/viewvc/servicemix/components/engines/servicemix-eip/trunk/src/main/java/org/apache/servicemix/eip/support/AbstractAggregator.java?rev=710038&r1=710037&r2=710038&view=diff
==============================================================================
--- servicemix/components/engines/servicemix-eip/trunk/src/main/java/org/apache/servicemix/eip/support/AbstractAggregator.java (original)
+++ servicemix/components/engines/servicemix-eip/trunk/src/main/java/org/apache/servicemix/eip/support/AbstractAggregator.java Mon Nov  3 03:56:18 2008
@@ -142,7 +142,7 @@
     }
 
     /**
-     * Sets wether the aggregator should report errors happening when sending the
+     * Sets whether the aggregator should report errors happening when sending the
      * aggregate on all exchanges that compose the aggregate.
      * The default value is <code>false</code>, meaning that if any error occur, this
      * error will be lost.
@@ -160,7 +160,7 @@
     }
 
     /**
-     * Sets wether the aggregator should report errors on incoming exchanges received after a given
+     * Sets whether the aggregator should report errors on incoming exchanges received after a given
      * aggregate has been closed.
      * The default value is <code>false</code>, meaning that such exchanges will be silently sent back
      * with a DONE status and discarded with respect to the aggregation process.
@@ -240,7 +240,7 @@
                 if (exchanges != null) {
                     for (MessageExchange me : exchanges) {
                         if (exchange.getStatus() == ExchangeStatus.ERROR) {
-                            exchange.setError(me.getError());
+                            me.setError(exchange.getError());
                         }
                         me.setStatus(exchange.getStatus());
                         send(me);

Modified: servicemix/components/engines/servicemix-eip/trunk/src/test/java/org/apache/servicemix/eip/RecipientListAggregatorTest.java
URL: http://svn.apache.org/viewvc/servicemix/components/engines/servicemix-eip/trunk/src/test/java/org/apache/servicemix/eip/RecipientListAggregatorTest.java?rev=710038&r1=710037&r2=710038&view=diff
==============================================================================
--- servicemix/components/engines/servicemix-eip/trunk/src/test/java/org/apache/servicemix/eip/RecipientListAggregatorTest.java (original)
+++ servicemix/components/engines/servicemix-eip/trunk/src/test/java/org/apache/servicemix/eip/RecipientListAggregatorTest.java Mon Nov  3 03:56:18 2008
@@ -54,6 +54,8 @@
         message.setContent(createSource("<hello/>"));
         client.sendSync(me);
         assertEquals(ExchangeStatus.ERROR, me.getStatus());
+        assertNotNull(me.getError());
+        assertEquals("Dummy error", me.getError().getMessage());
 
         listener.assertExchangeCompleted();
     }