fixed expect('').to.contain('') so it passes

[dereke]

Found a bug when you are trying to assert against a blank string.
Test case and fix included

[keithamus]

Hey @dereke, thanks for the PR. Could you please rebase your commits so as not to include chai.js. It is built per-release and so shouldn't be built for individual PRs.

[dereke]

Hi @keithamus when we ran the tests without having build chai.js they failed.
Does CI build chai.js when the tests run?

[keithamus]

The CI does indeed run the tests: https://travis-ci.org/chaijs/chai/builds/78216305. Also, you're welcome to build chai.js as many times as you want locally, just don't commit it 😉

[dereke]

OK cool. all done.

[dereke]

@keithamus when do you think this will show up in a release?

[keithamus]

Thanks for reminding me @dereke. I'm going to look at this in a couple of hours and make a release in about 5 hours, which will be 3.2.0 (hopefully)

[dereke]

any chance of publishing the new package?

[keithamus]

@dereke this should be in 3.3.0