Allow run-path to overwrite absolute path

In order to not use absolute path references for symlinks that point to
binaries outside the repository users can now use the run-path config
value to disable the detection behaviour and set the path to the
Captain binary manually.

Issue #219

Author: sebastianfeldmann

src/Hook/Template/Local/Shell.php

@@ -63,10 +63,6 @@ protected function getHookLines(string $hook): array
             ];
         }
 
-        $executable = $this->config->getPhpPath() === ''
-                    ? $this->pathInfo->getExecutablePath()
-                    : $this->config->getPhpPath() . ' ' . $this->pathInfo->getExecutablePath();
-
         return array_merge(
             [
                 '#!/bin/sh',
@@ -78,12 +74,28 @@ protected function getHookLines(string $hook): array
             $useTTY,
             [
                 '',
-                $executable
+                $this->getExecutable()
                     . ' $INTERACTIVE'
                     . ' --configuration=' . $this->pathInfo->getConfigPath()
                     . ' --bootstrap=' . $this->config->getBootstrap()
                     . ' hook:' . $hook . ' "$@"' . $useStdIn,
             ]
         );
     }
+
+    /**
+     * Returns the path to the executable including a configured php executable
+     *
+     * @return string
+     */
+    private function getExecutable(): string
+    {
+        $executable = !empty($this->config->getPhpPath()) ? $this->config->getPhpPath() . ' ' : '';
+
+        if (!empty($this->config->getRunPath())) {
+            return $executable . $this->config->getRunPath();
+        }
+
+        return $executable . $this->pathInfo->getExecutablePath();
+    }
 }

tests/unit/Hook/Template/Local/ShellTest.php

@@ -63,6 +63,29 @@ public function testTemplateWithDefinedPHP(): void
         $this->assertStringContainsString('vendor/bin/captainhook $INTERACTIVE', $code);
     }
 
+    /**
+     * Tests Shell::getCode
+     */
+    public function testTemplateWithDefinedPHPAndRunPath(): void
+    {
+        $pathInfo = $this->createMock(PathInfo::class);
+        $pathInfo->method('getExecutablePath')->willReturn('vendor/bin/captainhook');
+        $pathInfo->method('getConfigPath')->willReturn('captainhook.json');
+
+        $config = $this->createConfigMock(false, 'captainhook.json');
+        $config->method('getBootstrap')->willReturn('vendor/autoload.php');
+        $config->method('getPhpPath')->willReturn('/usr/bin/php7.4');
+        $config->method('getRunPath')->willReturn('tools/captainhook.phar');
+
+        $template = new Shell($pathInfo, $config, false);
+        $code     = $template->getCode('commit-msg');
+
+        $this->assertStringContainsString('#!/bin/sh', $code);
+        $this->assertStringContainsString('commit-msg', $code);
+        $this->assertStringContainsString('/usr/bin/php7.4', $code);
+        $this->assertStringContainsString('tools/captainhook.phar $INTERACTIVE', $code);
+    }
+
     /**
      * Tests Shell::getCode
      */