JBehave
  1. JBehave
  2. JBEHAVE-981

Changing the default of "useDelimiterNamedParameters"

    Details

    • Type: Improvement Improvement
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 4.0
    • Component/s: Core
    • Labels:
      None
    • Number of attachments :
      0

      Description

      I discovered that there is a problem with parameter injection if it is not configured in a special way.

      See the example below. All values should be injected. But that does not work, only "555" is injected correctly.

      Parameters from an examples table are not injected in an expected way if MostUsefulConfiguration is not set like: .useParameterControls(new
      ParameterControls().useDelimiterNamedParameters(true))

      The default is false for using delimeter named parameters.

      I suggest du use true as the default instead.

      See the discussion in the mailing list: http://www.mail-archive.com/user@jbehave.codehaus.org/msg02106.html

      Scenario: Example
      Given the account data from <creditAccount> is known and remembered
      And the account data from <debitAccount> is known and remembered
      And the account data from 555 is known and remembered
      And the account data from <account> is known and remembered
      Examples:

      creditAccount debitAccount account
      123 321 999

        Activity

        Mauro Talevi made changes -
        Field Original Value New Value
        Affects Version/s 3.9.1 [ 19830 ]
        Fix Version/s 3.9.1 [ 19830 ]
        Issue Type Wish [ 5 ] Improvement [ 4 ]
        Hide
        Mauro Talevi added a comment -

        Changing to 4.0 to avoid backward-incompatibility.

        Show
        Mauro Talevi added a comment - Changing to 4.0 to avoid backward-incompatibility.
        Mauro Talevi made changes -
        Fix Version/s 3.9.1 [ 19830 ]
        Fix Version/s 4.0 [ 18486 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Otto Diesel
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated: