search

jejacks0n / teaspoon

Teaspoon: Javascript test runner for Rails. Use Selenium, BrowserStack, or PhantomJS.
1.43k stars 243 forks source link

Fix the bug where coverage isn't run from default rake task #262

Closed tpbowden closed 10 years ago

tpbowden commented 10 years ago

When running the default Rake task, the Sprockets environment (set in teaspoon/suite.rb) is an instance of Sprockets::Index, but the Instrumentation code is only included in Sprockets::Environment. By adding the module to both, coverage will run regardless of how it is called.

coveralls commented 10 years ago

Coverage Status

Coverage remained the same when pulling e2a51f48cda5f007758d9708e78461bb4d5d179d on tpbowden:master into 9e022e7012234feb72233e889d03569b9e413889 on modeset:master.

jejacks0n commented 10 years ago

Thanks for sorting that out.. that was a really useful contribution. =)