ext/random: applied fixers to improve test robustness

Author: NickSdot

ext/random/tests/01_functions/bug46587.phpt

@@ -7,12 +7,12 @@ var_dump(mt_rand(3,8));
 try {
     var_dump(mt_rand(8,3));
 } catch (ValueError $e) {
-    echo $e->getMessage(), "\n";
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 echo "Done.\n";
 ?>
 --EXPECTF--
 int(%d)
-mt_rand(): Argument #2 ($max) must be greater than or equal to argument #1 ($min)
+ValueError: mt_rand(): Argument #2 ($max) must be greater than or equal to argument #1 ($min)
 Done.

ext/random/tests/01_functions/mt_rand_basic.phpt

@@ -60,4 +60,3 @@ PASSED: range min = 100 max = 1000
 PASSED: range min = 10500 max = 1050000
 PASSED: range min = 16 max = 65536
 PASSED: range min = 256 max = 448
-

ext/random/tests/01_functions/random_bytes_error.phpt

@@ -7,16 +7,16 @@ Test error operation of random_bytes()
 try {
     $bytes = random_bytes();
 } catch (TypeError $e) {
-    echo $e->getMessage().PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 try {
     $bytes = random_bytes(0);
 } catch (Error $e) {
-    echo $e->getMessage().PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 ?>
 --EXPECT--
-random_bytes() expects exactly 1 argument, 0 given
-random_bytes(): Argument #1 ($length) must be greater than 0
+ArgumentCountError: random_bytes() expects exactly 1 argument, 0 given
+ValueError: random_bytes(): Argument #1 ($length) must be greater than 0

ext/random/tests/01_functions/random_int_error.phpt

@@ -7,23 +7,23 @@ Test error operation of random_int()
 try {
     $randomInt = random_int();
 } catch (TypeError $e) {
-    echo $e->getMessage().PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 try {
     $randomInt = random_int(42);
 } catch (TypeError $e) {
-    echo $e->getMessage().PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 try {
     $randomInt = random_int(42,0);
 } catch (Error $e) {
-    echo $e->getMessage().PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 ?>
 --EXPECT--
-random_int() expects exactly 2 arguments, 0 given
-random_int() expects exactly 2 arguments, 1 given
-random_int(): Argument #1 ($min) must be less than or equal to argument #2 ($max)
+ArgumentCountError: random_int() expects exactly 2 arguments, 0 given
+ArgumentCountError: random_int() expects exactly 2 arguments, 1 given
+ValueError: random_int(): Argument #1 ($min) must be less than or equal to argument #2 ($max)

ext/random/tests/02_engine/all_serialize_error.phpt

@@ -8,9 +8,9 @@ use Random\Engine\Mt19937;
 try {
     new Mt19937(1234, 2);
 } catch (ValueError $e) {
-    echo $e->getMessage(), PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 ?>
 --EXPECT--
-Random\Engine\Mt19937::__construct(): Argument #2 ($mode) must be either MT_RAND_MT19937 or MT_RAND_PHP
+ValueError: Random\Engine\Mt19937::__construct(): Argument #2 ($mode) must be either MT_RAND_MT19937 or MT_RAND_PHP

ext/random/tests/02_engine/mt19937_error.phpt

@@ -11,7 +11,7 @@ $referenceEngine = new PcgOneseq128XslRr64(1234);
 try {
     $engine->jump(-1);
 } catch (ValueError $e) {
-    echo $e->getMessage(), PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 for ($i = 0; $i < 10_000; $i++) {
@@ -24,5 +24,5 @@ die('success');
 
 ?>
 --EXPECT--
-Random\Engine\PcgOneseq128XslRr64::jump(): Argument #1 ($advance) must be greater than or equal to 0
+ValueError: Random\Engine\PcgOneseq128XslRr64::jump(): Argument #1 ($advance) must be greater than or equal to 0
 success

ext/random/tests/02_engine/pcgoneseq128xslrr64_jump_error.phpt

@@ -17,15 +17,15 @@ echo "Invalid data type", PHP_EOL;
 try {
     $engine = new PcgOneseq128XslRr64(1.0);
 } catch (TypeError $e) {
-    echo $e->getMessage(), PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 echo PHP_EOL, PHP_EOL;
 
 echo "Invalid string seed length", PHP_EOL;
 try {
     $engine = new PcgOneseq128XslRr64('foobar');
 } catch (ValueError $e) {
-    echo $e->getMessage(), PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 echo PHP_EOL, PHP_EOL;
 
@@ -49,11 +49,11 @@ Random string seed
 
 
 Invalid data type
-Random\Engine\PcgOneseq128XslRr64::__construct(): Argument #1 ($seed) must be of type string|int|null, float given
+TypeError: Random\Engine\PcgOneseq128XslRr64::__construct(): Argument #1 ($seed) must be of type string|int|null, float given
 
 
 Invalid string seed length
-Random\Engine\PcgOneseq128XslRr64::__construct(): Argument #1 ($seed) must be a 16 byte (128 bit) string
+ValueError: Random\Engine\PcgOneseq128XslRr64::__construct(): Argument #1 ($seed) must be a 16 byte (128 bit) string
 
 
 Valid string seed

ext/random/tests/02_engine/pcgoneseq128xslrr64_seed.phpt

@@ -17,23 +17,23 @@ echo "Invalid data type", PHP_EOL;
 try {
     $engine = new Xoshiro256StarStar(1.0);
 } catch (Throwable $e) {
-    echo $e->getMessage(), PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 echo PHP_EOL, PHP_EOL;
 
 echo "Invalid string seed length", PHP_EOL;
 try {
     $engine = new Xoshiro256StarStar('foobar');
 } catch (Throwable $e) {
-    echo $e->getMessage(), PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 echo PHP_EOL, PHP_EOL;
 
 echo "Null seed", PHP_EOL;
 try {
     $engine = new Xoshiro256StarStar(str_repeat("\x00", 32));
 } catch (Throwable $e) {
-    echo $e->getMessage(), PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 echo PHP_EOL, PHP_EOL;
 
@@ -57,15 +57,15 @@ Random string seed
 
 
 Invalid data type
-Random\Engine\Xoshiro256StarStar::__construct(): Argument #1 ($seed) must be of type string|int|null, float given
+TypeError: Random\Engine\Xoshiro256StarStar::__construct(): Argument #1 ($seed) must be of type string|int|null, float given
 
 
 Invalid string seed length
-Random\Engine\Xoshiro256StarStar::__construct(): Argument #1 ($seed) must be a 32 byte (256 bit) string
+ValueError: Random\Engine\Xoshiro256StarStar::__construct(): Argument #1 ($seed) must be a 32 byte (256 bit) string
 
 
 Null seed
-Random\Engine\Xoshiro256StarStar::__construct(): Argument #1 ($seed) must not consist entirely of NUL bytes
+ValueError: Random\Engine\Xoshiro256StarStar::__construct(): Argument #1 ($seed) must not consist entirely of NUL bytes
 
 
 Valid string seed

ext/random/tests/02_engine/xoshiro256starstar_seed.phpt

@@ -6,9 +6,9 @@ GH-21731: Xoshiro256StarStar::__unserialize() must reject the all-zero state
 try {
     var_dump(unserialize('O:32:"Random\Engine\Xoshiro256StarStar":2:{i:0;a:0:{}i:1;a:4:{i:0;s:16:"0000000000000000";i:1;s:16:"0000000000000000";i:2;s:16:"0000000000000000";i:3;s:16:"0000000000000000";}}'));
 } catch (\Exception $e) {
-    echo $e->getMessage(), PHP_EOL;
+    echo $e::class, ': ', $e->getMessage(), \PHP_EOL;
 }
 
 ?>
 --EXPECT--
-Invalid serialization data for Random\Engine\Xoshiro256StarStar object
+Exception: Invalid serialization data for Random\Engine\Xoshiro256StarStar object