[fix][test] Fix flaky test: PersistentStreamingDispatcherBlockConsumerTest.testBrokerDispatchBlockAndSubAckBackRequiredMsgs

[nicoloboschi]

Motivation

java.lang.AssertionError: expected [600] but found [265]
	at org.testng.Assert.fail(Assert.java:99)
	at org.testng.Assert.failNotEquals(Assert.java:1037)
	at org.testng.Assert.assertEqualsImpl(Assert.java:140)
	at org.testng.Assert.assertEquals(Assert.java:122)
	at org.testng.Assert.assertEquals(Assert.java:907)
	at org.testng.Assert.assertEquals(Assert.java:917)
	at org.apache.pulsar.client.api.DispatcherBlockConsumerTest.testBrokerDispatchBlockAndSubAckBackRequiredMsgs(DispatcherBlockConsumerTest.java:1036)

Modifications

In the scope of this test is important to not group acks because the ack for messages may happens after receiving a new message from the topic. If the acks are sent in batch it may happen that the dispatcher is still blocked until it receives the batched acks and the Consumer#receive will return null and the test will fail.

• Set acknowledgmentGroupTime to 0 for the consumer

• Moved the test to the broker suite (it was flaky)

• doc-not-needed

[eolivelli]

I agree with the patch.
But I suggested an alternative fix PTAL

[nicoloboschi]

@eolivelli see my response #17161 (comment)