Skip RemotePackageStorage on Windows

smnandre · smnandre · commit f578c722d404 · 2024-09-02T01:12:38.000+02:00
diff --git a/src/Symfony/Component/AssetMapper/Tests/ImportMap/RemotePackageStorageTest.php b/src/Symfony/Component/AssetMapper/Tests/ImportMap/RemotePackageStorageTest.php
@@ -43,14 +43,14 @@ public function testGetStorageDir()
 
     public function testSaveThrowsWhenFailing()
     {
+        if ('\\' === \DIRECTORY_SEPARATOR) {
+            $this->markTestSkipped('This test cannot run on Windows.');
+        }
+
         $vendorDir = self::$writableRoot.'/assets/acme/vendor';
         $this->filesystem->mkdir($vendorDir.'/module_specifier');
         $this->filesystem->touch($vendorDir.'/module_specifier/module_specifier.index.js');
-        if ('\\' === \DIRECTORY_SEPARATOR) {
-            $this->filesystem->chmod($vendorDir.'/module_specifier/module_specifier.index.js', 0555);
-        } else {
-            $this->filesystem->chmod($vendorDir.'/module_specifier/', 0555);
-        }
+        $this->filesystem->chmod($vendorDir.'/module_specifier/', 0555);
 
         $storage = new RemotePackageStorage($vendorDir);
         $entry = ImportMapEntry::createRemote('foo', ImportMapType::JS, '/does/not/matter', '1.0.0', 'module_specifier', false);
@@ -61,11 +61,7 @@ public function testSaveThrowsWhenFailing()
         try {
             $storage->save($entry, 'any content');
         } finally {
-            if ('\\' === \DIRECTORY_SEPARATOR) {
-                $this->filesystem->chmod($vendorDir.'/module_specifier/module_specifier.index.js', 0777);
-            } else {
-                $this->filesystem->chmod($vendorDir.'/module_specifier/', 0777);
-            }
+            $this->filesystem->chmod($vendorDir.'/module_specifier/', 0777);
         }
     }
 
