hachikuji commented on a change in pull request #8782:
URL: https://github.com/apache/kafka/pull/8782#discussion_r434153210
##########
File path:
core/src/test/scala/unit/kafka/coordinator/transaction/TransactionMarkerChannelManagerTest.scala
##########
@@ -221,7 +221,6 @@ class TransactionMarkerChannelManagerTest {
EasyMock.replay(metadataCache)
channelManager.addTxnMarkersToSend(coordinatorEpoch, txnResult,
txnMetadata1, txnMetadata1.prepareComplete(time.milliseconds()))
- channelManager.addTxnMarkersToSend(coordinatorEpoch, txnResult,
txnMetadata2, txnMetadata2.prepareComplete(time.milliseconds()))
Review comment:
The change to use `mock` instead of `niceMock` led to a failure because
of an unexpected append to the log. It seemed like this call was not necessary
to test the behavior we were interested in here, so I removed it rather than
adding the expected call to append.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
