Make "run" default action if missing in config

Author: tomasfejfar

src/Config/BaseConfig.php

@@ -136,7 +136,7 @@ public function getAuthorization(): array
      */
     public function getAction(): string
     {
-        return $this->getValue(['action'], '');
+        return $this->getValue(['action'], 'run');
     }
 
     /**

tests/BaseComponentTest.php

@@ -102,6 +102,29 @@ protected function run(): void
         $this->assertTrue($handler->hasAlert('Log message from run'));
     }
 
+    public function testWillNotFailWithEmptyConfigAction(): void
+    {
+        $logger = $this->getLogger();
+        $handler = new TestHandler();
+        $logger->setHandlers([$handler]);
+        putenv(sprintf(
+            'KBC_DATADIR=%s',
+            __DIR__ . '/fixtures/base-component-data-dir/empty-config-file'
+        ));
+        $baseComponent = new class ($logger) extends BaseComponent
+        {
+            protected function run(): void
+            {
+                echo 'Shitty output';
+                $this->getLogger()->alert('Log message from run');
+            }
+        };
+        $this->expectOutputString('Shitty output');
+        $baseComponent->execute();
+
+        $this->assertTrue($handler->hasAlert('Log message from run'));
+    }
+
     public function testLoadInputStateFileEmptyThrowsException(): void
     {
         putenv(sprintf(

tests/Config/BaseConfigTest.php

@@ -108,7 +108,7 @@ public function testGettersWillNotFailIfKeyIsMissing(): void
             ],
         ]);
         $this->assertSame([], $config->getParameters());
-        $this->assertSame('', $config->getAction());
+        $this->assertSame('run', $config->getAction());
         $this->assertSame([], $config->getAuthorization());
         $this->assertSame('', $config->getOAuthApiAppKey());
         $this->assertSame('', $config->getOAuthApiAppSecret());

tests/fixtures/base-component-data-dir/empty-config-file/config.json

@@ -0,0 +1,2 @@
+{
+}