X-Git-Url: http://juplo.de/gitweb/?a=blobdiff_plain;f=src%2Ftest%2Fjava%2Fde%2Fjuplo%2Fkafka%2FApplicationTests.java;h=ee8e8c418d91ee08f92f21973e064a19dfbed75e;hb=4aef11c37556c827397af6c25f5c129cd0147a68;hp=cbf215ef4f47b97f89b46abee37aa58b87195c66;hpb=a9200a876060edc8683dfd6d0d16c23407c189ad;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 cbf215e..ee8e8c4 100644 --- a/src/test/java/de/juplo/kafka/ApplicationTests.java +++ b/src/test/java/de/juplo/kafka/ApplicationTests.java @@ -6,10 +6,8 @@ import org.apache.kafka.clients.consumer.KafkaConsumer; import org.apache.kafka.clients.producer.KafkaProducer; import org.apache.kafka.clients.producer.ProducerRecord; import org.apache.kafka.common.TopicPartition; -import org.apache.kafka.common.serialization.BytesDeserializer; -import org.apache.kafka.common.serialization.BytesSerializer; -import org.apache.kafka.common.serialization.LongSerializer; -import org.apache.kafka.common.serialization.StringSerializer; +import org.apache.kafka.common.errors.RecordDeserializationException; +import org.apache.kafka.common.serialization.*; import org.apache.kafka.common.utils.Bytes; import org.junit.jupiter.api.*; import org.springframework.beans.factory.annotation.Autowired; @@ -19,6 +17,7 @@ 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.support.serializer.JsonSerializer; import org.springframework.kafka.test.context.EmbeddedKafka; import org.springframework.test.context.TestPropertySource; import org.springframework.test.context.junit.jupiter.SpringJUnitConfig; @@ -27,14 +26,14 @@ import java.time.Duration; 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.function.Function; import java.util.stream.Collectors; import java.util.stream.IntStream; import static de.juplo.kafka.ApplicationTests.PARTITIONS; import static de.juplo.kafka.ApplicationTests.TOPIC; -import static org.assertj.core.api.Assertions.assertThat; +import static org.assertj.core.api.Assertions.*; import static org.awaitility.Awaitility.*; @@ -58,19 +57,25 @@ class ApplicationTests StringSerializer stringSerializer = new StringSerializer(); - LongSerializer longSerializer = new LongSerializer(); + @Autowired + Serializer valueSerializer; @Autowired KafkaProducer kafkaProducer; @Autowired + KafkaConsumer kafkaConsumer; + @Autowired KafkaConsumer offsetConsumer; @Autowired ApplicationProperties properties; @Autowired + EndlessConsumer endlessConsumer; + @Autowired RecordHandler recordHandler; Map oldOffsets; Map newOffsets; + Set> receivedRecords; /** Tests methods */ @@ -79,14 +84,11 @@ class ApplicationTests @Order(1) // << The poistion pill is not skipped. Hence, this test must run first void commitsCurrentOffsetsOnSuccess() throws ExecutionException, InterruptedException { - send100Messages(i -> new Bytes(longSerializer.serialize(TOPIC, i))); - - Set> received = new HashSet<>(); - recordHandler.testHandler = record -> received.add(record); + send100Messages((key, counter) -> serialize(key, counter)); await("100 records received") .atMost(Duration.ofSeconds(30)) - .until(() -> received.size() >= 100); + .until(() -> receivedRecords.size() >= 100); await("Offsets committed") .atMost(Duration.ofSeconds(10)) @@ -95,22 +97,36 @@ class ApplicationTests checkSeenOffsetsForProgress(); compareToCommitedOffsets(newOffsets); }); + + assertThatExceptionOfType(IllegalStateException.class) + .isThrownBy(() -> endlessConsumer.exitStatus()) + .describedAs("Consumer should still be running"); } @Test @Order(2) void commitsOffsetOfErrorForReprocessingOnError() { - send100Messages(counter -> + send100Messages((key, counter) -> counter == 77 ? new Bytes(stringSerializer.serialize(TOPIC, "BOOM!")) - : new Bytes(longSerializer.serialize(TOPIC, counter))); + : serialize(key, counter)); await("Consumer failed") .atMost(Duration.ofSeconds(30)) .untilAsserted(() -> 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()) + .containsInstanceOf(RecordDeserializationException.class) + .describedAs("Consumer should have exited abnormally"); } @@ -122,7 +138,9 @@ class ApplicationTests { Long expected = offsetsToCheck.get(tp) + 1; log.debug("Checking, if the offset for {} is {}", tp, expected); - assertThat(offset).isEqualTo(expected); + assertThat(offset) + .describedAs("Committed offset corresponds to the offset of the consumer") + .isEqualTo(expected); }); } @@ -140,7 +158,9 @@ class ApplicationTests withProgress.add(tp); } }); - assertThat(withProgress).isNotEmpty().describedAs("Found no partitions with any offset-progress"); + assertThat(withProgress) + .describedAs("Some offsets must have changed, compared to the old offset-positions") + .isNotEmpty(); } @@ -163,7 +183,7 @@ class ApplicationTests } - void send100Messages(Function messageGenerator) + void send100Messages(BiFunction messageGenerator) { long i = 0; @@ -171,7 +191,7 @@ class ApplicationTests { for (int key = 0; key < 10; key++) { - Bytes value = messageGenerator.apply(++i); + Bytes value = messageGenerator.apply(key, ++i); ProducerRecord record = new ProducerRecord<>( @@ -180,6 +200,7 @@ class ApplicationTests Integer.toString(key%2), value); + record.headers().add("__TypeId__", "message".getBytes()); kafkaProducer.send(record, (metadata, e) -> { if (metadata != null) @@ -204,6 +225,14 @@ class ApplicationTests } } + Bytes serialize(Integer key, Long value) + { + ClientMessage message = new ClientMessage(); + message.setClient(key.toString()); + message.setMessage(value.toString()); + return new Bytes(valueSerializer.serialize(TOPIC, message)); + } + @BeforeEach public void init() @@ -212,6 +241,7 @@ class ApplicationTests oldOffsets = new HashMap<>(); newOffsets = new HashMap<>(); + receivedRecords = new HashSet<>(); doForCurrentOffsets((tp, offset) -> { @@ -221,20 +251,37 @@ class ApplicationTests recordHandler.captureOffsets = record -> + { + receivedRecords.add(record); newOffsets.put( new TopicPartition(record.topic(), record.partition()), record.offset()); + }; + + endlessConsumer.start(); } + @AfterEach + public void deinit() + { + try + { + endlessConsumer.stop(); + } + catch (Exception e) + { + log.info("Exception while stopping the consumer: {}", e.toString()); + } + } - public static class RecordHandler implements Consumer> + public static class RecordHandler implements Consumer> { - Consumer> captureOffsets; - Consumer> testHandler; + Consumer> captureOffsets; + Consumer> testHandler; @Override - public void accept(ConsumerRecord record) + public void accept(ConsumerRecord record) { captureOffsets .andThen(testHandler) @@ -248,11 +295,17 @@ class ApplicationTests { @Primary @Bean - public Consumer> testHandler() + public Consumer> testHandler() { return new RecordHandler(); } + @Bean + Serializer serializer() + { + return new JsonSerializer<>(); + } + @Bean KafkaProducer kafkaProducer(ApplicationProperties properties) {