Merge pull request #3 from solocommand/master

Ensure that file contents are actually written correctly.
limit-zero · Feb 20, 2017 · ce9c713 · ce9c713
2 parents 5e66121 + 448a06c
commit ce9c713
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/StorageEngine/LocalStorageEngine.php b/src/StorageEngine/LocalStorageEngine.php
@@ -95,7 +95,7 @@ public function store(Asset $asset, $path = null, $filename = null)
             throw new StorageException($error['message']);
         }
 
-        file_put_contents($key, $asset);
+        file_put_contents($key, file_get_contents($asset));
 
         return true;
     }

diff --git a/tests/StorageEngine/AbstractStorageEngineTest.php b/tests/StorageEngine/AbstractStorageEngineTest.php
@@ -47,6 +47,12 @@ public function testEngineRetrieval()
         $this->assertInstanceOf('Limit0\Assets\Asset', $asset);
     }
 
+    public function testFileContents()
+    {
+        $asset = self::$engine->retrieve($this->getTestIdentifier());
+        $this->assertSame(file_get_contents($this->getTestAsset()), file_get_contents($asset), 'StorageEngine did not store file contents properly.');
+    }
+
     public function testEngineRemoval()
     {
         $result = self::$engine->remove($this->getTestIdentifier());