refactor(Bench): Remove BenchOptions DTO

Author: roxblnfk

src/Bench/BenchOptions.php

@@ -23,7 +23,25 @@ public function __construct(
          *      It might be a callable or an array with class name and non-public method name.
          */
         public readonly array $callables,
+
+        /**
+         * @var array Arguments to pass to the benchmarked functions.
+         */
         public readonly array $arguments = [],
-        public readonly BenchOptions $options = new BenchOptions(),
-    ) {}
+
+        /**
+         * @var int<1, max> Number of iterations to run for each benchmark.
+         */
+        public readonly int $iterations = 1000,
+
+        /**
+         * @var int<1, max> Number of revolutions to run for each benchmark.
+         *      A revolution is a single execution of the benchmarked function.
+         */
+        public readonly int $revolutions = 5,
+    ) {
+        \count($callables) < 1 or throw new \InvalidArgumentException('At least one callable must be provided.');
+        $iterations > 0 or throw new \InvalidArgumentException('Iterations must be greater than 0.');
+        $revolutions > 0 or throw new \InvalidArgumentException('Revolutions must be greater than 0.');
+    }
 }

src/Bench/BenchWith.php

@@ -39,7 +39,7 @@ public function __invoke(TestInfo $info): mixed
 
         Benchmark::start()
             ->withoutData()
-            ->iterations($attr->options->iterations)
+            ->iterations($attr->iterations)
             ->compare(
                 ...$functions,
             );

src/Bench/Internal/BenchInvoker.php

@@ -4,7 +4,6 @@
 
 namespace Tests\Bench\Self;
 
-use Testo\Bench\BenchOptions;
 use Testo\Bench\BenchWith;
 
 final class BenchWithAttr
@@ -15,9 +14,7 @@ final class BenchWithAttr
             [self::class, 'sumSlow'],
         ],
         arguments: [1, 2],
-        options: new BenchOptions(
-            iterations: 10_000,
-        )
+        iterations: 10_000,
     )]
     public static function sumFast(int $a, int $b): int
     {