Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Should simplecov work with chefspec? #19

Closed
jperry opened this issue Mar 26, 2012 · 10 comments
Closed

Should simplecov work with chefspec? #19

jperry opened this issue Mar 26, 2012 · 10 comments

Comments

@jperry
Copy link

jperry commented Mar 26, 2012

Configuring and using simplecov with tests that use chefspec don't seem to show up as being executed. Is this supported?

@acrmp
Copy link

acrmp commented Mar 28, 2012

Hi Jay,

I haven't done this but agree that it would be awesome.

You'll need to configure simplecov with additional filters for standard recipe directories. If you get a working configure block let's ship it as an adapter so others can use it.

@jperry
Copy link
Author

jperry commented Mar 28, 2012

Sounds good. I'll keep you posted. Thanks!

@ranjib
Copy link
Contributor

ranjib commented Jul 18, 2012

Any update on this?

@ranjib
Copy link
Contributor

ranjib commented Sep 19, 2012

I think we should close this ticket. Most of the current coverage tools (e.g simplecov) uses the coverage stdlib. Which provides information on which files are read (which lines, etc). But the way chef works , it reads the entire recipe and evals it. Unless chef require's the file , coverage data for a recipe wont be generated, and i dont think thats gonna happen,

@jperry
Copy link
Author

jperry commented Jun 21, 2013

Hi @ranjib,

I just wanted to revisit and see if anything has changed in Chef to possibly allow a code coverage tool to work.

Thanks,
Jay

@sethvargo
Copy link
Contributor

@jperry this would need to change in Chef. I would open a ticket thete

@sethvargo
Copy link
Contributor

(tickets.opscode.com)

@jperry
Copy link
Author

jperry commented Jun 21, 2013

thanks!

@sethvargo
Copy link
Contributor

@jperry something like this is now present

@runwaldarshu
Copy link

I am seeing an issue where even though I have tests for library helper simplecov coverage does not recognize it. Coverage on recipe specs looks good though. It looks like similar issue.
Can you provide the opscode ticket number you had open for this?

@lock lock bot locked as resolved and limited conversation to collaborators Aug 4, 2018
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

5 participants