Skip to content

Commit 8771f8e

Browse files
garyrussellartembilan
authored andcommitted
Fix tests for latest SK snapshot
1 parent 04019db commit 8771f8e

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

spring-integration-kafka/src/test/java/org/springframework/integration/kafka/outbound/KafkaProducerMessageHandlerTests.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@
8080
import org.springframework.kafka.core.KafkaTemplate;
8181
import org.springframework.kafka.core.ProducerFactory;
8282
import org.springframework.kafka.listener.ContainerProperties;
83+
import org.springframework.kafka.listener.ContainerProperties.AssignmentCommitOption;
8384
import org.springframework.kafka.listener.KafkaMessageListenerContainer;
8485
import org.springframework.kafka.requestreply.ReplyingKafkaTemplate;
8586
import org.springframework.kafka.support.DefaultKafkaHeaderMapper;
@@ -503,12 +504,14 @@ void testConsumeAndProduceTransaction() throws Exception {
503504
ConsumerFactory cf = mock(ConsumerFactory.class);
504505
willReturn(mockConsumer).given(cf).createConsumer("group", "", null, KafkaTestUtils.defaultPropertyOverrides());
505506
Producer producer = mock(Producer.class);
507+
given(producer.send(any(), any())).willReturn(new SettableListenableFuture<>());
506508
final CountDownLatch closeLatch = new CountDownLatch(2);
507509
willAnswer(i -> {
508510
closeLatch.countDown();
509511
return null;
510512
}).given(producer).close(any());
511513
ProducerFactory pf = mock(ProducerFactory.class);
514+
given(pf.isProducerPerConsumerPartition()).willReturn(true);
512515
given(pf.transactionCapable()).willReturn(true);
513516
final List<String> transactionalIds = new ArrayList<>();
514517
willAnswer(i -> {
@@ -519,7 +522,7 @@ void testConsumeAndProduceTransaction() throws Exception {
519522
ContainerProperties props = new ContainerProperties("foo");
520523
props.setGroupId("group");
521524
props.setTransactionManager(ptm);
522-
props.setMissingTopicsFatal(false);
525+
props.setAssignmentCommitOption(AssignmentCommitOption.ALWAYS);
523526
final KafkaTemplate template = new KafkaTemplate(pf);
524527
KafkaMessageListenerContainer container = new KafkaMessageListenerContainer<>(cf, props);
525528
container.setBeanName("commit");
@@ -613,6 +616,7 @@ void testConsumeAndProduceTransactionTxIdOverride() throws Exception {
613616
ConsumerFactory cf = mock(ConsumerFactory.class);
614617
willReturn(mockConsumer).given(cf).createConsumer("group", "", null, KafkaTestUtils.defaultPropertyOverrides());
615618
Producer producer = mock(Producer.class);
619+
given(producer.send(any(), any())).willReturn(new SettableListenableFuture<>());
616620
final CountDownLatch closeLatch = new CountDownLatch(2);
617621
willAnswer(i -> {
618622
closeLatch.countDown();
@@ -637,7 +641,7 @@ protected Producer createTransactionalProducerForPartition(String txIdPrefix) {
637641
ContainerProperties props = new ContainerProperties("foo");
638642
props.setGroupId("group");
639643
props.setTransactionManager(tm);
640-
props.setMissingTopicsFatal(false);
644+
props.setAssignmentCommitOption(AssignmentCommitOption.ALWAYS);
641645
final KafkaTemplate template = new KafkaTemplate(pf);
642646
template.setTransactionIdPrefix("template.tx.id.");
643647
KafkaMessageListenerContainer container = new KafkaMessageListenerContainer<>(cf, props);

0 commit comments

Comments
 (0)