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

No coverage on .hpp files? #226

Closed
cdeterman opened this issue Sep 26, 2016 · 5 comments
Closed

No coverage on .hpp files? #226

cdeterman opened this issue Sep 26, 2016 · 5 comments

Comments

@cdeterman
Copy link

I have some c++ header files that I use extensively within my package. Yet, the resulting coveralls (or codecov) show 0% of lines covered by my tests. You can see this here primarily referring to the dynEigenMat.hpp file which is used in at least half of my unit tests. Is this something to be expected when using classes in headers or did I somehow miss something? Thanks for any assistance.

@Enchufa2
Copy link

I think this is related to #219 and #220. Please, try it out using my fork to see if #220 fixes your problem.

@cdeterman
Copy link
Author

@Enchufa2 your fork does appear to fix the problem. I look forward to the pull request, until then I will likely use your fork.

@Enchufa2
Copy link

Ok, thanks for the verification.

@Enchufa2
Copy link

@cdeterman I've verified that commit 586e65e fixed the problem, and @jimhester is going to close this issue and the PR. I'm going to remove my fork, so please switch back to Jim's repo.

@cdeterman
Copy link
Author

@Enchufa2 excellent, thank you for the update.

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

No branches or pull requests

3 participants