Allow php-path to be an executable with arguments

You could use an executable to determine the real executable path.
For example '/usr/local/bin/find-php 7.4'. The validation method
can now handle this kind of scenarios.

Author: sebastianfeldmann

src/Config/Factory.php

@@ -195,7 +195,24 @@ private function validatePhpPath(Config $config): void
         if (empty($config->getPhpPath())) {
             return;
         }
-        if (!file_exists($config->getPhpPath())) {
+        $foundPHP    = false;
+        $pathToCheck = [$config->getPhpPath()];
+        $parts       = explode(' ', $config->getPhpPath());
+        // if there are spaces in the php-path and they are not escaped
+        // it looks like an executable is used to find the PHP binary
+        // so at least check if the executable exists
+        if (count($parts) > 1 && substr($parts[0], -1) !== '\\') {
+            $pathToCheck[] = $parts[0];
+        }
+
+        foreach ($pathToCheck as $path) {
+            if (file_exists($path)) {
+                $foundPHP = true;
+                break;
+            }
+        }
+
+        if (!$foundPHP) {
             throw new RuntimeException('The configured php-path is wrong: ' . $config->getPhpPath());
         }
     }

tests/files/config/valid-with-strange-settings.json

@@ -0,0 +1,37 @@
+{
+  "config": {
+    "php-path": "tests/files/bin/success foo"
+  },
+  "prepare-commit-msg": {
+    "enabled": true,
+    "actions": []
+  },
+  "commit-msg": {
+    "enabled": true,
+    "actions": []
+  },
+  "pre-commit": {
+    "enabled": true,
+    "actions": [
+      {
+        "action": "phpunit --configuration=build/phpunit-hook.xml",
+        "options": [],
+        "conditions": [
+          {
+            "exec": "\\CaptainHook\\App\\Hook\\Condition\\AnyFileChanged",
+            "args": [
+              ["foo.php", "bar.php"]
+            ]
+          }
+        ],
+        "config": {
+          "allow-failure": true
+        }
+      }
+    ]
+  },
+  "pre-push": {
+    "enabled": false,
+    "actions": []
+  }
+}

tests/unit/Config/FactoryTest.php

@@ -113,6 +113,18 @@ public function testCreateWithSettings(): void
         $this->assertTrue($config->getHookConfig('pre-commit')->getActions()[0]->isFailureAllowed());
     }
 
+    /**
+     * Tests Factory::create
+     *
+     * @throws \Exception
+     */
+    public function testCreateWithCrazyPHPPath(): void
+    {
+        $config = Factory::create(realpath(__DIR__ . '/../../files/config/valid-with-strange-settings.json'));
+
+        $this->assertEquals("tests/files/bin/success foo", $config->getPhpPath());
+    }
+
     /**
      * Tests Factory::create
      *