Improved tests.

h2/src/main/org/h2/util/MathUtils.java

@@ -104,7 +104,7 @@ public class MathUtils {
         return cachedSecureRandom;
     }
 
-    private static byte[] generateAlternativeSeed() {
+    public static byte[] generateAlternativeSeed() {
         try {
             ByteArrayOutputStream bout = new ByteArrayOutputStream();
             DataOutputStream out = new DataOutputStream(bout);

h2/src/test/org/h2/test/unit/TestMathUtils.java

@@ -10,6 +10,7 @@ import java.math.BigInteger;
 import java.util.Random;
 import org.h2.test.TestBase;
 import org.h2.test.utils.AssertThrows;
+import org.h2.util.MathUtils;
 
 /**
  * Tests math utility methods.
@@ -26,10 +27,32 @@ public class TestMathUtils extends TestBase {
     }
 
     public void test() {
+        testRandom();
         testReverse();
         testFactorial();
     }
 
+    private void testRandom() {
+        int bits = 0;
+        for (int i = 0; i < 1000; i++) {
+            bits |= 1 << MathUtils.randomInt(8);
+        }
+        assertEquals(255, bits);
+        bits = 0;
+        for (int i = 0; i < 1000; i++) {
+            bits |= 1 << MathUtils.secureRandomInt(8);
+        }
+        assertEquals(255, bits);
+        bits = 0;
+        for (int i = 0; i < 1000; i++) {
+            bits |= 1 << (MathUtils.secureRandomLong() & 7);
+        }
+        assertEquals(255, bits);
+        // just verify the method doesn't throw an exception
+        byte[] data = MathUtils.generateAlternativeSeed();
+        assertTrue(data.length > 10);
+    }
+
     private void testReverse() {
         assertEquals(Integer.reverse(0), Integer.reverse(0));
         assertEquals(Integer.reverse(Integer.MAX_VALUE), Integer.reverse(Integer.MAX_VALUE));