Skip to content

Commit

Permalink
ARTEMIS-3698 fix test
Browse files Browse the repository at this point in the history
When copying message properties from the core message to the OpenWire
message we intentially omit any properties starting with `_AMQ` and
`__HDR_`. However, we were effectively negating that logic because we
copied the marshalled properties directly to the message without any
filtering. Now that we no longer copy the marshalled properties directly
to the message the test breaks because it expects properties starting
with `__HDR_`. This commit fixes the test by removing those
expectations. The test is still valid because the message is still
receieved rather than being swallowed due to an exception (which was the
original problem).
  • Loading branch information
jbertram committed Mar 4, 2022
1 parent e949e38 commit f6372d8
Showing 1 changed file with 0 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1810,9 +1810,6 @@ public void testPropertyConversions() throws Exception {
connection.start();
TextMessage messageReceived = (TextMessage) messageConsumer.receive(5000);
assertNotNull(messageReceived);
assertEquals(BROKER_PATH, messageReceived.getStringProperty("__HDR_BROKER_PATH"));
assertEquals(CLUSTER, messageReceived.getStringProperty("__HDR_CLUSTER"));
assertEquals(USER_ID, messageReceived.getStringProperty("__HDR_USER_ID"));
}
}

Expand Down

0 comments on commit f6372d8

Please sign in to comment.