diff --git a/spring-integration-amqp/src/main/java/org/springframework/integration/amqp/channel/PollableAmqpChannel.java b/spring-integration-amqp/src/main/java/org/springframework/integration/amqp/channel/PollableAmqpChannel.java index 6b69fa98c1..1324bd7e2e 100644 --- a/spring-integration-amqp/src/main/java/org/springframework/integration/amqp/channel/PollableAmqpChannel.java +++ b/spring-integration-amqp/src/main/java/org/springframework/integration/amqp/channel/PollableAmqpChannel.java @@ -80,7 +80,7 @@ public class PollableAmqpChannel extends AbstractAmqpChannel */ public PollableAmqpChannel(String channelName, AmqpTemplate amqpTemplate, AmqpHeaderMapper outboundMapper, AmqpHeaderMapper inboundMapper) { - super(amqpTemplate, inboundMapper, outboundMapper); + super(amqpTemplate, outboundMapper, inboundMapper); Assert.hasText(channelName, "channel name must not be empty"); this.channelName = channelName; } diff --git a/spring-integration-amqp/src/test/java/org/springframework/integration/amqp/channel/ChannelTests-context.xml b/spring-integration-amqp/src/test/java/org/springframework/integration/amqp/channel/ChannelTests-context.xml index bea75823e0..42804f08de 100644 --- a/spring-integration-amqp/src/test/java/org/springframework/integration/amqp/channel/ChannelTests-context.xml +++ b/spring-integration-amqp/src/test/java/org/springframework/integration/amqp/channel/ChannelTests-context.xml @@ -15,7 +15,8 @@ - + @@ -29,4 +30,10 @@ + + + + diff --git a/spring-integration-amqp/src/test/java/org/springframework/integration/amqp/channel/ChannelTests.java b/spring-integration-amqp/src/test/java/org/springframework/integration/amqp/channel/ChannelTests.java index d4a496de78..3ce0455b5a 100644 --- a/spring-integration-amqp/src/test/java/org/springframework/integration/amqp/channel/ChannelTests.java +++ b/spring-integration-amqp/src/test/java/org/springframework/integration/amqp/channel/ChannelTests.java @@ -20,6 +20,7 @@ import static org.hamcrest.Matchers.equalTo; import static org.hamcrest.Matchers.instanceOf; import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertNotNull; +import static org.junit.Assert.assertSame; import static org.junit.Assert.assertThat; import static org.junit.Assert.assertTrue; import static org.mockito.Mockito.mock; @@ -44,6 +45,7 @@ import org.springframework.beans.factory.BeanFactory; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.integration.amqp.config.AmqpChannelFactoryBean; import org.springframework.integration.amqp.rule.BrokerRunning; +import org.springframework.integration.amqp.support.AmqpHeaderMapper; import org.springframework.integration.support.MessageBuilder; import org.springframework.integration.test.support.LogAdjustingTestSupport; import org.springframework.integration.test.util.TestUtils; @@ -90,6 +92,12 @@ public class ChannelTests extends LogAdjustingTestSupport { @Autowired private CachingConnectionFactory factory; + @Autowired + private AmqpHeaderMapper mapperIn; + + @Autowired + private AmqpHeaderMapper mapperOut; + public ChannelTests() { super("org.springframework.integration", "org.springframework.integration.amqp", "org.springframework.amqp"); } @@ -224,6 +232,9 @@ public class ChannelTests extends LogAdjustingTestSupport { assertNotNull(received); assertThat((Foo) received.getPayload(), equalTo(foo)); assertThat((String) received.getHeaders().get("baz"), equalTo("qux")); + + assertSame(this.mapperIn, TestUtils.getPropertyValue(this.pollableWithEP, "inboundHeaderMapper")); + assertSame(this.mapperOut, TestUtils.getPropertyValue(this.pollableWithEP, "outboundHeaderMapper")); } public static class Foo {