[GitHub] commons-numbers issue #8: Add-some-Unit-Tests

2017-07-10 Thread coveralls
Github user coveralls commented on the issue: https://github.com/apache/commons-numbers/pull/8 [![Coverage Status](https://coveralls.io/builds/12328326/badge)](https://coveralls.io/builds/12328326) Coverage increased (+3.2%) to 86.303% when pulling

[GitHub] commons-numbers pull request #8: Add-some-Unit-Tests

2017-07-10 Thread ameyjadiye
Github user ameyjadiye commented on a diff in the pull request: https://github.com/apache/commons-numbers/pull/8#discussion_r126514781 --- Diff: commons-numbers-combinatorics/src/test/java/org/apache/commons/numbers/combinatorics/FactorialDoubleTest.java --- @@ -121,10 +124,31 @@

[GitHub] commons-numbers pull request #8: Add-some-Unit-Tests

2017-07-10 Thread ameyjadiye
Github user ameyjadiye commented on a diff in the pull request: https://github.com/apache/commons-numbers/pull/8#discussion_r126515495 --- Diff: commons-numbers-combinatorics/src/test/java/org/apache/commons/numbers/combinatorics/LogFactorialTest.java --- @@ -115,4 +116,20 @@