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 7beb9aaa639888d11777dc87d9e3234dbdff78fe Author: Alex Herbert <aherb...@apache.org> AuthorDate: Mon Nov 21 18:19:20 2022 +0000 Test method naming conventions --- .../HypergeometricDistributionTest.java | 6 ++-- .../distribution/ParetoDistributionTest.java | 7 +++- .../distribution/TrapezoidalDistributionTest.java | 2 +- .../distribution/ZipfDistributionTest.java | 41 ++++++++++++++-------- 4 files changed, 36 insertions(+), 20 deletions(-) diff --git a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/HypergeometricDistributionTest.java b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/HypergeometricDistributionTest.java index 34ecdb6..b5c72c7 100644 --- a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/HypergeometricDistributionTest.java +++ b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/HypergeometricDistributionTest.java @@ -180,7 +180,7 @@ class HypergeometricDistributionTest extends BaseDiscreteDistributionTest { } @Test - void testZeroTrial() { + void testZeroTrials() { final int n = 11; // population final int m = 4; // successes in population final int s = 0; // number of trials @@ -221,7 +221,7 @@ class HypergeometricDistributionTest extends BaseDiscreteDistributionTest { } @Test - void testHighPrecisionCumulativeProbabilities() { + void testAdditionalCumulativeProbabilityHighPrecision() { // computed using R version 3.4.4 testCumulativeProbabilityHighPrecision( HypergeometricDistribution.of(500, 70, 300), @@ -231,7 +231,7 @@ class HypergeometricDistributionTest extends BaseDiscreteDistributionTest { } @Test - void testHighPrecisionSurvivalProbabilities() { + void testAdditionalSurvivalProbabilityHighPrecision() { // computed using R version 3.4.4 testSurvivalProbabilityHighPrecision( HypergeometricDistribution.of(500, 70, 300), diff --git a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/ParetoDistributionTest.java b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/ParetoDistributionTest.java index 38450cf..9319717 100644 --- a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/ParetoDistributionTest.java +++ b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/ParetoDistributionTest.java @@ -69,13 +69,18 @@ class ParetoDistributionTest extends BaseContinuousDistributionTest { } @Test - void testHighPrecision() { + void testAdditionalCumulativeProbabilityHighPrecision() { final ParetoDistribution dist = ParetoDistribution.of(2.1, 1.4); testCumulativeProbabilityHighPrecision( dist, new double[] {2.100000000000001, 2.100000000000005}, new double[] {6.217248937900875e-16, 3.2640556923979585e-15}, createHighPrecisionTolerance()); + } + + @Test + void testAdditionalSurvivalProbabilityHighPrecision() { + final ParetoDistribution dist = ParetoDistribution.of(2.1, 1.4); testSurvivalProbabilityHighPrecision( dist, new double[] {42e11, 64e11}, diff --git a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TrapezoidalDistributionTest.java b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TrapezoidalDistributionTest.java index 2670124..b14a605 100644 --- a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TrapezoidalDistributionTest.java +++ b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TrapezoidalDistributionTest.java @@ -107,7 +107,7 @@ class TrapezoidalDistributionTest extends BaseContinuousDistributionTest { * probability is high precision. */ @Test - void testHighPrecisionSurvivalProbabilities() { + void testAdditionalSurvivalProbabilityHighPrecision() { final double a = 0; final double b = 0; final double c = 1; diff --git a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/ZipfDistributionTest.java b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/ZipfDistributionTest.java index e3675f8..2a7ff45 100644 --- a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/ZipfDistributionTest.java +++ b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/ZipfDistributionTest.java @@ -17,10 +17,14 @@ package org.apache.commons.statistics.distribution; +import java.util.stream.Stream; import org.apache.commons.math3.util.MathArrays; import org.apache.commons.rng.simple.RandomSource; import org.junit.jupiter.api.Assertions; import org.junit.jupiter.api.Test; +import org.junit.jupiter.params.ParameterizedTest; +import org.junit.jupiter.params.provider.Arguments; +import org.junit.jupiter.params.provider.MethodSource; /** * Test cases for {@link ZipfDistribution}. @@ -51,21 +55,28 @@ class ZipfDistributionTest extends BaseDiscreteDistributionTest { //-------------------- Additional test cases ------------------------------- - @Test - void testHighPrecisionSurvivalProbabilities() { + @ParameterizedTest + @MethodSource + void testAdditionlSurvivalProbabilityHighPrecision(int n, double e, int[] x, double[] expected, DoubleTolerance tol) { + testSurvivalProbabilityHighPrecision(ZipfDistribution.of(n, e), x, expected, tol); + } + + static Stream<Arguments> testAdditionlSurvivalProbabilityHighPrecision() { // computed using scipy.stats (1.7.1) zipfian - testSurvivalProbabilityHighPrecision(ZipfDistribution.of(60, 10), - new int[] {57, 59}, - new double[] {2.3189337454689757e-18, 1.6521739576668957e-18}, - DoubleTolerances.absolute(1e-25)); - testSurvivalProbabilityHighPrecision(ZipfDistribution.of(60, 50.5), - new int[] {57, 59}, - new double[] {8.8488396450491320e-90, 1.5972093932264611e-90}, - DoubleTolerances.absolute(1e-95)); - testSurvivalProbabilityHighPrecision(ZipfDistribution.of(60, 100.5), - new int[] {57, 59}, - new double[] {5.9632998443758656e-178, 1.9760564023408183e-179}, - DoubleTolerances.absolute(1e-185)); + return Stream.of( + Arguments.of(60, 10, + new int[] {57, 59}, + new double[] {2.3189337454689757e-18, 1.6521739576668957e-18}, + DoubleTolerances.absolute(1e-25)), + Arguments.of(60, 50.5, + new int[] {57, 59}, + new double[] {8.8488396450491320e-90, 1.5972093932264611e-90}, + DoubleTolerances.absolute(1e-95)), + Arguments.of(60, 100.5, + new int[] {57, 59}, + new double[] {5.9632998443758656e-178, 1.9760564023408183e-179}, + DoubleTolerances.absolute(1e-185)) + ); } /** @@ -74,7 +85,7 @@ class ZipfDistributionTest extends BaseDiscreteDistributionTest { * it should be a complement to the CDF value. */ @Test - void testHighPrecisionSurvivalProbabilitiesWithOverflow() { + void testAdditionalSurvivalAndCumulativeProbabilityComplement() { // Requires (x+1)^a to overflow final int n = 60; final double a = 200.5;