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.concurrent.ExecutorService;
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;
@TestPropertySource(
properties = {
"consumer.bootstrap-server=${spring.embedded.kafka.brokers}",
- "consumer.topic=" + TOPIC })
+ "consumer.topic=" + TOPIC,
+ "consumer.commit-interval=1s" })
@EmbeddedKafka(topics = TOPIC, partitions = PARTITIONS)
@Slf4j
class ApplicationTests
@Autowired
KafkaProducer<String, Bytes> kafkaProducer;
@Autowired
- KafkaConsumer<String, ClientMessage> kafkaConsumer;
+ KafkaConsumer<String, ValidMessage> kafkaConsumer;
@Autowired
KafkaConsumer<Bytes, Bytes> offsetConsumer;
@Autowired
@Autowired
ExecutorService executor;
- Consumer<ConsumerRecord<String, ClientMessage>> testHandler;
- EndlessConsumer<String, ClientMessage> endlessConsumer;
+ Consumer<ConsumerRecord<String, ValidMessage>> testHandler;
+ EndlessConsumer<String, ValidMessage> endlessConsumer;
Map<TopicPartition, Long> oldOffsets;
Map<TopicPartition, Long> newOffsets;
- Set<ConsumerRecord<String, ClientMessage>> receivedRecords;
+ Set<ConsumerRecord<String, ValidMessage>> receivedRecords;
/** Tests methods */
@Test
- @Order(1) // << The poistion pill is not skipped. Hence, this test must run first
- void commitsCurrentOffsetsOnSuccess() throws ExecutionException, InterruptedException
+ void commitsCurrentOffsetsOnSuccess()
{
- send100Messages((key, counter) -> serialize(key, counter));
+ send100Messages((partition, key, counter) ->
+ {
+ Bytes value;
+ String type;
+
+ if (counter%3 != 0)
+ {
+ value = serializeClientMessage(key, counter);
+ type = "message";
+ }
+ else {
+ value = serializeGreeting(key);
+ type = "greeting";
+ }
+
+ return toRecord(partition, key, value, Optional.of(type));
+ });
await("100 records received")
.atMost(Duration.ofSeconds(30))
+ .pollInterval(Duration.ofSeconds(1))
.until(() -> receivedRecords.size() >= 100);
await("Offsets committed")
.atMost(Duration.ofSeconds(10))
+ .pollInterval(Duration.ofSeconds(1))
.untilAsserted(() ->
{
checkSeenOffsetsForProgress();
}
@Test
- @Order(2)
- void commitsOffsetOfErrorForReprocessingOnError()
+ void commitsOffsetOfErrorForReprocessingOnDeserializationErrorInvalidMessage()
+ {
+ send100Messages((partition, key, counter) ->
+ {
+ Bytes value;
+ String type;
+
+ if (counter == 77)
+ {
+ value = serializeFooMessage(key, counter);
+ type = null;
+ }
+ else
+ {
+ if (counter%3 != 0)
+ {
+ value = serializeClientMessage(key, counter);
+ type = "message";
+ }
+ else {
+ value = serializeGreeting(key);
+ type = "greeting";
+ }
+ }
+
+ return toRecord(partition, key, value, Optional.ofNullable(type));
+ });
+
+ await("Consumer failed")
+ .atMost(Duration.ofSeconds(30))
+ .pollInterval(Duration.ofSeconds(1))
+ .until(() -> !endlessConsumer.running());
+
+ checkSeenOffsetsForProgress();
+ compareToCommitedOffsets(newOffsets);
+
+ endlessConsumer.start();
+ await("Consumer failed")
+ .atMost(Duration.ofSeconds(30))
+ .pollInterval(Duration.ofSeconds(1))
+ .until(() -> !endlessConsumer.running());
+
+ checkSeenOffsetsForProgress();
+ compareToCommitedOffsets(newOffsets);
+ assertThat(receivedRecords.size())
+ .describedAs("Received not all sent events")
+ .isLessThan(100);
+
+ assertThatNoException()
+ .describedAs("Consumer should not be running")
+ .isThrownBy(() -> endlessConsumer.exitStatus());
+ assertThat(endlessConsumer.exitStatus())
+ .describedAs("Consumer should have exited abnormally")
+ .containsInstanceOf(RecordDeserializationException.class);
+ }
+
+ @Test
+ void commitsOffsetOfErrorForReprocessingOnDeserializationErrorOnUnknownMessage()
{
- send100Messages((key, counter) ->
- counter == 77
- ? new Bytes(stringSerializer.serialize(TOPIC, "BOOM!"))
- : serialize(key, counter));
+ send100Messages((partition, key, counter) ->
+ {
+ Bytes value;
+ String type;
+
+ if (counter == 77)
+ {
+ value = serializeFooMessage(key, counter);
+ type = "foo";
+ }
+ else
+ {
+ if (counter%3 != 0)
+ {
+ value = serializeClientMessage(key, counter);
+ type = "message";
+ }
+ else {
+ value = serializeGreeting(key);
+ type = "greeting";
+ }
+ }
+
+ return toRecord(partition, key, value, Optional.of(type));
+ });
await("Consumer failed")
.atMost(Duration.ofSeconds(30))
+ .pollInterval(Duration.ofSeconds(1))
.until(() -> !endlessConsumer.running());
checkSeenOffsetsForProgress();
endlessConsumer.start();
await("Consumer failed")
.atMost(Duration.ofSeconds(30))
+ .pollInterval(Duration.ofSeconds(1))
.until(() -> !endlessConsumer.running());
checkSeenOffsetsForProgress();
Set<TopicPartition> withProgress = new HashSet<>();
partitions().forEach(tp ->
{
- Long oldOffset = oldOffsets.get(tp);
- Long newOffset = newOffsets.get(tp);
+ Long oldOffset = oldOffsets.get(tp) + 1;
+ Long newOffset = newOffsets.get(tp) + 1;
if (!oldOffset.equals(newOffset))
{
log.debug("Progress for {}: {} -> {}", tp, oldOffset, newOffset);
/** Helper methods for setting up and running the tests */
+ void seekToEnd()
+ {
+ offsetConsumer.assign(partitions());
+ offsetConsumer.seekToEnd(partitions());
+ partitions().forEach(tp ->
+ {
+ // seekToEnd() works lazily: it only takes effect on poll()/position()
+ Long offset = offsetConsumer.position(tp);
+ log.info("New position for {}: {}", tp, offset);
+ });
+ // The new positions must be commited!
+ offsetConsumer.commitSync();
+ offsetConsumer.unsubscribe();
+ }
+
void doForCurrentOffsets(BiConsumer<TopicPartition, Long> consumer)
{
offsetConsumer.assign(partitions());
}
- void send100Messages(BiFunction<Integer, Long, Bytes> messageGenerator)
+ public interface RecordGenerator<K, V>
+ {
+ public ProducerRecord<String, Bytes> generate(int partition, String key, int counter);
+ }
+
+ void send100Messages(RecordGenerator recordGenerator)
{
- long i = 0;
+ int i = 0;
for (int partition = 0; partition < 10; partition++)
{
for (int key = 0; key < 10; key++)
{
- Bytes value = messageGenerator.apply(key, ++i);
-
ProducerRecord<String, Bytes> record =
- new ProducerRecord<>(
- TOPIC,
- partition,
- Integer.toString(key%2),
- value);
+ recordGenerator.generate(partition, Integer.toString(partition*10+key%2), ++i);
kafkaProducer.send(record, (metadata, e) ->
{
}
}
- Bytes serialize(Integer key, Long value)
+ ProducerRecord<String, Bytes> toRecord(int partition, String key, Bytes value, Optional<String> type)
+ {
+ ProducerRecord<String, Bytes> record =
+ new ProducerRecord<>(TOPIC, partition, key, value);
+
+ type.ifPresent(typeId -> record.headers().add("__TypeId__", typeId.getBytes()));
+ return record;
+ }
+
+ Bytes serializeClientMessage(String key, int value)
{
- ClientMessage message = new ClientMessage();
- message.setClient(key.toString());
- message.setMessage(value.toString());
+ TestClientMessage message = new TestClientMessage(key, Integer.toString(value));
return new Bytes(valueSerializer.serialize(TOPIC, message));
}
+ Bytes serializeGreeting(String key)
+ {
+ TestGreeting message = new TestGreeting(key, LocalDateTime.now());
+ return new Bytes(valueSerializer.serialize(TOPIC, message));
+ }
+
+ Bytes serializeFooMessage(String key, int value)
+ {
+ TestFooMessage message = new TestFooMessage(key, (long)value);
+ return new Bytes(valueSerializer.serialize(TOPIC, message));
+ }
@BeforeEach
public void init()
{
testHandler = record -> {} ;
+ seekToEnd();
+
oldOffsets = new HashMap<>();
newOffsets = new HashMap<>();
receivedRecords = new HashSet<>();
newOffsets.put(tp, offset - 1);
});
- Consumer<ConsumerRecord<String, ClientMessage>> captureOffsetAndExecuteTestHandler =
+ Consumer<ConsumerRecord<String, ValidMessage>> captureOffsetAndExecuteTestHandler =
record ->
{
newOffsets.put(
public static class Configuration
{
@Bean
- Serializer<ClientMessage> serializer()
+ Serializer<ValidMessage> serializer()
{
return new JsonSerializer<>();
}