clean

laminas · Nov 22, 2024 · aea99b3 · aea99b3
1 parent 2e0649e
commit aea99b3
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 3 deletions.
diff --git a/phpcs.xml.dist b/phpcs.xml.dist
@@ -16,11 +16,15 @@
     <file>src</file>
     <file>test</file>
     <exclude-pattern>*/_files/*</exclude-pattern>
+    <exclude-pattern>test/PhpEnvironment/Assets/*</exclude-pattern>
 
     <!-- Include all rules from Laminas Coding Standard -->
     <rule ref="LaminasCodingStandard"/>
 
     <rule ref="SlevomatCodingStandard.TypeHints.DeclareStrictTypes.DeclareStrictTypesMissing">
         <exclude-pattern>src/</exclude-pattern>
     </rule>
+    <rule ref="SlevomatCodingStandard.Namespaces.ReferenceUsedNamesOnly.ReferenceViaFallbackGlobalName">
+        <exclude-pattern>src/PhpEnvironment/Response.php</exclude-pattern>
+    </rule>
 </ruleset>
diff --git a/src/PhpEnvironment/Response.php b/src/PhpEnvironment/Response.php
@@ -8,7 +8,6 @@
 
 use function call_user_func;
 use function header;
-use function headers_sent;
 
 /**
  * HTTP Response for current PHP environment

diff --git a/test/PhpEnvironment/Assets/MockHeadersSent.php b/test/PhpEnvironment/Assets/MockHeadersSent.php
@@ -0,0 +1,7 @@
+<?php
+
+namespace Laminas\Http\PhpEnvironment;
+// Define a custom headers_sent function within the namespace
+function headers_sent(&$filename = null, &$line = null): bool {
+    return true;
+}
diff --git a/test/PhpEnvironment/RequestTest.php b/test/PhpEnvironment/RequestTest.php
@@ -431,8 +431,12 @@ public static function serverHostnameProvider(): array
     }
 
     #[DataProvider('serverHostnameProvider')]
-    public function testServerHostnameProvider(array $server, string $expectedHost, string $expectedPort, string $expectedRequestUri): void
-    {
+    public function testServerHostnameProvider(
+        array $server,
+        string $expectedHost,
+        string $expectedPort,
+        string $expectedRequestUri
+    ): void {
         $_SERVER = $server;
         $request = new Request();
 

diff --git a/test/PhpEnvironment/ResponseTest.php b/test/PhpEnvironment/ResponseTest.php
@@ -106,8 +106,11 @@ public function testSendHeadersReturnsInstanceIfHeadersNotAlreadySent(): void
         $this->assertInstanceOf(Response::class, $response->sendHeaders());
     }
 
+    #[RunInSeparateProcess]
     public function testSendHeadersInvokesHeadersSentHandlerIfHeadersAreAlreadySent(): void
     {
+        include __DIR__ . '/Assets/MockHeadersSent.php';
+
         $response = new Response();
         $response->setHeadersSentHandler(function ($response): void {
             throw new RuntimeException('Cannot send headers, headers already sent');