curationexperts / laevigata

A Hyrax-based ETD repository including deposit workflow
Other
5 stars 4 forks source link

[REFACTOR] Set coverage reporting default to off #2363

Closed mark-dce closed 2 months ago

mark-dce commented 2 months ago

ISSUE Rather than having to always set an environment variable to disable coverage if we don't want it; change the configuration so that coverage reporting is only generated during the test suite when it is explicitly requested, e.g.

COVERAGE=true bundle exec rspec

Otherwise, run the test suite without generating a coverage report.

coveralls commented 2 months ago

Coverage Status

coverage: 93.37%. remained the same when pulling 694a7ba2b396faf5df5a7f4ef609c38de6336eb9 on coverage into 7b0e80d75f0c119b3af06bd43ca4e0ab662e540f on main.