[GitHub] [maven-surefire] Tibor17 commented on pull request #169: [SUREFIRE-1405] Allows user to extend RunOrder & RunOrderCalculator

[GitBox <gi...@apache.org>]

Tibor17 commented on pull request #169:
URL: https://github.com/apache/maven-surefire/pull/169#issuecomment-780521834


   > 
   > 
   > @Tibor17 I noticed that JUnit 5 will have some kind test class ordering in 5.8.0: [junit-team/junit5#2488](https://github.com/junit-team/junit5/pull/2488)
   > 
   > I'm wondering whether this is going to just work with surefire?
   > Do we still need something like this PR here for ordering on the surefire side?
   
   @famod 
   This is very good patch from @dipak-pawar and I will try to rebase it on the latest source code in master. But this has nothing to do with the JUnit5. They do their job and we do ours. We must have the ability to order the classes. How can you otherwise order the classes while multiple forks wants to execute the tests? You cannot do it in JUnit5. You have to do it before the fork starter. 
   
   So this was wrong question. You must see the pipeline in order to understand what config parameter and how to use.


----------------------------------------------------------------
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.

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