Alaske commented on PR #10548: URL: https://github.com/apache/rocketmq/pull/10548#issuecomment-4829320038
The transaction producer test setup now uses a dedicated MQClientInstance instead of the shared test-level client factory. When the full client test suite runs, other DefaultMQProducerTest cases can mutate or stub the shared MQClientInstance state. The new transactional selector test depends on broker lookup through findBrokerAddressInPublish(...) after the selector chooses a queue, so sharing that client factory made the test order-sensitive. To avoid cross-test pollution, prepareTransactionProducer(...) now creates an isolated MQClientInstance spy for the transaction producer, injects the mocked MQClientAPIImpl into it, and stubs only the broker lookup required by this test path. This keeps the transactional send selector test deterministic in both focused runs and the full Maven package run. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
