Throw exception if JmsSerializerBundle is not registered

Author: wouterj

src/DependencyInjection/CmfResourceRestExtension.php

@@ -35,6 +35,11 @@ public function load(array $configs, ContainerBuilder $container)
         $configuration = new Configuration();
         $config = $processor->processConfiguration($configuration, $configs);
 
+        $bundles = $container->getParameter('kernel.bundles');
+        if (!array_key_exists('JMSSerializerBundle', $bundles)) {
+            throw new \LogicException('The JMSSerializerBundle must be registered in order to use the CmfResourceRestBundle.');
+        }
+
         $container->setParameter('cmf_resource_rest.max_depth', $config['max_depth']);
 
         $loader->load('serializer.xml');

tests/Unit/DependencyInjection/CmfResourceRestExtensionTest.php

@@ -49,9 +49,23 @@ public function provideExtension()
      */
     public function testExtension($config)
     {
-        $this->container->setParameter('kernel.bundles', array());
+        $this->container->setParameter('kernel.bundles', ['JMSSerializerBundle' => true]);
         $this->container->addCompilerPass(new EnhancerPass());
+
         $this->load($config);
+
+        $this->compile();
+    }
+
+    /**
+     * @expectedException \LogicException
+     * @expectedExceptionMessage The JMSSerializerBundle must be registered
+     */
+    public function testNoJmsSerializerBundleRegistered()
+    {
+        $this->container->setParameter('kernel.bundles', []);
+
+        $this->load([]);
         $this->compile();
     }
 }

tests/Unit/DependencyInjection/ConfigurationTest.php

@@ -53,6 +53,7 @@ public function testConfig($source)
                     'enhancer' => 'my_enhancer',
                 ),
             ),
+            'max_depth' => 2,
         ), array($source));
     }
 }