Test case for the PrettyFormatter #544

Merged
merged 2 commits into from Jul 16, 2013

Conversation

Projects
None yet
1 participant
@brasmusson
Contributor

brasmusson commented Jun 17, 2013

Add a test case for the PrettyFormatter to verify that the pretty formatter aligns the location strings correct, when a scenario is executed with Cucumber-JVM.

This test case expose the well known IndexOutOfBoundsException in Cucumber-JVM 1.1.3. It also shows that when using the current snapshot of the Gherkin library (cucumber-attic/gherkin@74bb687) the pretty formatter still does not align the location strings correctly.

The test case is something of an acceptance test for resolving the miss match between the expectations of the pretty formatter in the Gherkin libarary and the formatter call sequence in Cucumber-JVM.

Add a test case for the PrettyFormatter
Add a test case for the PrettyFormatter to verify that the pretty
formatter aligns the location strings correct, when a scenario is
executed with Cucumber-JVM.

brasmusson added a commit to brasmusson/cucumber-jvm that referenced this pull request Jul 14, 2013

Merge PR #557 into PR #544
Verify that the changes in PR #557 indeed fixes #491 and the other
reports of that the PrettyFormatter is broken.
Merge cucumber:master into brasmusson:pretty-formatter-test
Now when cucumber-jvm has reveted to all-steps-first execution, the
test case for the PrettyFormatter should pass.

brasmusson added a commit that referenced this pull request Jul 16, 2013

Merge pull request #544 from brasmusson/pretty-formatter-test
Merge pull request #544 Test case for the PrettyFormatter
Update the History.md

@brasmusson brasmusson merged commit 09c71de into cucumber:master Jul 16, 2013

1 check passed

default The Travis CI build passed
Details

@brasmusson brasmusson deleted the brasmusson:pretty-formatter-test branch Aug 3, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment