Squashed 2 commits

Remove redundant file_exists check, remove unnessissary ltrim call

added empty root path check in FileSystem loader constructor

Binary/Loader/FileSystemLoader.php

@@ -38,7 +38,7 @@ public function __construct(
         $this->mimeTypeGuesser = $mimeTypeGuesser;
         $this->extensionGuesser = $extensionGuesser;
 
-        if (!($realRootPath = realpath($rootPath))) {
+        if (empty($rootPath) || !($realRootPath = realpath($rootPath))) {
             throw new InvalidArgumentException(sprintf('Root image path not resolvable "%s"', $rootPath));
         }
 
@@ -50,18 +50,14 @@ public function __construct(
      */
     public function find($path)
     {
-        if (!($absolutePath = realpath($this->rootPath.DIRECTORY_SEPARATOR.ltrim($path, DIRECTORY_SEPARATOR)))) {
+        if (!($absolutePath = realpath($this->rootPath.DIRECTORY_SEPARATOR.$path))) {
             throw new NotLoadableException(sprintf('Source image not resolvable "%s"', $path));
         }
 
         if (0 !== strpos($absolutePath, $this->rootPath)) {
             throw new NotLoadableException(sprintf('Source image invalid "%s" as it is outside of the defined root path', $absolutePath));
         }
 
-        if (false == file_exists($absolutePath)) {
-            throw new NotLoadableException(sprintf('Source image not found in "%s"', $absolutePath));
-        }
-
         $mimeType = $this->mimeTypeGuesser->guess($absolutePath);
 
         return new FileBinary(

Tests/Binary/Loader/FileSystemLoaderTest.php

@@ -41,6 +41,20 @@ public function testCouldBeConstructedWithExpectedArguments()
         );
     }
 
+    public function testThrowExceptionIfRootPathIsEmpty()
+    {
+        $this->setExpectedException(
+            'Liip\ImagineBundle\Exception\InvalidArgumentException',
+            'Root image path not resolvable'
+        );
+
+        new FileSystemLoader(
+            MimeTypeGuesser::getInstance(),
+            ExtensionGuesser::getInstance(),
+            ''
+        );
+    }
+
     public function testThrowExceptionIfRootPathDoesNotExist()
     {
         $this->setExpectedException(