/simmer

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

Complete test coverage #12

Closed
Enchufa2 opened this Issue Nov 16, 2015 · 5 comments

Comments

Assignees

@Enchufa2 Enchufa2

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@Enchufa2 @Bart6114
@Enchufa2
Member

Enchufa2 commented Nov 16, 2015

Working on it.

@Enchufa2 Enchufa2 self-assigned this Nov 16, 2015

@Bart6114

This comment has been minimized.

Sign in to view
Member

Bart6114 commented Nov 16, 2015

at 97.19% ;)
@Enchufa2

This comment has been minimized.

Sign in to view
Member

Enchufa2 commented Nov 16, 2015

This is the only file that reports lines without coverage, but I'm not sure if there's something we can do about them, as it is an auto-generated file...
@Enchufa2

This comment has been minimized.

Sign in to view
Member

Enchufa2 commented Nov 16, 2015

Mmmh... and covr reports a 100 % covered. It seems a kind of bug from the Coveralls side. So I consider it done.

@Enchufa2 Enchufa2 closed this Nov 16, 2015

@Enchufa2 Enchufa2 reopened this Nov 19, 2015

@Enchufa2

This comment has been minimized.

Sign in to view
Member

Enchufa2 commented Nov 19, 2015

Ok, this wasn't a bug. What happened is that END_RCPP is a macro that encloses a catch{} block. So, now, I force some functions to fail with some tricks in order to get a complete coverage. 100 % 😄

@Enchufa2 Enchufa2 closed this Nov 19, 2015

@Bart6114

This comment has been minimized.

Sign in to view
Member

Bart6114 commented Nov 19, 2015

Nice :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment