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 fa9d1a88d6e007de2354caefee08767b0c7c9c73
Author: Alex Herbert <aherb...@apache.org>
AuthorDate: Fri Jul 23 18:41:41 2021 +0100

    Remove public modifiers of package level TestUtils
---
 .../commons/statistics/distribution/TestUtils.java | 58 +++++++++++-----------
 1 file changed, 29 insertions(+), 29 deletions(-)

diff --git 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TestUtils.java
 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TestUtils.java
index 19920e8..1e84c83 100644
--- 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TestUtils.java
+++ 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TestUtils.java
@@ -26,7 +26,7 @@ import org.junit.jupiter.api.Assertions;
 /**
  * Test utilities.
  */
-public final class TestUtils {
+final class TestUtils {
     /**
      * Collection of static methods used in math unit tests.
      */
@@ -39,9 +39,9 @@ public final class TestUtils {
      * @param observed Observed values.
      * @param tolerance Amount of absolute error to allow.
      */
-    public static void assertEquals(double[] expected,
-                                    double[] observed,
-                                    double tolerance) {
+    static void assertEquals(double[] expected,
+                             double[] observed,
+                             double tolerance) {
         assertEquals(() -> "Array comparison failure", expected, observed, 
tolerance);
     }
 
@@ -54,9 +54,9 @@ public final class TestUtils {
      * @param actual  observed value
      * @param relativeError  maximum allowable relative error
      */
-    public static void assertRelativelyEquals(double expected,
-                                              double actual,
-                                              double relativeError) {
+    static void assertRelativelyEquals(double expected,
+                                       double actual,
+                                       double relativeError) {
         assertRelativelyEquals(null, expected, actual, relativeError);
     }
 
@@ -70,10 +70,10 @@ public final class TestUtils {
      * @param actual  observed value
      * @param relativeError  maximum allowable relative error
      */
-    public static void assertRelativelyEquals(Supplier<String> msg,
-                                              double expected,
-                                              double actual,
-                                              double relativeError) {
+    static void assertRelativelyEquals(Supplier<String> msg,
+                                       double expected,
+                                       double actual,
+                                       double relativeError) {
         if (Double.isNaN(expected)) {
             Assertions.assertTrue(Double.isNaN(actual), msg);
         } else if (Double.isNaN(actual)) {
@@ -96,10 +96,10 @@ public final class TestUtils {
      * @param observed Observed values.
      * @param tolerance Amount of absolute error to allow.
      */
-    public static void assertEquals(Supplier<String> msg,
-                                    double[] expected,
-                                    double[] observed,
-                                    double tolerance) {
+    static void assertEquals(Supplier<String> msg,
+                             double[] expected,
+                             double[] observed,
+                             double tolerance) {
         if (expected.length != observed.length) {
             final StringBuilder out = new StringBuilder(msg.get());
             out.append("\n Arrays not same length. \n");
@@ -181,10 +181,10 @@ public final class TestUtils {
      * @param observed observed counts
      * @param alpha significance level of the test
      */
-    public static void assertChiSquareAccept(int[] values,
-                                             double[] expected,
-                                             long[] observed,
-                                             double alpha) {
+    static void assertChiSquareAccept(int[] values,
+                                      double[] expected,
+                                      long[] observed,
+                                      double alpha) {
         assertChiSquare(values, expected, observed, alpha);
     }
 
@@ -196,9 +196,9 @@ public final class TestUtils {
      * @param observed observed counts
      * @param alpha significance level of the test
      */
-    public static void assertChiSquareAccept(double[] expected,
-                                             long[] observed,
-                                             double alpha) {
+    static void assertChiSquareAccept(double[] expected,
+                                      long[] observed,
+                                      double alpha) {
         final int[] values = new int[expected.length];
         for (int i = 0; i < values.length; i++) {
             values[i] = i + 1;
@@ -213,7 +213,7 @@ public final class TestUtils {
      * @param distribution Distribution.
      * @return the quartiles
      */
-    public static double[] getDistributionQuartiles(ContinuousDistribution 
distribution) {
+    static double[] getDistributionQuartiles(ContinuousDistribution 
distribution) {
         final double[] quantiles = new double[3];
         quantiles[0] = distribution.inverseCumulativeProbability(0.25d);
         quantiles[1] = distribution.inverseCumulativeProbability(0.5d);
@@ -229,7 +229,7 @@ public final class TestUtils {
      * @param counts Counts for each quartile.
      * @param quartiles Quartiles.
      */
-    public static void updateCounts(double value, long[] counts, double[] 
quartiles) {
+    static void updateCounts(double value, long[] counts, double[] quartiles) {
         if (value > quartiles[1]) {
             counts[value <= quartiles[2] ? 2 : 3]++;
         } else {
@@ -247,7 +247,7 @@ public final class TestUtils {
      * @param densityValues Density values.
      * @return number of positive mass points
      */
-    public static int eliminateZeroMassPoints(int[] densityPoints, double[] 
densityValues) {
+    static int eliminateZeroMassPoints(int[] densityPoints, double[] 
densityValues) {
         int positiveMassCount = 0;
         for (int i = 0; i < densityValues.length; i++) {
             if (densityValues[i] > 0) {
@@ -279,8 +279,8 @@ public final class TestUtils {
      * @param sampler Sampler.
      * @return an array of size {@code n}.
      */
-    public static double[] sample(int n,
-                                  ContinuousDistribution.Sampler sampler) {
+    static double[] sample(int n,
+                           ContinuousDistribution.Sampler sampler) {
         final double[] samples = new double[n];
         for (int i = 0; i < n; i++) {
             samples[i] = sampler.sample();
@@ -296,8 +296,8 @@ public final class TestUtils {
      * @param sampler Sampler.
      * @return an array of size {@code n}.
      */
-    public static int[] sample(int n,
-                               DiscreteDistribution.Sampler sampler) {
+    static int[] sample(int n,
+                        DiscreteDistribution.Sampler sampler) {
         final int[] samples = new int[n];
         for (int i = 0; i < n; i++) {
             samples[i] = sampler.sample();

Reply via email to