JBehave
  1. JBehave
  2. JBEHAVE-711

empty lines in example tables should be ignored

    Details

    • Type: Improvement Improvement
    • Status: Resolved Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.5.4
    • Fix Version/s: 3.6
    • Component/s: Core
    • Labels:
      None
    • Number of attachments :
      0

      Description

      I tried to improve the layout of an example table in one story by adding newlines before each comment line (to improve the legibility a bit),

      however it turns out that an empty line in the table gives a list of empty parameters in the examples (e.g. ||| when you have 2 parameters).

      I think it would be better if empty lines are ignored, unless I'm missing something.

      If that is ok, I can write a patch.

        Activity

        Hide
        Alexander Lehmann added a comment -

        for example:

        Scenario: a scenario

        Given <string>

        Examples:
        |string|

        |-- a comment

        |parameter|

        gives

        Examples:
        Given <string>

        |string|
        ||
        ||
        |parameter|

        Show
        Alexander Lehmann added a comment - for example: Scenario: a scenario Given <string> Examples: |string| |-- a comment |parameter| gives Examples: Given <string> |string| || || |parameter|
        Hide
        Alexander Lehmann added a comment -
        Show
        Alexander Lehmann added a comment - https://github.com/alexlehm/jbehave-core/commit/50b47f5f2222b8ec849a4002fd1389f157f4c95f (also fixed a typo and some whitespace in the test file)
        Hide
        Mauro Talevi added a comment -

        Pulled with thanks.

        Show
        Mauro Talevi added a comment - Pulled with thanks.
        Mauro Talevi made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 3.6 [ 17721 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Alexander Lehmann
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: