[Config][cache factory] check type of callback argument.

Author: aitboudad

ConfigCacheFactory.php

@@ -37,8 +37,11 @@ public function __construct($debug)
      */
     public function cache($file, $callback)
     {
-        $cache = new ConfigCache($file, $this->debug);
+        if (!is_callable($callback)) {
+            throw new \InvalidArgumentException(sprintf('Invalid type for callback argument. Expected callable, but got "%s".', gettype($callback)));
+        }
 
+        $cache = new ConfigCache($file, $this->debug);
         if (!$cache->isFresh()) {
             call_user_func($callback, $cache);
         }

Tests/ConfigCacheFactoryTest.php

@@ -0,0 +1,28 @@
+<?php
+
+/*
+ * This file is part of the Symfony package.
+ *
+ * (c) Fabien Potencier <[email protected]>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Symfony\Component\Config\Tests;
+
+use Symfony\Component\Config\ConfigCacheFactory;
+
+class ConfigCacheFactoryTest extends \PHPUnit_Framework_TestCase
+{
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage Invalid type for callback argument. Expected callable, but got "object".
+     */
+    public function testCachWithInvalidCallback()
+    {
+        $cacheFactory = new ConfigCacheFactory(true);
+
+        $cacheFactory->cache('file', new \stdClass());
+    }
+}