The way I understand you this is halfway a documentation issue; any
non-legal value including the string "filesystem" will actually give you
filesystem order.

So I could "solve" this by just adding "filesystem" to the list of legal
options, right ?

Another issue is that Dennis actually converted the default value to
"alphabetical" in his original fix. When introducing more options I
reverted to the old-style behaviour as default (filesystem).

Actually changing the default order will break some people's tests, but
I have a marginal preference for alphabetical. But it's sufficiently
marginal taht I prefer compatibility ;)

KRistian


ti., 14.12.2010 kl. 17.32 +0100, skrev Benjamin Bentmann:
> Hi Kristian,
> 
> > Author: krosenvold
> > Date: Tue Dec 14 14:55:52 2010
> > New Revision: 1049118
> >
> > URL: http://svn.apache.org/viewvc?rev=1049118&view=rev
> > Log:
> > [SUREFIRE-321] Added runOrder mojo attribute, including documentation.
> > [...]
> > @@ -595,6 +595,20 @@ public class IntegrationTestMojo
> >       private Boolean parallelMavenExecution;
> >
> >       /**
> > +      * Defines the order the tests will be run in. Supported values are 
> > alphabetical, reversealphabetical
> > +      * random, hourly (alphabetical on even hours, reverse alphabetical 
> > on odd hours).
> > +      *
> > +      * Not supplying a value for this setting will run tests in 
> > filesystem order.
> > +      *
> > +      * Odd/Even is determined at the time the of scanning the classpath, 
> > meaning it could change during
> > +      * a multi-module build.
> > +      *
> > +      * @parameter
> > +      * @since 2.7
> > +      */
> > +     private String runOrder;
> 
> Should there be an explicit value for the filesystem order? I'm thinking 
> about a parent POM where runOrder gets set to say hourly but a child 
> project wants to stick with the (current) filesystem order.
> 
> 
> Benjamin
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
> 



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to