This is an automated email from the ASF dual-hosted git repository. aherbert pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/commons-statistics.git
commit 2e4f084462d18c1bc09d555dc00e9977147f80eb Author: aherbert <[email protected]> AuthorDate: Tue Jun 23 17:04:06 2020 +0100 Increase LogisticsDistribution test coverage. --- .../distribution/LogisticsDistributionTest.java | 64 +++++++++++++--------- 1 file changed, 39 insertions(+), 25 deletions(-) diff --git a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/LogisticsDistributionTest.java b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/LogisticsDistributionTest.java index 310d8d8..ecf45fc 100644 --- a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/LogisticsDistributionTest.java +++ b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/LogisticsDistributionTest.java @@ -25,31 +25,6 @@ import org.junit.jupiter.api.Test; */ public class LogisticsDistributionTest extends ContinuousDistributionAbstractTest { - @Test - public void testParameters() { - LogisticDistribution d = makeDistribution(); - Assertions.assertEquals(2, d.getLocation(), Precision.EPSILON); - Assertions.assertEquals(5, d.getScale(), Precision.EPSILON); - } - - @Test - public void testSupport() { - LogisticDistribution d = makeDistribution(); - Assertions.assertTrue(Double.isInfinite(d.getSupportLowerBound())); - Assertions.assertTrue(Double.isInfinite(d.getSupportUpperBound())); - Assertions.assertTrue(d.isSupportConnected()); - } - - @Test - public void testMeanAndVariance() { - LogisticDistribution d = makeDistribution(); - // Constructor 'location' parameter = mean - Assertions.assertEquals(2.0, d.getMean()); - // Variance = (s^2 * pi^2) / 3 - // Constructor 'scale' parameter = s - Assertions.assertEquals(5 * 5 * Math.PI * Math.PI / 3, d.getVariance()); - } - @Override public LogisticDistribution makeDistribution() { return new LogisticDistribution(2, 5); @@ -77,4 +52,43 @@ public class LogisticsDistributionTest extends ContinuousDistributionAbstractTes 0.4501660, 0.5000000, 0.5498340, 0.5986877, 0.6456563 }; } + + //---------------------------- Additional test cases ------------------------- + + @Test + public void testInverseCumulativeProbabilityExtremes() { + setInverseCumulativeTestPoints(new double[] {0, 1}); + setInverseCumulativeTestValues(new double[] {0, Double.POSITIVE_INFINITY}); + verifyInverseCumulativeProbabilities(); + } + + @Test + public void testConstructorPrecondition1() { + Assertions.assertThrows(IllegalArgumentException.class, () -> new LogisticDistribution(1, 0)); + } + + @Test + public void testParameters() { + LogisticDistribution d = makeDistribution(); + Assertions.assertEquals(2, d.getLocation(), Precision.EPSILON); + Assertions.assertEquals(5, d.getScale(), Precision.EPSILON); + } + + @Test + public void testSupport() { + LogisticDistribution d = makeDistribution(); + Assertions.assertTrue(Double.isInfinite(d.getSupportLowerBound())); + Assertions.assertTrue(Double.isInfinite(d.getSupportUpperBound())); + Assertions.assertTrue(d.isSupportConnected()); + } + + @Test + public void testMeanAndVariance() { + LogisticDistribution d = makeDistribution(); + // Constructor 'location' parameter = mean + Assertions.assertEquals(2.0, d.getMean()); + // Variance = (s^2 * pi^2) / 3 + // Constructor 'scale' parameter = s + Assertions.assertEquals(5 * 5 * Math.PI * Math.PI / 3, d.getVariance()); + } }
