[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[GitHub] activemq-artemis pull request #2174: ARTEMIS-1941 fix failing tests

Github user tabish121 commented on a diff in the pull request:

    --- Diff: artemis-protocols/artemis-amqp-protocol/src/test/java/org/apache/activemq/artemis/protocol/amqp/converter/message/JMSMappingOutboundTransformerTest.java ---
    @@ -361,7 +361,7 @@ public void testConvertEmptyObjectMessageToAmqpMessageWithAmqpValueBody() throws
           assertTrue(amqp.getBody() instanceof AmqpValue);
           assertTrue(((AmqpValue) amqp.getBody()).getValue() instanceof Binary);
    -      assertEquals(0, ((Binary) ((AmqpValue) amqp.getBody()).getValue()).getLength());
    +      assertEquals(5, ((Binary) ((AmqpValue) amqp.getBody()).getValue()).getLength());
    --- End diff --
    At first glance this looks wrong as I'd have expected the sender to have sent a zero length Binary and so the resulting message should also contain a zero length Binary as the value in the encoded AmqpValue.