From: Kai Moritz Date: Mon, 18 Apr 2022 11:37:33 +0000 (+0200) Subject: Springify: Refactor - Aufruf der Assertions vereinfacht X-Git-Url: http://juplo.de/gitweb/?a=commitdiff_plain;h=87f644aee3c6f20bb89590d936654ec8e96f754d;p=demos%2Fkafka%2Ftraining Springify: Refactor - Aufruf der Assertions vereinfacht * Warum auch immer: Der Compiler erkennt den Typ des Generics in der springifizierten Version nur dann ohne erzwungenen Cast korrekt, wenn der `describeAs()`-Aufruf als letzes erfolgt. * In der nicht springifizierten Version, ist die Aufrufreihenfolge egal: Der Compiler erkennt den Typ des Generics unabhängig davon korrekt. * An ggf. transitiv angezogenen Abhängigkeiten liegt es laut `mvn help:effective-pom` nicht. Dies zeigt für beide Versionen (abgesehen von dem explizit ergänzten `spring-kafka`) exakt die selben Abhängigkeiten. --- diff --git a/src/test/java/de/juplo/kafka/ApplicationTests.java b/src/test/java/de/juplo/kafka/ApplicationTests.java index 1d3546c..5a0f43d 100644 --- a/src/test/java/de/juplo/kafka/ApplicationTests.java +++ b/src/test/java/de/juplo/kafka/ApplicationTests.java @@ -9,7 +9,6 @@ import org.apache.kafka.common.TopicPartition; import org.apache.kafka.common.errors.RecordDeserializationException; import org.apache.kafka.common.serialization.*; import org.apache.kafka.common.utils.Bytes; -import org.assertj.core.api.OptionalAssert; import org.junit.jupiter.api.*; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.boot.autoconfigure.kafka.KafkaAutoConfiguration; @@ -122,9 +121,9 @@ class ApplicationTests assertThatNoException() .describedAs("Consumer should not be running") .isThrownBy(() -> endlessConsumer.exitStatus()); - ((OptionalAssert)assertThat(endlessConsumer.exitStatus())) - .describedAs("Consumer should have exited abnormally") - .containsInstanceOf(RecordDeserializationException.class); + assertThat(endlessConsumer.exitStatus()) + .containsInstanceOf(RecordDeserializationException.class) + .describedAs("Consumer should have exited abnormally"); }