Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding short stack traces as an option #227

Closed
wants to merge 32 commits into from

Conversation

AndrewJohnBenjamin
Copy link

Updated the package to include a command line option that allows the user to specify whether or not they would like single line stack traces upon step failure.

Currently the stack traces that are output by the package in the event of a step failure are pretty large and make it a little more difficult to see what has failed. The changes I have made cut this trace down to a single line and shows only the line in the code that caused the step to fail.

The changes involved slight modifications to runtime, configuration, argument parser and step_definition.

I believe this addition would benefit others as it seems this issue has been flagged already.

#157

@jlin412
Copy link

jlin412 commented Sep 3, 2014

+1 Any idea when this feature will be merged.

@filipstachura
Copy link

+1

i-e-b and others added 26 commits November 7, 2014 15:01
Purpose
=======

This is designed to make the filesystem layout and arrangement of features
more flexible. Target audience is larger projects with multiple
developers.

Changes:
 - Throws errors if more than one step definition matches a feature step.
 - Small fixes for file path matching in Windows.
 - Special tag format for selecting which files are used for steps.

The special tags are designed to be used where the `-r` tag is specified.
They should work where file name matching is done, but will be less
useful.

Tags
====

If no source tags are set, default behaviour is used: all matching step
definitions will be used to fulfill a step.

If one or more file source tags are set in a file, then only the
specified sources are used. Sources accumulate, so each tag ADDS to the
available pool of definitions.

Tags are inherited, so setting at a feature level sets for all
scenarios.

Specific named file tags don't use path or `.js` ending. Files with the
matching name will be allowed as step definitions:

 @:MySteps

Will match (e.g.)

 ./step_definitions/MySteps.js
 ./step_definitions/specific/path/MySteps.js

Directory path tags match against the end of the step files' path.
This is non-recursive, so child directories will not be used.

 @./Interpretation

Will match

 ./step_definition/Interpretation/x.js
 ./step_definition/Interpretation/y.js
 ./step_definition/deeper/Interpretation/x.js

But not

 ./step_definition/Interpretation/deeper/y.js

Any steps loaded from {step path}/support/ will be included
regardless of tags
…e supplied primary key. Note: Method assumes the first cell in a row is the primary key
Updated the table object to include a function that gets
the entire row for where the first cell matches a value

Merge remote-tracking branch 'andy/master' into sigmaNPM
…nt once all the support files are loaded.

Command line option is : brkOnSupportCodeLoaded
Closes cucumber#3

Includes source mapping. Probably won't map minified
outputs well.
Closes cucumber#4
Requires grunt-cucumber-sigma 0.3.3+
Closes cucumber#5

Adds a `deferredStep` definition which makes any scenario
containing it run after all non-deferred scenarios.
@jbpros
Copy link
Member

jbpros commented May 15, 2015

Thanks @AndrewJohnBenjamin for this.

The short stack traces are now implemented (see f584233).

There are several other features in this PR. Please open new, separate, PRs for those.

@jbpros jbpros closed this May 15, 2015
@lock
Copy link

lock bot commented Oct 25, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 25, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants