istambul, ibrik, coverage, etc

[tardyp]

Hi Cary,

Do you have any method, on how you would do test coverage with karma and ng-classify?

I can get coverage with the compiled js, and I'm trying to do the same with the coffee script file directly, to get a nice report, with annotated coffeescript files, but I miss the karma-ng-classify-preprocessor...

[tardyp]

I manage to move forward by preprocessing the coffee using ng-classify only.

the next problem is that ibrik uses https://github.com/michaelficarra/CoffeeScriptRedux

ng-classify generates ambiguous coffeescript:

angular.module "app"
.directive("foo", Foo)

can be either interpreted as

angular.module("app".directive("foo", Foo))

or

angular.module("app").directive("foo", Foo)

coffeescript-redux interprets the former, and thus the generated js is wrong.
I had to hack in moduleOptions.coffee to move forward, and finally get my coverage.

[CaryLandholt]

Looks like it depends on the version of CoffeeScript being used.
ng-classify uses 1.7.1 and compiles correctly.

See the demo here.

I'm looking into it further, however.

[CaryLandholt]

The Redux project hasn't upgraded to 1.7.0 yet.
michaelficarra/CoffeeScriptRedux#279

[CaryLandholt]

Pierre, I've ensured the CoffeeScript ng-classify emits cannot be considered ambiguous by even older versions of the CoffeeScript compiler.

[tardyp]

Thanks you very much for the fast fix!

[CaryLandholt]

Pierre, I've created an ng-classify Atom package, in case you're interested.

ng-classify-preview