Atomic write file via symfony/filesystem

Author: alexpts

src/PTS/SymfonyDiLoader/LoaderContainer.php

@@ -10,13 +10,15 @@
 use Symfony\Component\DependencyInjection\Dumper\PhpDumper;
 use Symfony\Component\DependencyInjection\Extension\ExtensionInterface;
 use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
+use Symfony\Component\Filesystem\Filesystem;
 use Throwable;
 
 class LoaderContainer implements LoaderContainerInterface
 {
 	/** @var string[] */
 	protected array $configFiles = [];
 
+    protected ?Filesystem $fs = null;
 	protected FactoryContainer $factory;
 	protected ?ContainerInterface $container = null;
 	protected CacheWatcher $cacheWatcher;
@@ -84,7 +86,7 @@ protected function createContainer(array $configs, array $extensions = []): Cont
 	protected function dumpMeta(string $filePath, array $configFiles): void
 	{
 		try {
-			file_put_contents($filePath, serialize($configFiles));
+		    $this->getFilesystem()->dumpFile($filePath, serialize($configFiles));
 		} catch (Throwable $throwable) {
 			throw new RuntimeException('Can`t dump meta for DI container', 0, $throwable);
 		}
@@ -95,9 +97,9 @@ protected function dump(string $filePath, string $className, ContainerBuilder $c
 		$dumper = new PhpDumper($container);
 
 		try {
-			file_put_contents($filePath, $dumper->dump([
-				'class' => $className,
-			]));
+		    $this->getFilesystem()->dumpFile($filePath, $dumper->dump([
+                'class' => $className,
+            ]));
 		} catch (Throwable $throwable) {
 			throw new RuntimeException('Can`t dump cache for DI container', 0, $throwable);
 		}
@@ -127,4 +129,13 @@ protected function getWatcher(): CacheWatcher
 	{
 		return $this->cacheWatcher;
 	}
+
+	protected function getFilesystem(): Filesystem
+    {
+        if ($this->fs === null) {
+            $this->fs = new Filesystem;
+        }
+
+        return $this->fs;
+    }
 }

test/unit/LoaderContainer/DumpMetaTest.php

@@ -41,8 +41,8 @@ public function testDumpMetaNotPermission(): void
 		$cacheFileName = 'cache.php';
 		$loader = new LoaderContainer($configs, $cacheFileName);
 
-		$vfs = vfsStream::setup('temp-di-loader');
-		$path = vfsStream::newFile($cacheFileName . '.meta', 0444)->at($vfs)->url();
+		$vfs = vfsStream::setup('temp-di-loader', 0550);
+		$path = vfsStream::newFile($cacheFileName . '.meta', 0440)->at($vfs)->url();
 
 		$this->expectException(\RuntimeException::class);
 		$this->expectExceptionMessage('Can`t dump meta for DI container');

test/unit/LoaderContainer/DumpTest.php

@@ -55,8 +55,8 @@ public function testDumpNotPermission(): void
 			->getMock();
 		$container->method('isCompiled')->willReturn(true);
 
-		$vfs = vfsStream::setup('temp-di-loader');
-		$path = vfsStream::newFile($cacheFileName, 0444)->at($vfs)->url();
+		$vfs = vfsStream::setup('temp-di-loader', 0550);
+		$path = vfsStream::newFile($cacheFileName, 0440)->at($vfs)->url();
 
 		$this->expectException(RuntimeException::class);
 		$this->expectExceptionMessage('Can`t dump cache for DI container');