Merge pull request #3 from Micro-PHP/1.6.1

v1.6.1 implements list of allowed SAPI modes

Author: Asisyas

src/Listener/ApplicationStartedListener.php

@@ -25,6 +25,15 @@
  */
 class ApplicationStartedListener implements EventListenerInterface
 {
+    public const ALLOWED_MODES = [
+        'apache2handler',
+        'apache',
+        'cgi-fcgi',
+        'fpm-fcgi',
+        'litespeed',
+        'cli-server',
+    ];
+
     public function __construct(
         private readonly HttpFacadeInterface $httpFacade
     ) {
@@ -38,12 +47,11 @@ public function __construct(
     public function on(EventInterface $event): void
     {
         $sysenv = $event->systemEnvironment();
-        if ('cli' === $sysenv) {
+        if (!\in_array($sysenv, self::ALLOWED_MODES)) {
             return;
         }
 
         $request = Request::createFromGlobals();
-
         $this->httpFacade->execute($request);
     }
 

tests/Unit/Listener/ApplicationStartedListenerTest.php

@@ -31,20 +31,20 @@ protected function setUp(): void
         $this->applicationStartedListener = new ApplicationStartedListener($this->httpFacade);
     }
 
-    protected function createEvent(bool $isHttp)
+    protected function createEvent(string $env)
     {
         $evt = $this->createMock(ApplicationReadyEventInterface::class);
-        $evt->method('systemEnvironment')->willReturn($isHttp ? 'http' : 'cli');
+        $evt->method('systemEnvironment')->willReturn($env);
 
         return $evt;
     }
 
     /**
      * @dataProvider dataProvider
      */
-    public function testOn(bool $isHttp): void
+    public function testOn(string $extension, bool $isHttp): void
     {
-        $event = $this->createEvent($isHttp);
+        $event = $this->createEvent($extension);
 
         if (!$isHttp) {
             $this->httpFacade
@@ -64,17 +64,21 @@ public function testOn(bool $isHttp): void
 
     public function dataProvider(): array
     {
-        return [
-            [true],
-            [false],
-        ];
+        $allowed = ApplicationStartedListener::ALLOWED_MODES;
+        $data = [];
+
+        foreach ($allowed as $val) {
+            $data[] = [$val, true];
+        }
+
+        return $data;
     }
 
     public function testSupports()
     {
         $httpFacade = $this->createMock(HttpFacadeInterface::class);
         $listener = new ApplicationStartedListener($httpFacade);
 
-        $this->assertTrue($listener->supports($this->createEvent(true)));
+        $this->assertTrue($listener->supports($this->createEvent(ApplicationStartedListener::ALLOWED_MODES[0])));
     }
 }