X-Git-Url: http://juplo.de/gitweb/?a=blobdiff_plain;f=src%2Ftest%2Fjava%2Fde%2Fjuplo%2Fkafka%2FApplicationTests.java;h=c3fbe45724281fb38ab05cbc4de87b8b226472ac;hb=bf6e53f9941724fffeb9e254937bd261c028f666;hp=8a485b33dd01402b83dbd72c7ca0fdb3f260dbe6;hpb=9fe455e4ccd0afdad0a9f12c009aa42151c2d098;p=demos%2Fkafka%2Ftraining diff --git a/src/test/java/de/juplo/kafka/ApplicationTests.java b/src/test/java/de/juplo/kafka/ApplicationTests.java index 8a485b3..c3fbe45 100644 --- a/src/test/java/de/juplo/kafka/ApplicationTests.java +++ b/src/test/java/de/juplo/kafka/ApplicationTests.java @@ -1,7 +1,7 @@ package de.juplo.kafka; +import lombok.Value; import lombok.extern.slf4j.Slf4j; -import org.apache.kafka.clients.consumer.ConsumerRecord; import org.apache.kafka.clients.consumer.KafkaConsumer; import org.apache.kafka.clients.producer.KafkaProducer; import org.apache.kafka.clients.producer.ProducerRecord; @@ -17,17 +17,17 @@ import org.springframework.boot.test.context.TestConfiguration; import org.springframework.context.annotation.Bean; import org.springframework.context.annotation.Import; import org.springframework.context.annotation.Primary; +import org.springframework.kafka.listener.adapter.ConsumerRecordMetadata; import org.springframework.kafka.support.serializer.JsonSerializer; import org.springframework.kafka.test.context.EmbeddedKafka; import org.springframework.test.context.TestPropertySource; import org.springframework.test.context.junit.jupiter.SpringJUnitConfig; import java.time.Duration; +import java.time.LocalDateTime; import java.util.*; -import java.util.concurrent.ExecutionException; import java.util.function.BiConsumer; import java.util.function.BiFunction; -import java.util.function.Consumer; import java.util.stream.Collectors; import java.util.stream.IntStream; @@ -56,8 +56,6 @@ class ApplicationTests public static final int PARTITIONS = 10; - StringSerializer stringSerializer = new StringSerializer(); - @Autowired Serializer valueSerializer; @Autowired @@ -71,23 +69,51 @@ class ApplicationTests @Autowired EndlessConsumer endlessConsumer; @Autowired - RecordHandler recordHandler; + MessageHandler clientMessageHandler; + @Autowired + MessageHandler greetingsHandler; Map oldOffsets; Map newOffsets; - Set> receivedRecords; + Set received; /** Tests methods */ @Test - void commitsCurrentOffsetsOnSuccess() throws ExecutionException, InterruptedException + void commitsCurrentOffsetsOnSuccess() { - send100Messages((key, counter) -> serialize(key, counter)); + send100Messages((key, counter) -> serializeAsClientMessage(key, counter)); + + await("100 records received") + .atMost(Duration.ofSeconds(30)) + .until(() -> received.size() == 100); + + await("Offsets committed") + .atMost(Duration.ofSeconds(10)) + .untilAsserted(() -> + { + checkSeenOffsetsForProgress(); + compareToCommitedOffsets(newOffsets); + }); + + assertThat(endlessConsumer.isRunning()) + .describedAs("Consumer should still be running") + .isTrue(); + } + + + @Test + void mixedMessages() + { + send100Messages((key, counter) -> + counter%3 == 0 + ? serializeAsGreeting(key) + : serializeAsClientMessage(key, counter)); await("100 records received") .atMost(Duration.ofSeconds(30)) - .until(() -> receivedRecords.size() == 100); + .until(() -> received.size() == 100); await("Offsets committed") .atMost(Duration.ofSeconds(10)) @@ -107,12 +133,12 @@ class ApplicationTests { send100Messages((key, counter) -> counter == 77 - ? new Bytes(stringSerializer.serialize(TOPIC, "BOOM!")) - : serialize(key, counter)); + ? serializeString("BOOM!", "message") + : serializeAsClientMessage(key, counter)); await("99 records received") .atMost(Duration.ofSeconds(30)) - .until(() -> receivedRecords.size() == 99); + .until(() -> received.size() == 99); await("Offsets committed") .atMost(Duration.ofSeconds(10)) @@ -137,17 +163,17 @@ class ApplicationTests @Test void commitsOffsetOnProgramLogicErrorFoo() { - recordHandler.testHandler = (record) -> + clientMessageHandler.testHandler = (clientMessage, metadata) -> { - if (Integer.parseInt(record.value().message)%10 ==0) - throw new RuntimeException("BOOM: " + record.value().message + "%10 == 0"); + if (Integer.parseInt(clientMessage.message)%10 ==0) + throw new RuntimeException("BOOM: " + clientMessage.message + "%10 == 0"); }; - send100Messages((key, counter) -> serialize(key, counter)); + send100Messages((key, counter) -> serializeAsClientMessage(key, counter)); await("80 records received") .atMost(Duration.ofSeconds(30)) - .until(() -> receivedRecords.size() == 100); + .until(() -> received.size() == 100); await("Offsets committed") .atMost(Duration.ofSeconds(10)) @@ -218,7 +244,7 @@ class ApplicationTests } - void send100Messages(BiFunction messageGenerator) + void send100Messages(BiFunction recordGenerator) { long i = 0; @@ -226,16 +252,16 @@ class ApplicationTests { for (int key = 0; key < 10; key++) { - Bytes value = messageGenerator.apply(key, ++i); + BytesAndType bat = recordGenerator.apply(key, ++i); ProducerRecord record = new ProducerRecord<>( TOPIC, partition, Integer.toString(key%2), - value); + bat.getValue()); - record.headers().add("__TypeId__", "message".getBytes()); + record.headers().add("__TypeId__", bat.getType()); kafkaProducer.send(record, (metadata, e) -> { if (metadata != null) @@ -260,11 +286,29 @@ class ApplicationTests } } - Bytes serialize(Integer key, Long value) + BytesAndType serializeAsClientMessage(Integer key, Long value) { ClientMessage message = new ClientMessage(); message.setClient(key.toString()); message.setMessage(value.toString()); + return new BytesAndType(serialize(message), "message"); + } + + BytesAndType serializeAsGreeting(Integer key) + { + Greeting greeting = new Greeting(); + greeting.setName(key.toString()); + greeting.setWhen(LocalDateTime.now()); + return new BytesAndType(serialize(greeting), "greeting"); + } + + BytesAndType serializeString(String message, String messageType) + { + return new BytesAndType(new Bytes(message.getBytes()), messageType); + } + + Bytes serialize(Object message) + { return new Bytes(valueSerializer.serialize(TOPIC, message)); } @@ -272,11 +316,12 @@ class ApplicationTests @BeforeEach public void init() { - recordHandler.testHandler = (record) -> {}; + clientMessageHandler.testHandler = (clientMessage, metadata) -> {}; + greetingsHandler.testHandler = (greeting, metadata) -> {}; oldOffsets = new HashMap<>(); newOffsets = new HashMap<>(); - receivedRecords = new HashSet<>(); + received = new HashSet<>(); doForCurrentOffsets((tp, offset) -> { @@ -284,16 +329,20 @@ class ApplicationTests newOffsets.put(tp, offset - 1); }); - recordHandler.captureOffsets = - record -> + BiConsumer captureOffsets = + (clientMessage, metadata) -> { - receivedRecords.add(record); - log.debug("TEST: Processing record #{}: {}", receivedRecords.size(), record.value()); + received.add(clientMessage); + log.debug("TEST: Processing record #{}: {}", received.size(), clientMessage); newOffsets.put( - new TopicPartition(record.topic(), record.partition()), - record.offset()); + new TopicPartition(metadata.topic(), metadata.partition()), metadata.offset()); }; + clientMessageHandler.captureOffsets = + (BiConsumer)captureOffsets; + greetingsHandler.captureOffsets = + (BiConsumer)captureOffsets; + endlessConsumer.start(); } @@ -310,18 +359,18 @@ class ApplicationTests } } - public static class RecordHandler implements Consumer> + public static class MessageHandler implements BiConsumer { - Consumer> captureOffsets; - Consumer> testHandler; + BiConsumer captureOffsets; + BiConsumer testHandler; @Override - public void accept(ConsumerRecord record) + public void accept(T message, ConsumerRecordMetadata metadata) { captureOffsets .andThen(testHandler) - .accept(record); + .accept(message, metadata); } } @@ -331,9 +380,16 @@ class ApplicationTests { @Primary @Bean - public Consumer> testHandler() + public MessageHandler messageHandler() { - return new RecordHandler(); + return new MessageHandler<>(); + } + + @Primary + @Bean + public MessageHandler greetingsHandler() + { + return new MessageHandler<>(); } @Bean @@ -367,4 +423,19 @@ class ApplicationTests return new KafkaConsumer<>(props); } } + + + @Value + static class BytesAndType + { + private final Bytes value; + private final byte[] type; + + + BytesAndType(Bytes value, String type) + { + this.value = value; + this.type = type.getBytes(); + } + } }