Adapt the test suite to newer coffee-source, that uses multiple newlines #16

Closed
wants to merge 1 commit into
from

Projects

None yet

4 participants

@bkabrda

Fixes #15

@josh
Owner

I just removed them.

@josh josh closed this Jul 17, 2012
@bkabrda

I think that keeping them would be a good idea, at least as kind of smoketests. Is that a problem?

@josh
Owner

This gem hasn't changed in over a year. And it will unlikely never until CoffeeScript 2.

@michaelficarra

I just removed them.

Best response to a failing test, ever.

Tests failing? Remove them.

@josh
Owner

Also the tests were originally written because we supported multiple backends. Thats all been extract to execjs which has plenty of test cases.

@bkabrda

I still think that even simple tests have their meaning. At least they tell you that the library gets loaded properly, there is not syntax error, etc - all that by running a single command. Is it really a problem to keep these few tests?

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