Merge pull request #69 from TheDragonCode/4.x

[4.x] Add snapshots for `RoundTest` with multiple data sets for improved test validation

Author: andrey-helldar

src/Services/AssertService.php

@@ -4,7 +4,7 @@
 
 namespace DragonCode\Benchmark\Services;
 
-use function assert;
+use AssertionError;
 
 class AssertService
 {
@@ -111,22 +111,34 @@ public function toBeTotalMemory(?float $from = null, ?float $till = null): stati
         return $this;
     }
 
-    protected function assertGreaterThan(?float $actual, ?float $expected, string $name): void
+    protected function assertGreaterThan(float $actual, ?float $expected, string $name): void
     {
         if ($expected === null) {
             return;
         }
 
-        assert($actual >= $expected, "The $name value must be greater than or equal to $expected.");
+        if ($actual >= $expected) {
+            return;
+        }
+
+        throw new AssertionError(
+            "The $name value must be greater than or equal to $expected."
+        );
     }
 
-    protected function assertLessThan(?float $actual, ?float $expected, string $name): void
+    protected function assertLessThan(float $actual, ?float $expected, string $name): void
     {
         if ($expected === null) {
             return;
         }
 
-        assert($actual <= $expected, "The $name value must be less than or equal to $expected.");
+        if ($actual <= $expected) {
+            return;
+        }
+
+        throw new AssertionError(
+            "The $name value must be less than or equal to $expected."
+        );
     }
 
     protected function resolveFrom(?float $from, ?float $till): ?float

tests/.pest/snapshots/Unit/Round/RoundTest/round_with_data_set___0____0_.snap

@@ -0,0 +1,10 @@
++-------+--------------------+-------------------+
+| #     | 0                  | 1                 |
++-------+--------------------+-------------------+
+| min   | 16 ms - 202 bytes  | 2 ms - 102 bytes  |
+| max   | 113 ms - 209 bytes | 10 ms - 109 bytes |
+| avg   | 53 ms - 205 bytes  | 6 ms - 105 bytes  |
+| total | 424 ms - 1.61 KB   | 48 ms - 844 bytes |
++-------+--------------------+-------------------+
+| order | 2                  | 1                 |
++-------+--------------------+-------------------+

tests/.pest/snapshots/Unit/Round/RoundTest/round_with_data_set___2____2_.snap

@@ -0,0 +1,10 @@
++-------+-----------------------+----------------------+
+| #     | 0                     | 1                    |
++-------+-----------------------+----------------------+
+| min   | 15.68 ms - 202 bytes  | 2.35 ms - 102 bytes  |
+| max   | 112.79 ms - 209 bytes | 9.76 ms - 109 bytes  |
+| avg   | 53.03 ms - 205 bytes  | 5.94 ms - 105 bytes  |
+| total | 424.2 ms - 1.61 KB    | 47.54 ms - 844 bytes |
++-------+-----------------------+----------------------+
+| order | 2                     | 1                    |
++-------+-----------------------+----------------------+

tests/.pest/snapshots/Unit/Round/RoundTest/round_with_data_set___5____5_.snap

@@ -0,0 +1,10 @@
++-------+--------------------------+------------------------+
+| #     | 0                        | 1                      |
++-------+--------------------------+------------------------+
+| min   | 15.6789 ms - 202 bytes   | 2.34568 ms - 102 bytes |
+| max   | 112.78901 ms - 209 bytes | 9.75679 ms - 109 bytes |
+| avg   | 53.02525 ms - 205 bytes  | 5.9429 ms - 105 bytes  |
+| total | 424.20199 ms - 1.61 KB   | 47.5432 ms - 844 bytes |
++-------+--------------------------+------------------------+
+| order | 2                        | 1                      |
++-------+--------------------------+------------------------+

tests/Unit/Assertions/ToBeAvgMemoryTest.php

@@ -18,13 +18,13 @@
     expect(true)->toBeTrue();
 });
 
-test('failure less than', function () {
+test('failure greater than', function () {
     benchmark()
         ->assert()
         ->toBeAvgMemory(from: 10000);
 })->throws(AssertionError::class, 'The average memory value must be greater than or equal to 10000.');
 
-test('failure greater than', function () {
+test('failure less than', function () {
     benchmark()
         ->assert()
         ->toBeAvgMemory(till: 1);

tests/Unit/Assertions/ToBeAvgTimeTest.php

@@ -18,13 +18,13 @@
     expect(true)->toBeTrue();
 });
 
-test('failure less than', function () {
+test('failure greater than', function () {
     benchmark()
         ->assert()
         ->toBeAvgTime(from: 1000);
 })->throws(AssertionError::class, 'The average time value must be greater than or equal to 1000.');
 
-test('failure greater than', function () {
+test('failure less than', function () {
     benchmark()
         ->assert()
         ->toBeAvgTime(till: 1);

tests/Unit/Assertions/ToBeMaxMemoryTest.php

@@ -18,13 +18,13 @@
     expect(true)->toBeTrue();
 });
 
-test('failure less than', function () {
+test('failure greater than', function () {
     benchmark()
         ->assert()
         ->toBeMaxMemory(from: 10000);
 })->throws(AssertionError::class, 'The maximum memory value must be greater than or equal to 10000.');
 
-test('failure greater than', function () {
+test('failure less than', function () {
     benchmark()
         ->assert()
         ->toBeMaxMemory(till: 1);

tests/Unit/Assertions/ToBeMaxTimeTest.php

@@ -18,13 +18,13 @@
     expect(true)->toBeTrue();
 });
 
-test('failure less than', function () {
+test('failure greater than', function () {
     benchmark()
         ->assert()
         ->toBeMaxTime(from: 1000);
 })->throws(AssertionError::class, 'The maximum time value must be greater than or equal to 1000.');
 
-test('failure greater than', function () {
+test('failure less than', function () {
     benchmark()
         ->assert()
         ->toBeMaxTime(till: 1);

tests/Unit/Assertions/ToBeMinMemoryTest.php

@@ -18,13 +18,13 @@
     expect(true)->toBeTrue();
 });
 
-test('failure less than', function () {
+test('failure greater than', function () {
     benchmark()
         ->assert()
         ->toBeMinMemory(from: 10000);
 })->throws(AssertionError::class, 'The minimum memory value must be greater than or equal to 10000.');
 
-test('failure greater than', function () {
+test('failure less than', function () {
     benchmark()
         ->assert()
         ->toBeMinMemory(till: 1);

tests/Unit/Assertions/ToBeMinTimeTest.php

@@ -18,13 +18,13 @@
     expect(true)->toBeTrue();
 });
 
-test('failure less than', function () {
+test('failure greater than', function () {
     benchmark()
         ->assert()
         ->toBeMinTime(from: 1000);
 })->throws(AssertionError::class, 'The minimum time value must be greater than or equal to 1000.');
 
-test('failure greater than', function () {
+test('failure less than', function () {
     benchmark()
         ->assert()
         ->toBeMinTime(till: 1);