Move stream_wrapper_register() to bootstrap.php

composer · BrianHenryIE · May 2, 2025 · May 2, 2025 · May 2, 2025 · May 2, 2025
commit 0cd95aa44e84b64a0af61e52f8d8ff76b846c90f
diff --git a/tests/Composer/Test/Util/FilesystemTest.php b/tests/Composer/Test/Util/FilesystemTest.php
@@ -412,7 +412,7 @@ public static function isAbsolutePathDataProvider(): array
             'unixPath' => ['/foo/bar', true],
             'smbPath' => ['\\\\smb\\folder\\file.txt', true],
             'windowsPath' => ['C:\\foo\\bar', true],
-            'streamPath' => ['customstreamwrapper://path/to/whatever', true],
+            'streamPath' => ['composertestsstreamwrapper://path/to/whatever', true],
             'fileStreamAbsolutePath' => ['file:///path/to/whatever', true],
             'fileStreamRelativePath' => ['file://path/to/whatever', false],
             'relativeSubPath' => ['foo/bar', false],
@@ -427,10 +427,6 @@ public static function isAbsolutePathDataProvider(): array
      */
     public function testIsAbsolutePath(string $path, bool $expected): void
     {
-        if (str_starts_with($path, 'customstreamwrapper://') && !in_array('customstreamwrapper', stream_get_wrappers(), true)) {
-            stream_wrapper_register('customstreamwrapper', __CLASS__);
-        }
-
         self::assertEquals($expected, (new Filesystem())->isAbsolutePath($path));
     }
 }
diff --git a/tests/bootstrap.php b/tests/bootstrap.php
@@ -35,3 +35,12 @@
 Platform::clearEnv('COMPOSER');
 Platform::clearEnv('COMPOSER_VENDOR_DIR');
 Platform::clearEnv('COMPOSER_BIN_DIR');
+
+/**
+ * The stream wrapper checks use a static property to store the stream wrapper names. If it is accessed before
+ * the stream wrapper is registered, and a later test relies on that stream wrapper, the static property will be
+ * outdated and the test will fail.
+ *
+ * @see \Composer\Util\Filesystem::isStreamWrapperPath()
+ */
+stream_wrapper_register('composertestsstreamwrapper', stdclass::class);