[POOL-244] [PATCH] Add more tests to [pool]

• Type: Test
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 2.1
• Labels: coverage patch tests
• Assignee: Unassigned
• Reporter: Bruno P. Kinoshita
• Created: 19/Dec/13 13:01
• Updated: 31/Dec/14 16:50
• Resolved: 20/Dec/13 11:03

Description

Hi!

I read a new version of [pool] was being prepared to be released and decided to chime in and add a few more tests.

With these new tests, I believe the line coverage is above 70% is almost all the inner/classes but one that I thought wasn't worth writing more tests.

In future dev cycles I hope to be able to increase the branch coverage to somewhere near 70% too, as well as fix some of the TODO tasks in the test code.

I found a few trivial things that could be changed I think (typos, unreachable if's and naming inconsistencies) but I'll create separate issues for each one.

Please, feel free to massage the code as much as needed or drop any parts if needed.

Hope that helps,
Bruno