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

Daniel Godfrey commented on JBEHAVE-398:
----------------------------------------

Makes sense, Parameters is a much better name.

I don't like the hierarchy reduction though, it means ChainedParameters needs 
to implement all of the valueAs methods, which as far as I can tell, won't ever 
be called anywhere. I've pushed another patch to 
https://github.com/dgodfrey/jbehave-core/tree/jbehave_398, that introduces 
another interface, RawParameters, that removes this necessity. It also removes 
the hasValue method completely, as it's not required any more. IMO, it's enough 
of a code reduction to add the tiny new interface for.

> Provide table row values as converted parameters
> ------------------------------------------------
>
>                 Key: JBEHAVE-398
>                 URL: http://jira.codehaus.org/browse/JBEHAVE-398
>             Project: JBehave
>          Issue Type: New Feature
>          Components: Core
>    Affects Versions: 3.1.2
>            Reporter: Daniel Godfrey
>            Assignee: Mauro Talevi
>             Fix For: 3.2
>
>
> Currently, ExamplesTable provides the row values as Map<String,String>.
> In most cases, users will need to convert these values.  We should provide 
> the row values converted as specified types using the same 
> ParameterConverters defined for the step conversion. 
> We should also allow for provision of defaults, both accessing single value 
> and as a table structure.

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