[jira] Commented: (MSUREFIRE-45) The test attribute is incorrectly described

2006-02-27 Thread Gr?gory Joseph (JIRA)
[ http://jira.codehaus.org/browse/MSUREFIRE-45?page=comments#action_59603 ] 

Grégory Joseph commented on MSUREFIRE-45:
-

great :)

> The test attribute is incorrectly described
> ---
>
>  Key: MSUREFIRE-45
>  URL: http://jira.codehaus.org/browse/MSUREFIRE-45
>  Project: Maven 2.x Surefire Plugin
> Type: Improvement

> Reporter: Grégory Joseph
> Priority: Trivial
>  Fix For: 2.1.3

>
>
> The the surefire plugin describe its test parameter with "Specify this 
> parameter if you want to use the test regex notation to select tests to run."
> However, from what I see in the SurefirePlugin code, this is abusing the 
> "regex" term, since it is concatening "**/" + regex + ".java", which doesn't 
> look much like a standard regex.
> i.e, using the regex syntax, one would expect to set "test" to ".*TestCase", 
> but this doesn't work - "*TestCase" gives the expected results.
> All in all, I think this is just a matter of claryfiyng the doc - I haven't 
> checked the surefire code itself, but from the results I get, I bet it 
> doesn't use "real" regexes either.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[jira] Commented: (MSUREFIRE-45) The test attribute is incorrectly described

2006-02-27 Thread Jason van Zyl (JIRA)
[ http://jira.codehaus.org/browse/MSUREFIRE-45?page=comments#action_59601 ] 

Jason van Zyl commented on MSUREFIRE-45:


This is really the ant pattern matching code, so the docs should be updated.

> The test attribute is incorrectly described
> ---
>
>  Key: MSUREFIRE-45
>  URL: http://jira.codehaus.org/browse/MSUREFIRE-45
>  Project: Maven 2.x Surefire Plugin
> Type: Improvement

> Reporter: Grégory Joseph
> Priority: Trivial
>  Fix For: 2.1.3

>
>
> The the surefire plugin describe its test parameter with "Specify this 
> parameter if you want to use the test regex notation to select tests to run."
> However, from what I see in the SurefirePlugin code, this is abusing the 
> "regex" term, since it is concatening "**/" + regex + ".java", which doesn't 
> look much like a standard regex.
> i.e, using the regex syntax, one would expect to set "test" to ".*TestCase", 
> but this doesn't work - "*TestCase" gives the expected results.
> All in all, I think this is just a matter of claryfiyng the doc - I haven't 
> checked the surefire code itself, but from the results I get, I bet it 
> doesn't use "real" regexes either.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]