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

Unit tests covering should.exist() behavior #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

leedm777
Copy link
Contributor

@leedm777 leedm777 commented Jun 4, 2015

dirty-chai breaks the should[.not].exist() matchers from chai. I can't quite figure out how it's broken, but I've written some tests that demonstrate it.

These tests validate whether chai's should.exist(obj) behavior still
works as expected.

Added a set of matching expect() style tests just to confirm that they
behave.

These tests validate whether chai's `should.exist(obj)` behavior still
works as expected.

Added a set of matching expect() style tests just to confirm that they
behave.
@leedm777
Copy link
Contributor Author

leedm777 commented Jun 4, 2015

To be clear, the failing Travis CI tests are the point of the PR. The tests should pass, but I don't know how to make them pass. :-(

@joshperry
Copy link
Contributor

@leedm777 Thanks man, I'll take a look at it and see if I can figure something out.

@leedm777
Copy link
Contributor Author

@joshperry Figured anything out yet?

@rovansteen
Copy link

Any progress here? I'm running into a similar problem with a combination of chai, chai-enzyme & dirty-chai. The not part is not working properly due to dirty-chai. It just seems to ignore it. Putting dirty-chai before chai-enzyme does fix it, but if I'm watching my tests (with mocha-webpack) the problem comes back with the second test.

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.

None yet

3 participants