Added doc and test

Author: bobvandevijver

README.md

@@ -382,7 +382,16 @@ liip_imagine:
                 data_root: /path/to/source/images/dir
 ```
 
-This location must be readable by your web server. On a system that supports 
+When you need assets from bundles which have a `Resources/public`-folder, you can automatically add them to the `data_root` by setting the `bundle_resources` parameter to `true`.
+```yml
+liip_imagine:
+    loaders:
+        default:
+            filesystem:
+                bundle_resources: true
+```
+
+Image locations must be readable by your web server. On a system that supports 
 `setfacl` (such as Linux/BSD), use
 
 ```sh

Resources/doc/data-loader/filesystem.rst

@@ -36,3 +36,17 @@ You can configure the ``data_root``, used as the root path to search for images:
             profile_photos:
                 filesystem:
                     data_root: "%kernel.root_dir%/../web"
+
+You can automatically load bundle resources in their ``Resources/public`` by enabling
+the ``bundle_resources`` parameter. This will automatically add those directories to the
+root path.
+
+.. code-block:: yaml
+
+    # app/config/config.yml
+
+    liip_imagine:
+        loaders:
+            profile_photos:
+                filesystem:
+                    bundle_resources: true

Tests/DependencyInjection/Factory/Loader/FileSystemLoaderFactoryTest.php

@@ -48,7 +48,8 @@ public function testCreateLoaderDefinitionOnCreate()
         $loader = new FileSystemLoaderFactory();
 
         $loader->create($container, 'the_loader_name', array(
-            'data_root' => 'theDataRoot',
+            'bundle_resources' => false,
+            'data_root' => array('theDataRoot'),
             'locator' => 'filesystem',
         ));
 
@@ -59,7 +60,27 @@ public function testCreateLoaderDefinitionOnCreate()
         $this->assertInstanceOfChildDefinition($loaderDefinition);
         $this->assertEquals('liip_imagine.binary.loader.prototype.filesystem', $loaderDefinition->getParent());
 
-        $this->assertEquals('theDataRoot', $loaderDefinition->getArgument(2));
+        $this->assertEquals(array('theDataRoot'), $loaderDefinition->getArgument(2));
+    }
+
+    public function testCreateLoaderDefinitionOnCreateWithBundlesEnabled()
+    {
+        $container = new ContainerBuilder();
+
+        // We need at least an empty bundle parameter
+        $container->setParameter('kernel.bundles', array());
+
+        $loader = new FileSystemLoaderFactory();
+
+        $loader->create($container, 'the_loader_name', array(
+            'bundle_resources' => true,
+            'data_root' => array('theDataRoot'),
+            'locator' => 'filesystem',
+        ));
+
+        $loaderDefinition = $container->getDefinition('liip_imagine.binary.loader.the_loader_name');
+
+        $this->assertEquals(array('theDataRoot'), $loaderDefinition->getArgument(2));
     }
 
     public function testProcessCorrectlyOptionsOnAddConfiguration()