Merge pull request #37 from Wolfgang-check24/fix-sse-streaming-issue

Fixes #36 SSE stream is blocking

Author: klapaudius

phpunit.xml.dist

@@ -3,7 +3,7 @@
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
     xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.3/phpunit.xsd"
     backupGlobals="false"
-    bootstrap="vendor/autoload.php"
+    bootstrap="tests/bootstrap.php"
     colors="true"
     processIsolation="false"
     stopOnFailure="false"

src/Transports/AbstractTransport.php

@@ -97,10 +97,14 @@ public function start(): void
             return;
         }
 
-        set_time_limit(0);
-        ini_set('output_buffering', 'off');
-        ini_set('zlib.output_compression', false);
-        ini_set('zlib.default_socket_timeout', 5);
+        // Only modify these settings if not in test environment
+        // Tests need output buffering to capture output
+        if (!defined('PHPUNIT_RUNNING')) {
+            set_time_limit(0);
+            ini_set('output_buffering', 'off');
+            ini_set('zlib.output_compression', false);
+            ini_set('zlib.default_socket_timeout', 5);
+        }
 
         $this->connected = true;
         $this->initialize();
@@ -148,7 +152,12 @@ protected function sendEvent(string $event, string $data): void
         echo sprintf('data: %s', $data).PHP_EOL;
         echo PHP_EOL;
 
-        flush(); // Ensure the data is sent to the client
+        // Only flush when not in test environment
+        if (!defined('PHPUNIT_RUNNING')
+            && false !== ob_get_length()) {
+            ob_flush();  // Flush PHP's output buffer first
+        }
+        flush();         // Then flush system/web server buffers
     }
 
     /**

tests/bootstrap.php

@@ -0,0 +1,9 @@
+<?php
+
+declare(strict_types=1);
+
+// Define constant to indicate we're in a test environment
+define('PHPUNIT_RUNNING', true);
+
+// Load the composer autoloader
+require dirname(__DIR__).'/vendor/autoload.php';