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

Feature: Custom user assertions in hooks to fail a transaction #156

Merged
merged 13 commits into from
Feb 27, 2015

Conversation

netmilk
Copy link
Contributor

@netmilk netmilk commented Feb 26, 2015

  • ability to fail a transaction in a hook
  • get rid with advisable in hook's infrastructure, it does not work with sinon
  • support for writing custom assertions with chai


it 'should run all hooks', (done) ->
runner.executeAllTransactions [transaction], hooksStub, () ->
assert.ok loggerStub.info.calledWith "first"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are not checking order of calls.
I'd vote to use loggerStub.info.firstCall.calledWith 'first'or similar.

@netmilk netmilk changed the title Writing custom assertions in hooks to fail a transaction Feature: Custom user assertions in hooks to fail a transaction Feb 27, 2015
kuba-kubula added a commit that referenced this pull request Feb 27, 2015
…matically

Feature: Custom user assertions in hooks to fail a transaction
@kuba-kubula kuba-kubula merged commit 2d321d4 into master Feb 27, 2015
@kuba-kubula kuba-kubula deleted the netmilk/fail-transaction-programatically branch February 27, 2015 17:02
artem-zakharchenko pushed a commit that referenced this pull request Oct 9, 2019
…r-0.18.0

Update fury-adapter-swagger to version 0.18.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants