search

cohere-coop / nourish.party

Celebrating and nourishing creative communities
Other
6 stars 0 forks source link

Include Cucumber coverage in Coveralls #66

Closed zspencer closed 6 years ago

zspencer commented 6 years ago

This is because I got tired of seeing all the "uncovered" bits when there was coverage at the feature level and I didn't want to duplicate the intent of the feature tests with request tests :-p.

Thoughts?

coveralls commented 6 years ago

Coverage Status

Coverage increased (+36.1%) to 70.588% when pulling 86759f245407511ba7eed4c1e9eb9404fd8c65ce on zs-add-cucumber-to-coveralls into b92f067879d456f8808d3776b06f9cba6f08b22c on primary.

bhaibel commented 6 years ago

I approve of measuring coverage based on what is tested rather than trying to define a strict hierarchy of "what counts as a test."