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

[discussion] [question] [feature request] Support assertion data diff view for sinon-chai #186

Closed
hellboy81 opened this issue Jul 10, 2015 · 5 comments

Comments

@hellboy81
Copy link

Is there possible to compare visually parameters in sinon-chai assertions:

.. .should.have.been.calledWith(....)

Is there possible in release 1.0.24 display chai.js assertion errors (demo from twitter)?

@hellboy81 hellboy81 changed the title [discussion] [question] [freature request] Support for sinon-chai [discussion] [question] [feature request] Support for sinon-chai Jul 10, 2015
@ArtemGovorov
Copy link
Member

We are using chai convention where AssertionError is raised with showDiff flag for those assertions that it makes sense to show the diff for. So it's up to the assertion library to set the flag and if I'm not mistaken, sinon-chai doesn't set the flag for its assertions.

I'll se what we can do about it.

@boneskull
Copy link

I'm going to be taking a look at Wallaby, but sinon-chai and chai-as-expected support is going to be crucial for me.

@ArtemGovorov ArtemGovorov changed the title [discussion] [question] [feature request] Support for sinon-chai [discussion] [question] [feature request] Support assertion data diff view for sinon-chai Jul 11, 2015
@ArtemGovorov
Copy link
Member

@boneskull wallaby supports both sinon-chai and chai-as-expected (any assertion framework really), this issue is about this feature support for sinon-chai assertion errors

@hellboy81
Copy link
Author

Discussion updated

@ArtemGovorov
Copy link
Member

@hellboy81 Thanks, so as soon Sinon-chai is fixed to pass expected, actual and showDiff for certain assertions, wallaby.js 'Show Diff' feature will just work.

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

No branches or pull requests

4 participants