search

BlessCSS / bless

CSS Post-Processor
blesscss.com
MIT License
282 stars 60 forks source link

Working tests, mocha #47

Closed michaek closed 9 years ago

michaek commented 9 years ago

Following from #46.

I was hoping to not do any major changes to get tests to run, but after running against the cryptic error messages from vows for a while, I switched the tests to mocha to get helpful errors.

I believe this also supports making the tests easier to reason about, because mocha avoids the magical context that vows uses (and requires), opting for variable scoping.

The package.json reformatting doesn't reflect my opinions - just the result of running npm install.

michaek commented 9 years ago

Ok. I didn't see the long-running 4.0.0 branch. This is probably a no-go.

paulyoung commented 9 years ago

Hey @michaek

Thanks for your contribution and your interest.

I'm not sure what happened to the test suite. Nothing has changed on master for a while.

I'm hesitant to make changes there since I'm trying to focus any efforts on v4, which is almost ready for some other contributors to dive in. I have a couple of things to add first but can't spend any time on it right now.