Allow multi-line action configs

Fixes issue #302

Author: sebastianfeldmann

src/Config/Factory.php

@@ -113,7 +113,6 @@ private function includeConfig(string $path): Config
     private function setupConfig(string $path, array $settings = []): Config
     {
         $file = new Json($path);
-
         return $file->exists()
             ? $this->loadConfigFromFile($path, $settings)
             : new Config($path, false, $settings);
@@ -199,10 +198,11 @@ private function configureHook(Config\Hook $config, array $json): void
     {
         $config->setEnabled($json['enabled'] ?? true);
         foreach ($json['actions'] as $actionJson) {
+            $action     = is_array($actionJson['action']) ? implode(' ', $actionJson['action']) : $actionJson['action'];
             $options    = $this->extractOptions($actionJson);
             $conditions = $this->extractConditions($actionJson);
             $settings   = $this->extractSettings($actionJson);
-            $config->addAction(new Config\Action($actionJson['action'], $options, $conditions, $settings));
+            $config->addAction(new Config\Action($action, $options, $conditions, $settings));
         }
     }
 

tests/_files/config/valid-with-multi-line-action.json

@@ -0,0 +1,10 @@
+{
+  "pre-commit": {
+    "enabled": true,
+    "actions": [
+      {
+        "action": ["foo", "bar", "baz"]
+      }
+    ]
+  }
+}

tests/unit/Config/FactoryTest.php

@@ -11,6 +11,7 @@
 
 namespace CaptainHook\App\Config;
 
+use CaptainHook\App\Config;
 use Exception;
 use PHPUnit\Framework\TestCase;
 
@@ -24,6 +25,14 @@ public function testCreate(): void
         $this->assertCount(1, $config->getHookConfig('pre-commit')->getActions());
     }
 
+    public function testItDoesAllowArrayActions(): void
+    {
+        $config  = Factory::create(CH_PATH_FILES . '/config/valid-with-multi-line-action.json');
+        $actions = $config->getHookConfig('pre-commit')->getActions();
+
+        $this->assertEquals('foo bar baz', $actions[0]->getAction());
+    }
+
     public function testCustomValuesCanBeDefined(): void
     {
         $config = Factory::create(CH_PATH_FILES . '/config/valid-with-all-settings.json');