Author: gtully
Date: Fri Oct 8 11:00:26 2010
New Revision: 1005773
URL: http://svn.apache.org/viewvc?rev=1005773&view=rev
Log:
resolve: https://issues.apache.org/activemq/browse/AMQ-2966 - related to fix
for: https://issues.apache.org/activemq/browse/AMQ-2103, test via camel
component - simple forwarder using vm transport
Added:
activemq/trunk/activemq-camel/src/test/java/org/apache/activemq/camel/CamelVMTransportRoutingTest.java
(with props)
Modified:
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQTextMessage.java
Added:
activemq/trunk/activemq-camel/src/test/java/org/apache/activemq/camel/CamelVMTransportRoutingTest.java
URL:
http://svn.apache.org/viewvc/activemq/trunk/activemq-camel/src/test/java/org/apache/activemq/camel/CamelVMTransportRoutingTest.java?rev=1005773&view=auto
==============================================================================
---
activemq/trunk/activemq-camel/src/test/java/org/apache/activemq/camel/CamelVMTransportRoutingTest.java
(added)
+++
activemq/trunk/activemq-camel/src/test/java/org/apache/activemq/camel/CamelVMTransportRoutingTest.java
Fri Oct 8 11:00:26 2010
@@ -0,0 +1,155 @@
+/**
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.apache.activemq.camel;
+
+import javax.jms.Connection;
+import javax.jms.Destination;
+import javax.jms.MessageConsumer;
+import javax.jms.MessageProducer;
+import javax.jms.Session;
+import javax.jms.TextMessage;
+
+import junit.framework.TestCase;
+
+import org.apache.activemq.ActiveMQConnectionFactory;
+import org.apache.activemq.broker.BrokerService;
+import org.apache.activemq.broker.TransportConnector;
+import org.apache.activemq.camel.component.ActiveMQComponent;
+import org.apache.activemq.util.ThreadTracker;
+import org.apache.camel.CamelContext;
+import org.apache.camel.builder.RouteBuilder;
+import org.apache.camel.impl.DefaultCamelContext;
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+import org.junit.Assert;
+
+// see: https://issues.apache.org/activemq/browse/AMQ-2966
+public class CamelVMTransportRoutingTest extends TestCase {
+
+ private static final Log log =
LogFactory.getLog(CamelVMTransportRoutingTest.class);
+
+ private BrokerService broker = null;
+ private TransportConnector connector = null;
+ private CamelContext camelContext = null;
+
+ private Connection senderConnection;
+ private Connection receiverConnection1;
+ private Connection receiverConnection2;
+
+ private final String MSG_STRING = "MESSAGE-TEXT";
+ private final String SENDER_TOPIC = "A";
+ private final String RECEIVER_TOPIC = "B";
+
+ public void testSendReceiveWithCamelRouteIntercepting() throws Exception {
+
+ final int MSG_COUNT = 1000;
+
+ Session sendSession = senderConnection.createSession(false,
Session.AUTO_ACKNOWLEDGE);
+ Session receiverSession1 = receiverConnection1.createSession(false,
Session.AUTO_ACKNOWLEDGE);
+ Session receiverSession2 = receiverConnection2.createSession(false,
Session.AUTO_ACKNOWLEDGE);
+
+ Destination sendTo = sendSession.createTopic(SENDER_TOPIC);
+ Destination receiveFrom = receiverSession1.createTopic(RECEIVER_TOPIC);
+
+ TextMessage message = sendSession.createTextMessage(MSG_STRING);
+
+ MessageConsumer receiver1 =
receiverSession1.createConsumer(receiveFrom);
+ MessageConsumer receiver2 =
receiverSession2.createConsumer(receiveFrom);
+
+ MessageProducer sender = sendSession.createProducer(sendTo);
+ for( int i = 0; i < MSG_COUNT; ++i ) {
+ sender.send(message);
+ }
+
+ for( int i = 0; i < MSG_COUNT; ++i ) {
+
+ log.debug("Attempting Received for Message #" + i);
+ TextMessage received1 = (TextMessage) receiver1.receive(5000);
+ Assert.assertNotNull(received1);
+ Assert.assertEquals(MSG_STRING, received1.getText());
+ }
+ }
+
+ protected BrokerService createBroker() throws Exception {
+
+ BrokerService service = new BrokerService();
+ service.setPersistent(false);
+ service.setUseJmx(false);
+ connector = service.addConnector("tcp://localhost:0");
+
+ return service;
+ }
+
+ public void setUp() throws Exception {
+
+ broker = createBroker();
+ broker.start();
+ broker.waitUntilStarted();
+
+ Thread.sleep(1000);
+
+ createCamelContext();
+
+ ActiveMQConnectionFactory connFactory = new
ActiveMQConnectionFactory(connector.getConnectUri());
+ senderConnection = connFactory.createConnection();
+ receiverConnection1 = connFactory.createConnection();
+ receiverConnection2 = connFactory.createConnection();
+
+ receiverConnection1.start();
+ receiverConnection2.start();
+ }
+
+ public void tearDown() throws Exception {
+
+ if( senderConnection != null ) {
+ senderConnection.close();
+ }
+
+ if( receiverConnection1 != null ) {
+ receiverConnection1.close();
+ }
+
+ if( receiverConnection2 != null ) {
+ receiverConnection2.close();
+ }
+
+ camelContext.stop();
+ broker.stop();
+
+ ThreadTracker.result();
+ }
+
+ private void createCamelContext() throws Exception {
+
+ final String fromEndpoint = "activemq:topic:" + SENDER_TOPIC;
+ final String toEndpoint = "activemq:topic:" + RECEIVER_TOPIC;
+
+ log.info("creating context and sending message");
+ camelContext = new DefaultCamelContext();
+ camelContext.addComponent("activemq",
+
ActiveMQComponent.activeMQComponent("vm://localhost?create=false&waitForStart=10000"));
+ camelContext.addRoutes(new RouteBuilder() {
+ @Override
+ public void configure() throws Exception {
+ from(fromEndpoint).to(toEndpoint);
+ }
+ });
+ camelContext.start();
+ }
+
+}
Propchange:
activemq/trunk/activemq-camel/src/test/java/org/apache/activemq/camel/CamelVMTransportRoutingTest.java
------------------------------------------------------------------------------
svn:eol-style = native
Propchange:
activemq/trunk/activemq-camel/src/test/java/org/apache/activemq/camel/CamelVMTransportRoutingTest.java
------------------------------------------------------------------------------
svn:keywords = Rev Date
Modified:
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQTextMessage.java
URL:
http://svn.apache.org/viewvc/activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQTextMessage.java?rev=1005773&r1=1005772&r2=1005773&view=diff
==============================================================================
---
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQTextMessage.java
(original)
+++
activemq/trunk/activemq-core/src/main/java/org/apache/activemq/command/ActiveMQTextMessage.java
Fri Oct 8 11:00:26 2010
@@ -118,11 +118,17 @@ public class ActiveMQTextMessage extends
MarshallingSupport.writeUTF8(dataOut, this.text);
dataOut.close();
setContent(bytesOut.toByteSequence());
- //see https://issues.apache.org/activemq/browse/AMQ-2103
- this.text=null;
}
}
+ @Override
+ public void afterMarshall(WireFormat wireFormat) throws IOException {
+ super.afterMarshall(wireFormat);
+ //see https://issues.apache.org/activemq/browse/AMQ-2103
+ // and https://issues.apache.org/activemq/browse/AMQ-2966
+ this.text=null;
+ }
+
/**
* Clears out the message body. Clearing a message's body does not clear
its
* header values or property entries. <p/>