kevinrr888 opened a new pull request, #5581:
URL: https://github.com/apache/accumulo/pull/5581

   - Removed the SIMPLE_MINI_CLUSTER_SUITE tag. Replaced with having each IT 
which was using it instead end with "IT_SimpleSuite". This avoids individual 
ITs from being picked up in the build (since they don't end in "IT").
   - SimpleSharedMacTestSuite -> SimpleSharedMacTestSuiteIT so suite is picked 
up and run in build.
   - Reverted all root pom changes back to what they were before 
41792a48ee1e84784aefa3ea0520a17765c77ff0, except for the one new ignored unused 
declared dependency (org.junit.platform:junit-platform-suite-engine:jar:*), 
which is still needed.
   
   Result:
   - Can run individual ITs via it.test=... (even if they are part of the suite)
   - Can run the whole suite via 
it.test=org.apache.accumulo.suites.SimpleSharedMacTestSuiteIT
   - By default, build will run all ITs including the suite, while avoiding 
running those that are part of the suite twice (i.e., does not run from suite 
then individually--just runs from suite)
   - Avoids any custom build changes for this suite
   
   This approach was discussed and outlined in 
https://github.com/apache/accumulo/pull/5555#issuecomment-2902463211


-- 
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: notifications-unsubscr...@accumulo.apache.org

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

Reply via email to