Add feature to rename directories

seregazhuk · seregazhuk · commit 50766f6fb9dc · 2018-02-22T17:39:34.000+03:00
diff --git a/examples/directory_rename.php b/examples/directory_rename.php
@@ -0,0 +1,18 @@
+<?php
+
+use React\EventLoop\Factory;
+use React\Filesystem\Filesystem;
+
+require '../vendor/autoload.php';
+
+$loop = Factory::create();
+$filesystem = Filesystem::create($loop);
+$dir = $filesystem->dir('new');
+
+$dir->rename('new_name')->then(function(\React\Filesystem\Node\DirectoryInterface $newDir){
+    echo 'Renamed to ' . $newDir->getPath() . PHP_EOL;
+}, function(Exception $e) {
+    echo 'Error: ' . $e->getMessage() . PHP_EOL;
+});
+
+$loop->run();
diff --git a/src/Node/Directory.php b/src/Node/Directory.php
@@ -164,6 +164,17 @@ public function remove()
         return $this->adapter->rmdir($this->path);
     }
 
+
+    /**
+     * {@inheritdoc}
+     */
+    public function rename($toDirectoryName)
+    {
+        return $this->adapter->rename($this->path, $toDirectoryName)->then(function () use ($toDirectoryName) {
+            return $this->filesystem->dir($toDirectoryName);
+        });
+    }
+
     /**
      * {@inheritDoc}
      */
diff --git a/src/Node/DirectoryInterface.php b/src/Node/DirectoryInterface.php
@@ -28,6 +28,14 @@ public function createRecursive($mode = AdapterInterface::CREATION_MODE);
      */
     public function remove();
 
+    /**
+     * Rename the directory and return the new directory through a promise
+     *
+     * @param string $toDirectoryName
+     * @return PromiseInterface<FileInterface>
+     */
+    public function rename($toDirectoryName);
+
     /**
      * List contents of the directory.
      *
diff --git a/tests/Node/DirectoryTest.php b/tests/Node/DirectoryTest.php
@@ -2,6 +2,7 @@
 
 namespace React\Tests\Filesystem\Node;
 
+use React\EventLoop\Factory;
 use React\Filesystem\Filesystem;
 use React\Filesystem\Node\Directory;
 use React\Filesystem\Node\File;
@@ -81,6 +82,24 @@ public function testCreate()
         $this->assertInstanceOf('React\Promise\PromiseInterface', (new Directory($path, Filesystem::createFromAdapter($filesystem)))->create());
     }
 
+    public function testRename()
+    {
+        $pathFrom = 'foo.bar';
+        $pathTo = 'bar.foo';
+        $filesystem = $this->mockAdapter();
+
+        $filesystem
+            ->expects($this->once())
+            ->method('rename')
+            ->with($pathFrom, $pathTo)
+            ->will($this->returnValue(new FulfilledPromise()))
+        ;
+
+        $newDirectory = \Clue\React\Block\await((new Directory($pathFrom, Filesystem::createFromAdapter($filesystem)))->rename($pathTo), Factory::create());
+        $this->assertInstanceOf('React\Filesystem\Node\DirectoryInterface', $newDirectory);
+        $this->assertSame($pathTo . NodeInterface::DS, $newDirectory->getPath());
+    }
+
     public function testRemove()
     {
         $path = 'foo.bar';
