deep-cover: Fail to require deep-cover/builtin_takeover 0.5.2

Trying to migrate from simplecov but cannot require builtin_takeover.

Reproducing the issue: Here is a minimal Rails app that reproduces the issue: https://github.com/stripedpumpkin/taking_cover

The issue:

bundle exec rake test

requiring deep-cover builtin_takeover
rake aborted!
LoadError: cannot load such file -- deep-cover/builtin_takeover
/home/pumpkin/playapps/taking_cover/test/test_helper.rb:2:in `require'
/home/pumpkin/playapps/taking_cover/test/test_helper.rb:2:in `<top (required)>'
/home/pumpkin/playapps/taking_cover/test/models/donut_test.rb:1:in `require'
/home/pumpkin/playapps/taking_cover/test/models/donut_test.rb:1:in `<top (required)>'
/home/pumpkin/.rbenv/versions/2.3.1/bin/bundle:23:in `load'
/home/pumpkin/.rbenv/versions/2.3.1/bin/bundle:23:in `<main>'
Tasks: TOP => test:run
(See full trace by running task with --trace)

About this issue

Original URL
State: closed
Created 6 years ago
Comments: 57 (27 by maintainers)

Commits related to this issue

Fix 'right-hand' instead of 'left-hand' [#28] — committed to deep-cover/deep-cover by marcandre 6 years ago
Handle partially executed branches [#28] — committed to deep-cover/deep-cover by marcandre 6 years ago

Most upvoted comments

Good news, we finally observed the failure on a branch that did not have any of the deep-cover related changes. So we have randomly freezing ruby builds anyway so we should be able to merge the deep-cover part.

stripedpumpkin on Jun 5, 2018

Faster, indeed. The boot time on the production app has been cut down from 100-120s to 16-17s. It looks like app/admin files are now covered as well. Thanks for all the improvements. I will look into all that precisely on Monday, but it is looking great.

stripedpumpkin on Jun 2, 2018