diff options
author | Martin Ritchie <ritchiem@apache.org> | 2008-06-03 12:32:47 +0000 |
---|---|---|
committer | Martin Ritchie <ritchiem@apache.org> | 2008-06-03 12:32:47 +0000 |
commit | 7d0c48f274b7bc1808f9afe0ec36ae37dc2dea72 (patch) | |
tree | 975fb89cc5ec6d79a1ccc27136d1ae1bb5d848f3 | |
parent | cd3f780556ff80df228f83945d2d1e76ea4190e1 (diff) | |
download | qpid-python-7d0c48f274b7bc1808f9afe0ec36ae37dc2dea72.tar.gz |
QPID-1092 : Changed toString to be String.valueOf(getObject())
Added MessageToStringTest, tests performing toString on Message before calling getObject().
git-svn-id: https://svn.apache.org/repos/asf/incubator/qpid/branches/M2.x@662770 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r-- | java/client/src/main/java/org/apache/qpid/client/message/JMSObjectMessage.java | 24 | ||||
-rw-r--r-- | java/systests/src/main/java/org/apache/qpid/test/client/message/MessageToStringTest.java | 101 |
2 files changed, 102 insertions, 23 deletions
diff --git a/java/client/src/main/java/org/apache/qpid/client/message/JMSObjectMessage.java b/java/client/src/main/java/org/apache/qpid/client/message/JMSObjectMessage.java index caf8741280..14cf949110 100644 --- a/java/client/src/main/java/org/apache/qpid/client/message/JMSObjectMessage.java +++ b/java/client/src/main/java/org/apache/qpid/client/message/JMSObjectMessage.java @@ -88,7 +88,7 @@ public class JMSObjectMessage extends AbstractJMSMessage implements ObjectMessag public String toBodyString() throws JMSException { - return toString(_data); + return String.valueOf(getObject()); } public AMQShortString getMimeTypeAsShortString() @@ -172,26 +172,4 @@ public class JMSObjectMessage extends AbstractJMSMessage implements ObjectMessag catch (IOException ignore) { } } - - private static String toString(ByteBuffer data) - { - if (data == null) - { - return null; - } - - int pos = data.position(); - try - { - return data.getString(Charset.forName("UTF8").newDecoder()); - } - catch (CharacterCodingException e) - { - return null; - } - finally - { - data.position(pos); - } - } } diff --git a/java/systests/src/main/java/org/apache/qpid/test/client/message/MessageToStringTest.java b/java/systests/src/main/java/org/apache/qpid/test/client/message/MessageToStringTest.java new file mode 100644 index 0000000000..b3a9ee69b8 --- /dev/null +++ b/java/systests/src/main/java/org/apache/qpid/test/client/message/MessageToStringTest.java @@ -0,0 +1,101 @@ +/* + * + * 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.qpid.test.client.message; + +import org.apache.qpid.test.VMTestCase; + +import javax.jms.BytesMessage; +import javax.jms.Connection; +import javax.jms.ConnectionFactory; +import javax.jms.JMSException; +import javax.jms.Message; +import javax.jms.MessageConsumer; +import javax.jms.MessageProducer; +import javax.jms.ObjectMessage; +import javax.jms.Queue; +import javax.jms.Session; +import java.util.UUID; + +public class MessageToStringTest extends VMTestCase +{ + private Connection _connection; + private Session _session; + private Queue _queue; + MessageConsumer _consumer; + + public void setUp() throws Exception + { + super.setUp(); + + //Create Producer put some messages on the queue + _connection = ((ConnectionFactory) _context.lookup("connection")).createConnection(); + + //Create Queue + _queue = (Queue) _context.lookup("queue"); + + //Create Consumer + _session = _connection.createSession(false, Session.AUTO_ACKNOWLEDGE); + + _consumer = _session.createConsumer(_queue); + + _connection.start(); + } + + public void tearDown() throws Exception + { + //clean up + _connection.close(); + + super.tearDown(); + } + + public void testObjectMessage() throws JMSException + { + MessageProducer producer = _session.createProducer(_queue); + + //Create Sample Message using UUIDs + UUID test = UUID.randomUUID(); + + Message testMessage = _session.createObjectMessage(test); + + producer.send(testMessage); + + Message receivedMessage = _consumer.receive(1000); + + assertNotNull("Message was not received.", receivedMessage); + + assertNotNull("Message returned null from toString", receivedMessage.toString()); + + UUID result = null; + + try + { + result = (UUID) ((ObjectMessage) receivedMessage).getObject(); + } + catch (Exception e) + { + fail(e.getMessage()); + } + + assertEquals("UUIDs were not equal", test, result); + } + +} |