Added simple check for glob paths in config.path value (#951)

Author: ricardoboss

lib/Extension/RunnerExtension.php

@@ -518,16 +518,36 @@ private function relativizeConfigPath(Container $container): void
             return;
         }
 
-        $container->setParameter(self::PARAM_PATH, array_map(function (string $path) use ($container) {
+        $normalizedPaths = array_map(static function (string $path) use ($container) {
             if (Path::isAbsolute($path)) {
-                return $path;
+                return [$path];
             }
 
-            return Path::join([
-                dirname($container->getParameter(CoreExtension::PARAM_CONFIG_PATH)),
-                $path
-            ]);
-        }, $paths));
+            if (str_contains($path, '*') || str_contains($path, '?')) {
+                $globPaths = glob($path, GLOB_NOSORT);
+
+                if (empty($globPaths)) {
+                    return [];
+                }
+
+                return array_map(static function (string $path) use ($container) {
+                    return Path::join([
+                        dirname($container->getParameter(CoreExtension::PARAM_CONFIG_PATH)),
+                        $path
+                    ]);
+                }, $globPaths);
+            }
+
+            return [
+                Path::join([
+                    dirname($container->getParameter(CoreExtension::PARAM_CONFIG_PATH)),
+                    $path
+                ])
+            ];
+        }, $paths);
+        $flattenedPaths = array_merge(...$normalizedPaths);
+
+        $container->setParameter(self::PARAM_PATH, $flattenedPaths);
     }
 
     private function registerExecutors(Container $container): void

tests/System/RunTest.php

@@ -17,7 +17,7 @@
 class RunTest extends SystemTestCase
 {
     /**
-     * It should use a speified, valid, configuration.
+     * It should use a specified, valid, configuration.
      */
     public function testSpecifiedConfig(): void
     {
@@ -78,6 +78,17 @@ public function testCommandWithNoPath(): void
         $this->assertStringContainsString('You must either specify', $process->getErrorOutput());
     }
 
+    /**
+     * It should resolve glob paths to match multiple files.
+     */
+    public function testCommandWithGlobPath(): void
+    {
+        $process = $this->phpbench('run benchmarks/set7/*/');
+        $this->assertExitCode(0, $process);
+        $this->assertStringContainsString('benchNothing2', $process->getErrorOutput());
+        $this->assertStringContainsString('benchNothing3', $process->getErrorOutput());
+    }
+
     public function testCommandWithMultiplePaths(): void
     {
         $process = $this->phpbench('run benchmarks/set4/NothingBench.php benchmarks/set1/BenchmarkBench.php');

tests/System/benchmarks/set7/nested1/Nothing2Bench.php

@@ -0,0 +1,20 @@
+<?php
+
+/*
+ * This file is part of the PHPBench package
+ *
+ * (c) Daniel Leech <daniel@dantleech.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ *
+ */
+
+namespace PhpBench\Tests\System\benchmarks\set7\nested1;
+
+class Nothing2Bench
+{
+    public function benchNothing2(): void
+    {
+    }
+}

tests/System/benchmarks/set7/nested2/Nothing3Bench.php

@@ -0,0 +1,20 @@
+<?php
+
+/*
+ * This file is part of the PHPBench package
+ *
+ * (c) Daniel Leech <daniel@dantleech.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ *
+ */
+
+namespace PhpBench\Tests\System\benchmarks\set7\nested2;
+
+class Nothing3Bench
+{
+    public function benchNothing3(): void
+    {
+    }
+}

tests/System/env/config_set7/phpbench.json

@@ -0,0 +1,4 @@
+{
+    "runner.bootstrap": "../../bootstrap/foobar.bootstrap",
+    "runner.path": "../../benchmarks/set7/*/"
+}