Re: [PR] MINOR: Ensure that DisplayName is set in all parameterized tests [kafka]

2023-12-04 Thread via GitHub
dajac merged PR #14850: URL: https://github.com/apache/kafka/pull/14850 -- 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:

Re: [PR] MINOR: Ensure that DisplayName is set in all parameterized tests [kafka]

2023-12-01 Thread via GitHub
mumrah commented on PR #14850: URL: https://github.com/apache/kafka/pull/14850#issuecomment-1836803801 I tried this in IntelliJ and I see the test name included in the individual test variation output. Looks great!

Re: [PR] MINOR: Ensure that DisplayName is set in all parameterized tests [kafka]

2023-11-28 Thread via GitHub
dajac commented on code in PR #14850: URL: https://github.com/apache/kafka/pull/14850#discussion_r1407964777 ## core/src/test/scala/kafka/utils/TestInfoUtils.scala: ## @@ -51,7 +51,7 @@ object TestInfoUtils { testInfo.getDisplayName().contains("quorum=zkMigration")

[PR] MINOR: Ensure that DisplayName is set in all parameterized tests [kafka]

2023-11-27 Thread via GitHub
dajac opened a new pull request, #14850: URL: https://github.com/apache/kafka/pull/14850 This is a follow-up to https://github.com/apache/kafka/pull/14687 as we found out that some parameterized tests do not include the test method name in their name. For the context, the JUnit XML report