Skip to content
This repository has been archived by the owner on Nov 11, 2020. It is now read-only.

Fix Rcov #7

Closed
builddoctor opened this issue Mar 7, 2012 · 3 comments
Closed

Fix Rcov #7

builddoctor opened this issue Mar 7, 2012 · 3 comments

Comments

@builddoctor
Copy link
Collaborator

Perhaps we bin metric_fu's bundled rcov, but coverage is good

@KushalP
Copy link
Contributor

KushalP commented Mar 7, 2012

From my previous experiments last week, I've now gone for the "someone else must have seen it" approach.

Turns out, it's fairly common for people to experience this issue with rcov when using rspec 2.0. Link: http://stackoverflow.com/questions/8886258/rcov-for-rspec-2-not-detecting-coverage-correctly-not-rails

Sadly, most of the solutions provided tend to be along the lines of "upgrade to 1.9 and use simplecov".

KushalP added a commit that referenced this issue Mar 7, 2012
- Alphabetise dev deps
- Directly use ~> 1.0.0 for rcov instead of calling it from under
  metric_fu
KushalP added a commit that referenced this issue Mar 7, 2012
This still doesn't fix the rcov usage, but it at least shows better
ouput, removing the gems coverage. Only showing the coverage for files
in lib/

Also removed rcov from being one of the outputs for metric_fu
@builddoctor
Copy link
Collaborator Author

The Ruby Community's default answer is to upgrade.

On Wed, Mar 7, 2012 at 12:44 PM, Kushal Pisavadia <
reply@reply.github.com

wrote:

From my previous experiments last week, I've now gone for the "someone
else must have seen it" approach.

Turns out, it's fairly common for people to experience this issue with
rcov when using rspec 2.0. Link:
http://stackoverflow.com/questions/8886258/rcov-for-rspec-2-not-detecting-coverage-correctly-not-rails

Sadly, most of the solutions provided tend to be along the lines of
"upgrade to 1.9 and use simplecov".


Reply to this email directly or view it on GitHub:
#7 (comment)

Julian Simpson
The Build Doctor Ltd.
http://www.build-doctor.com
julian@build-doctor.com
(+44) 207 183 0323

@KushalP KushalP closed this as completed in fe58900 Mar 7, 2012
@builddoctor
Copy link
Collaborator Author

nice work. Coverage didn't suffer too badly, either.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant