Merge pull request #20 from seregazhuk/directory-rename

Add feature to rename directories

Author: WyriHaximus

examples/directory_rename.php

@@ -0,0 +1,15 @@
+<?php
+
+require dirname(__DIR__) . '/vendor/autoload.php';
+
+$loop = React\EventLoop\Factory::create();
+$filesystem = React\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();

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}
      */

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<DirectoryInterface>
+     */
+    public function rename($toDirectoryName);
+
     /**
      * List contents of the directory.
      *

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';