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

Make it possible to run utilities tests with --watch #217

Merged
merged 1 commit into from
Nov 29, 2013

Conversation

hazeledmands
Copy link

Hey! I'm starting to play around with possibilities for an overwriteChainableMethod utility to address issue #215, and I was trying to run the utilities tests with:

mocha --watch --require ./test/bootstrap --reporter spec test/utilities.js

But the tests fail if they run more than once because they expect certain methods to not be part of the Assertion prototype. Would it be okay for me to add some cleanup in an after() block?

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e7cf4d4 on demands:test_cleanup into 1654808 on chaijs:master.

@hazeledmands
Copy link
Author

...updated my commit message to match the others, I think.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 4ce3d42 on demands:test_cleanup into 1654808 on chaijs:master.

logicalparadox added a commit that referenced this pull request Nov 29, 2013
Make it possible to run utilities tests with --watch
@logicalparadox logicalparadox merged commit 0e560c6 into chaijs:master Nov 29, 2013
@hazeledmands hazeledmands deleted the test_cleanup branch November 29, 2013 19:43
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.

3 participants