git.net

[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:

    https://github.com/apache/activemq-artemis/pull/2174#discussion_r201156531
  
    --- 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
           assertNotNull(amqp.getBody());
           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 --
    
    Reviewing the changes and tests and all looks good, the unexpected checks for content length in the object message bodies seems to be a side effect of the way the test constructs the message and not related to any new bugs.  


---