Scenarios can be marked as ignored by prefixing them with 'Pending'

[akikhtenko]

This pull request allows (as least for us) to do proper ATDD. I.e. define a scenario first, see it failing then do incremental changes to the production code until the scenario becomes green. So a scenario can remain in the 'pending' state and be merged to master without breaking CI builds.

There're other possible applications of the 'pending' scenario annotation.

You may find the TextParser refactoring quite intrusive - I don't mind if you throw it away and replace it with old (not very elegant) switch statement.

The change should be transparent for the existing library clients.

[akikhtenko]

Sure, no worries. Think a more generic annotation handling is a great idea.
Closing the PR for now.

[cressie176]

Support for Pending annotations is in 0.6.2

On 28 September 2013 19:52, akikhtenko [email protected] wrote:

This pull request allows (as least for us) to do proper ATDD. I.e. define
a scenario first, see it failing then do incremental changes to the
production code until the scenario becomes green. So a scenario can remain
in the 'pending' state and be merged to master without breaking CI builds.

There're other possible applications of the 'pending' scenario annotation.

You may find the TextParser refactoring quite intrusive - I don't mind if
you throw it away and replace it with old (not very elegant) switch
statement.

The change should be transparent for the existing library clients.

You can merge this Pull Request by running

git pull https://github.com/akikhtenko/yadda pending-scenarios

Or view, comment on, or merge it at:

#24
Commit Summary

• Scenarios can be marked as ignored by prefixing them with 'Pending'

File Changes

• M examples/casper/spec/google-spec.txthttps://github.com/Scenarios can be marked as ignored by prefixing them with 'Pending' #24/files#diff-0(8)
• M examples/casper/test.jshttps://github.com/Scenarios can be marked as ignored by prefixing them with 'Pending' #24/files#diff-1(10)
• M lib/parsers/TextParser.jshttps://github.com/Scenarios can be marked as ignored by prefixing them with 'Pending' #24/files#diff-2(77)
• M test/TextParserTests.jshttps://github.com/Scenarios can be marked as ignored by prefixing them with 'Pending' #24/files#diff-3(9)

Patch Links:

• https://github.com/acuminous/yadda/pull/24.patch
• https://github.com/acuminous/yadda/pull/24.diff

http://peaceoneday.org/