[ 
https://issues.apache.org/jira/browse/KAFKA-16472?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17833988#comment-17833988
 ] 

Kuan Po Tseng commented on KAFKA-16472:
---------------------------------------

Looks like 
[https://github.com/junit-team/junit5/blob/4c0dddad1b96d4a20e92a2cd583954643ac56ac0/junit-jupiter-params/src/main/java/org/junit/jupiter/params/ParameterizedTestNameFormatter.java#L93]
in this line, the scala tests can get the correct parameter name (i.e. quorum) 
back while java tests could only get Optional.empty. As PoAn pointed out, add 
compilerArgs {{-parameters}} can solve the missing parameter name in java tests.

> Integration tests in Java don't really run kraft case
> -----------------------------------------------------
>
>                 Key: KAFKA-16472
>                 URL: https://issues.apache.org/jira/browse/KAFKA-16472
>             Project: Kafka
>          Issue Type: Test
>            Reporter: PoAn Yang
>            Assignee: PoAn Yang
>            Priority: Major
>
> Following test cases don't really run kraft case. The reason is that the test 
> info doesn't contain parameter name, so it always returns false in 
> TestInfoUtils#isKRaft.
>  * TopicCommandIntegrationTest
>  * DeleteConsumerGroupsTest
>  * AuthorizerIntegrationTest
>  * DeleteOffsetsConsumerGroupCommandIntegrationTest
>  
> We can add `options.compilerArgs += '-parameters'` after 
> [https://github.com/apache/kafka/blob/376e9e20dbf7c7aeb6f6f666d47932c445eb6bd1/build.gradle#L264-L273]
>  to fix it.
>  
> Also, we have to add `String quorum` to cases in 
> DeleteOffsetsConsumerGroupCommandIntegrationTest.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to