mimaison commented on code in PR #16236:
URL: https://github.com/apache/kafka/pull/16236#discussion_r1631406584
##########
build.gradle:
##########
@@ -522,13 +522,6 @@ subprojects {
// The suites are for running sets of tests in IDEs.
// Gradle will run each test class, so we exclude the suites to avoid
redundantly running the tests twice.
def testsToExclude = ['**/*Suite.class']
- // Exclude PowerMock tests when running with Java 16 or newer until a
version of PowerMock that supports the relevant versions is released
Review Comment:
There are also comments about EasyMock in 4 places:
- 2 in `DescribeTopicPartitionsRequestHandlerTest`
- 2 in `KafkaApisTest`
Let's clean those up as well.
##########
build.gradle:
##########
@@ -522,13 +522,6 @@ subprojects {
// The suites are for running sets of tests in IDEs.
// Gradle will run each test class, so we exclude the suites to avoid
redundantly running the tests twice.
def testsToExclude = ['**/*Suite.class']
- // Exclude PowerMock tests when running with Java 16 or newer until a
version of PowerMock that supports the relevant versions is released
Review Comment:
There's another comment about PowerMock at line 822, can we remove it as
well?
--
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]
