chia7712 commented on PR #16301:
URL: https://github.com/apache/kafka/pull/16301#issuecomment-2211466626

   > What is the alternative to this PR?
   
   another solution recommended by gradle is using `jvm-test-suite`. for 
example:
   ```
   testing {
       suites { 
           test { 
               useJUnitJupiter() 
           }
       }
   }
   ```
   However, it is incubating so I prefer to declare the junit platform 
explicitly.
   
   > Does this solve the build warning?
   
   yep. see following console output
   
   ### before
   ```
   > Task :connect:mirror-client:test
   The automatic loading of test framework implementation dependencies has been 
deprecated. This is scheduled to be removed in Gradle 9.0. Declare the desired 
test framework directly on the test suite or explicitly declare the test 
framework implementation dependencies on the test's runtime classpath. Consult 
the upgrading guide for further information: 
https://docs.gradle.org/8.8/userguide/upgrading_version_8.html#test_framework_implementation_dependencies
   ```
   
   ### after
   ```
   > Task :connect:mirror-client:test
   
   Gradle Test Run :connect:mirror-client:test > Gradle Test Executor 146 > 
SourceAndTargetTest > testEquals() PASSED
   ```


-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to