shantanushukla opened a new pull request, #23279: URL: https://github.com/apache/kafka/pull/23279
JIRA: https://issues.apache.org/jira/browse/KAFKA-20988 Parent: https://issues.apache.org/jira/browse/KAFKA-20948 Migrate hamcrest `assertThat` usage in streams test fixtures and `org.apache.kafka.test` mock helpers to JUnit 5. Files changed: - `streams/src/testFixtures/java/org/apache/kafka/streams/utils/TestUtils.java` - `streams/src/testFixtures/java/org/apache/kafka/streams/utils/UniqueTopicSerdeScope.java` - `streams/src/testFixtures/java/org/apache/kafka/test/MockApiProcessor.java` - `streams/src/testFixtures/java/org/apache/kafka/test/MockClientSupplier.java` - `streams/src/testFixtures/java/org/apache/kafka/test/StreamsTestUtils.java` - `streams/src/test/java/org/apache/kafka/test/MockApiFixedKeyProcessor.java` `AssignmentTestUtils` is out of scope: it still exposes custom Hamcrest `Matcher`s used by unclaimed `processor.internals.assignment` tests, so `testFixturesImplementation libs.hamcrest` on `:streams` is retained. No production code changes. This contribution is original work and is licensed to the project under the Apache License, Version 2.0. ### Tests ``` ./gradlew :streams:compileTestFixturesJava :streams:compileTestJava \ :streams:checkstyleTest :streams:spotlessCheck \ :streams:test --tests org.apache.kafka.streams.kstream.internals.KStreamMapTest ``` BUILD SUCCESSFUL. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
