[AMQ-6809] activemq-client returns unexpected length code on first read from stream containing empty byte array

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 5.14.5
• Fix Version/s: 5.15.1, 5.16.0
• Component/s: JMS client
• Assignee: Timothy A. Bish
• Reporter: Jiri Daněk
• Created: 15/Sep/17 08:06
• Updated: 20/Sep/17 19:51
• Resolved: 20/Sep/17 19:51

Description

Consider this code

	private void testEmptyBufferInput() throws JMSException {
		byte[] readList = new byte[BYTE_LIST.length - 1];
		byte[] emptyList = {};
		LOG.info("Streaming BYTE_LIST message testing EMPTY buffer input");
		stream.writeBytes(emptyList);
		stream.reset();
		LOG.info("Reading BYTE_LIST message testing EMPTY buffer input");
		final int IS_EMPTY = 0;
		assertThat(stream.readBytes(readList)).isEqualTo(IS_EMPTY);
	}

The code above works with qpid-jms, but fails with activemq-client on the assertThat with error message java.lang.AssertionError: Not true that <-1> is equal to <0>. Judging by documentation http://docs.oracle.com/javaee/6/api/javax/jms/StreamMessage.html#readBytes(byte[]), I think that qpid-jms behavior is correct and what activemq-client does is a bug.

Issue Links

• is related to: QPIDJMS-325 reading from empty buffer of a BytesMessage should return 0, not -1