Merge pull request #80 from CharlotteDunoisLabs/file-node

Use Adapter *Contents instead in File Node

Author: WyriHaximus

src/Node/File.php

@@ -145,20 +145,15 @@ public function close()
      */
     public function getContents()
     {
-        return $this->open('r')->then(function ($stream) {
-            return Stream\buffer($stream);
-        });
+        return $this->adapter->getContents($this->path);
     }
 
     /**
      * {@inheritDoc}
      */
     public function putContents($contents)
     {
-        return $this->open('cw')->then(function (WritableStreamInterface $stream) use ($contents) {
-            $stream->write($contents);
-            return $this->close();
-        });
+        return $this->adapter->putContents($this->path, $contents);
     }
 
     /**

tests/Node/FileTest.php

@@ -280,36 +280,13 @@ public function testGetContents()
 
         $filesystem = $this->mockAdapter();
 
-        $filesystem
-            ->expects($this->any())
-            ->method('stat')
-            ->with($path)
-            ->will($this->returnValue(new FulfilledPromise([
-                'size' => 1,
-            ])))
-        ;
-
-        $filesystem
-            ->expects($this->once())
-            ->method('open')
-            ->with($path, 'r')
-            ->will($this->returnValue(new FulfilledPromise($fd)))
-        ;
-
         $filesystem
             ->expects($this->once())
-            ->method('read')
-            ->with($fd, 1, 0)
+            ->method('getContents')
+            ->with($path)
             ->will($this->returnValue(new FulfilledPromise('a')))
         ;
 
-        $filesystem
-            ->expects($this->once())
-            ->method('close')
-            ->with($fd)
-            ->will($this->returnValue(new FulfilledPromise()))
-        ;
-
         $getContentsPromise = (new File($path, Filesystem::createFromAdapter($filesystem)))->getContents();
         $this->assertInstanceOf('React\Promise\PromiseInterface', $getContentsPromise);
     }