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.

Sign up for GitHub

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

Jump to bottom

Complete Test Coverage #10

Merged
merged 12 commits into from Aug 3, 2016
Merged

Complete Test Coverage #10

merged 12 commits into from Aug 3, 2016

Conversation

elbrujohalcon
Copy link
Member

Many functions, like wpool_queue_manager:proc_info are not tested at all.

@elbrujohalcon
Copy link
Member Author

In the process, start properly using ct:pal, ct:comment, etc. instead of writing comments in the code.

@elbrujohalcon elbrujohalcon added this to the 2.1.0 milestone Jun 3, 2016
@elbrujohalcon elbrujohalcon self-assigned this Jul 20, 2016
@elbrujohalcon elbrujohalcon mentioned this pull request Aug 2, 2016
Closed
@elbrujohalcon
Copy link
Member Author

I left wpool_fsm_process on 72% coverage. I opened #79 for that.

@elbrujohalcon
Copy link
Member Author

And I left wpool_pool in 97% or 99% coverage depending on what version of Erlang you use, exactly for that reason.

@elbrujohalcon elbrujohalcon assigned igaray and unassigned elbrujohalcon Aug 2, 2016
@elbrujohalcon elbrujohalcon merged commit f7fd214 into master Aug 3, 2016
@elbrujohalcon elbrujohalcon deleted the elbrujohalcon.10.complete_test_coverage branch August 3, 2016 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@igaray igaray

Labels
finished
Projects
None yet
Milestone
2.1.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@elbrujohalcon @igaray