[ 
http://jira.codehaus.org/browse/JBEHAVE-312?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=232610#action_232610
 ] 

James Ravn commented on JBEHAVE-312:
------------------------------------

I would consider any dependency on the greedy matching to be broken. It makes 
no sense at all, because it essentially hides other steps - when would anyone 
ever depend on this? Keeping it for backwards compatibility is non-sensical and 
just causes pain.

For example, if I created this step:

{code}
@Given("$anything") public void anything(String anAnything) {}
{code}

It will hide every single other step in my tests. How is this useful? Remove 
the greedy qualifier or give us a config option. Please :-). This single issue 
makes me dislike jbehave more and more every time I have to waste effort on it.

> Configuration option for non-greedy matchers for steps
> ------------------------------------------------------
>
>                 Key: JBEHAVE-312
>                 URL: http://jira.codehaus.org/browse/JBEHAVE-312
>             Project: JBehave
>          Issue Type: Improvement
>          Components: Core
>    Affects Versions: 2.5
>            Reporter: James Ravn
>
> The biggest sore point for my team are the greedy reg-ex matchers for step 
> prose.
> This was somewhat solved in another ticket which added a {{priority}} field 
> to the annotation.
> However, this does not scale well with a large amount of steps. It's 
> unmanageable, and a bit ugly, to have to worry about a global priority number.
> This problem would be best fixed if the reg-ex matcher was simply non-greedy. 
> A usable solution, I think, would be to provide a configuration option in the 
> StepsConfiguration to enable this.
> (On a side note, you may want to consider moving to a feature rich templating 
> language like velocity).

-- 
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 from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to