Default coverage ignore regex should not include spec_helper

jejacks0n / teaspoon

Teaspoon: Javascript test runner for Rails. Use Selenium, BrowserStack, or PhantomJS.

1.43k stars 243 forks source link

Default coverage ignore regex should not include spec_helper #402

Closed andresbravog closed 9 years ago

andresbravog commented 9 years ago

Related to #401

mikepack commented 9 years ago

Hey Andrés, thanks so much for your efforts clarifying this issue. We've decided that the best default behavior is to exclude spec_helper from coverage as it remains today, and recommend that people avoid requiring application from within spec_helper. We've updated the README to reflect this caveat and provide the desired workaround: https://github.com/modeset/teaspoon#caveats