yeoman / generator-karma

Yeoman generator for Karma
http://yeoman.io
BSD 2-Clause "Simplified" License
172 stars 48 forks source link

Additional Karma dependencies needed for grunt test (with generator-angular) #47

Closed karlhorky closed 10 years ago

karlhorky commented 10 years ago

As noted in yeoman/generator-angular#614, there are missing Karma dependencies for running the tests with a new generator-angular project. This pull request includes the missing dependencies from @BirdInTheCity's comment and a few more required to fix the tests in a CoffeeScript project.

seriema commented 10 years ago

This should also fix https://github.com/yeoman/generator-karma/issues/44

eddiemonge commented 10 years ago

I want to close this as it doesn't seem to be the best solution. A better solution would be to have a flag that you can pass in these things to so not everything is installed by default, which is the reason the Karma team made this change.

karlhorky commented 10 years ago

Right, that makes sense. This would need to be built into the angular generator as well then, correct? Can you keep us updated on when there's a fix in generator-karma @eddiemonge?

eddiemonge commented 10 years ago

Right, anything relying on this would have to update to use the new flag. Generator-angular wouldn't be a problem to update