diff options
author | Rajith Muditha Attapattu <rajith@apache.org> | 2013-03-13 15:39:24 +0000 |
---|---|---|
committer | Rajith Muditha Attapattu <rajith@apache.org> | 2013-03-13 15:39:24 +0000 |
commit | a5fe06779d2c196ea954fc91a79dcba18203e38e (patch) | |
tree | c9f6e4b4b2fae2a684082c2976b78a50296cf393 | |
parent | 24a380383777903d18ae5d6d535bc0dda23f05f6 (diff) | |
download | qpid-python-a5fe06779d2c196ea954fc91a79dcba18203e38e.tar.gz |
QPID-3769 Modified the hashcode impl to match equals. Added a test case
to verify equals and hashcode for ADDR based destinations.
git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@1456008 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r-- | qpid/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java | 16 | ||||
-rw-r--r-- | qpid/java/client/src/test/java/org/apache/qpid/client/AMQDestinationTest.java | 46 |
2 files changed, 57 insertions, 5 deletions
diff --git a/qpid/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java b/qpid/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java index 58f7a465be..6ca2988186 100644 --- a/qpid/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java +++ b/qpid/java/client/src/main/java/org/apache/qpid/client/AMQDestination.java @@ -676,13 +676,19 @@ public abstract class AMQDestination implements Destination, Referenceable public int hashCode() { int result; - result = _exchangeName == null ? "".hashCode() : _exchangeName.hashCode(); - result = 29 * result + (_exchangeClass == null ? "".hashCode() :_exchangeClass.hashCode()); - if (_queueName != null) + if (_destSyntax == DestSyntax.ADDR) { - result = 29 * result + _queueName.hashCode(); + result = 29 * _addressType + _name.hashCode(); + } + else + { + result = _exchangeName == null ? "".hashCode() : _exchangeName.hashCode(); + result = 29 * result + (_exchangeClass == null ? "".hashCode() :_exchangeClass.hashCode()); + if (_queueName != null) + { + result = 29 * result + _queueName.hashCode(); + } } - return result; } diff --git a/qpid/java/client/src/test/java/org/apache/qpid/client/AMQDestinationTest.java b/qpid/java/client/src/test/java/org/apache/qpid/client/AMQDestinationTest.java new file mode 100644 index 0000000000..1a411c99bc --- /dev/null +++ b/qpid/java/client/src/test/java/org/apache/qpid/client/AMQDestinationTest.java @@ -0,0 +1,46 @@ +/* + * + * 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.client; + +import junit.framework.TestCase; + +public class AMQDestinationTest extends TestCase +{ + public void testEqaulsAndHashCodeForAddressBasedDestinations() throws Exception + { + AMQDestination dest = new AMQQueue("ADDR:Foo; {node :{type:queue}}"); + AMQDestination dest1 = new AMQQueue("ADDR:Foo; {node :{type:topic}}"); + AMQDestination dest2 = new AMQQueue( + "ADDR:Foo; {create:always,node :{type:queue}}"); + String bUrl = "direct://amq.direct/test-route/Foo?routingkey='Foo'"; + AMQDestination dest3 = new AMQQueue(bUrl); + + assertTrue(dest.equals(dest)); + assertFalse(dest.equals(dest1)); + assertTrue(dest.equals(dest2)); + assertFalse(dest.equals(dest3)); + + assertTrue(dest.hashCode() == dest.hashCode()); + assertTrue(dest.hashCode() != dest1.hashCode()); + assertTrue(dest.hashCode() == dest2.hashCode()); + assertTrue(dest.hashCode() != dest3.hashCode()); + } +} |