Revert "minor symfony#22039 Skip abstract definitions in compiler passes (chalasr)"

This reverts commit 207d068, reversing
changes made to 4836007.

Author: nicolas-grekas

src/Symfony/Bridge/Doctrine/DependencyInjection/CompilerPass/RegisterEventListenersAndSubscribersPass.php

@@ -79,7 +79,7 @@ public function process(ContainerBuilder $container)
                 uasort($subscribers, $sortFunc);
                 foreach ($subscribers as $id => $instance) {
                     if ($container->getDefinition($id)->isAbstract()) {
-                        continue;
+                        throw new InvalidArgumentException(sprintf('The abstract service "%s" cannot be tagged as a doctrine event subscriber.', $id));
                     }
 
                     $em->addMethodCall('addEventSubscriber', array(new Reference($id)));
@@ -95,7 +95,7 @@ public function process(ContainerBuilder $container)
                 uasort($listeners, $sortFunc);
                 foreach ($listeners as $id => $instance) {
                     if ($container->getDefinition($id)->isAbstract()) {
-                        continue;
+                        throw new InvalidArgumentException(sprintf('The abstract service "%s" cannot be tagged as a doctrine event listener.', $id));
                     }
 
                     $em->addMethodCall('addEventListener', array(

src/Symfony/Bridge/Doctrine/Tests/DependencyInjection/CompilerPass/RegisterEventListenersAndSubscribersPassTest.php

@@ -18,6 +18,9 @@
 
 class RegisterEventListenersAndSubscribersPassTest extends TestCase
 {
+    /**
+     * @expectedException \InvalidArgumentException
+     */
     public function testExceptionOnAbstractTaggedSubscriber()
     {
         $container = $this->createBuilder();
@@ -29,10 +32,12 @@ public function testExceptionOnAbstractTaggedSubscriber()
         $container->setDefinition('a', $abstractDefinition);
 
         $this->process($container);
-        $this->assertSame(array(), $container->getDefinition('doctrine.dbal.default_connection.event_manager')->getMethodCalls());
     }
 
-    public function testAbstractTaggedListenerIsSkipped()
+    /**
+     * @expectedException \InvalidArgumentException
+     */
+    public function testExceptionOnAbstractTaggedListener()
     {
         $container = $this->createBuilder();
 
@@ -43,7 +48,6 @@ public function testAbstractTaggedListenerIsSkipped()
         $container->setDefinition('a', $abstractDefinition);
 
         $this->process($container);
-        $this->assertSame(array(), $container->getDefinition('doctrine.dbal.default_connection.event_manager')->getMethodCalls());
     }
 
     public function testProcessEventListenersWithPriorities()

src/Symfony/Bundle/FrameworkBundle/Tests/DependencyInjection/Compiler/AddConsoleCommandPassTest.php

@@ -62,6 +62,10 @@ public function visibilityProvider()
         );
     }
 
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage The service "my-command" tagged "console.command" must not be abstract.
+     */
     public function testProcessThrowAnExceptionIfTheServiceIsAbstract()
     {
         $container = new ContainerBuilder();
@@ -73,8 +77,6 @@ public function testProcessThrowAnExceptionIfTheServiceIsAbstract()
         $container->setDefinition('my-command', $definition);
 
         $container->compile();
-
-        $this->assertSame(array(), $container->getParameter('console.command.ids'));
     }
 
     /**

src/Symfony/Component/Console/DependencyInjection/AddConsoleCommandPass.php

@@ -32,7 +32,7 @@ public function process(ContainerBuilder $container)
             $definition = $container->getDefinition($id);
 
             if ($definition->isAbstract()) {
-                continue;
+                throw new \InvalidArgumentException(sprintf('The service "%s" tagged "console.command" must not be abstract.', $id));
             }
 
             $class = $container->getParameterBag()->resolveValue($definition->getClass());

src/Symfony/Component/Console/Tests/DependencyInjection/AddConsoleCommandPassTest.php

@@ -50,7 +50,11 @@ public function visibilityProvider()
         );
     }
 
-    public function testProcessSkipAbstractDefinitions()
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage The service "my-command" tagged "console.command" must not be abstract.
+     */
+    public function testProcessThrowAnExceptionIfTheServiceIsAbstract()
     {
         $container = new ContainerBuilder();
         $container->setResourceTracking(false);
@@ -62,8 +66,6 @@ public function testProcessSkipAbstractDefinitions()
         $container->setDefinition('my-command', $definition);
 
         $container->compile();
-
-        $this->assertSame(array(), $container->getParameter('console.command.ids'));
     }
 
     /**

src/Symfony/Component/EventDispatcher/DependencyInjection/RegisterListenersPass.php

@@ -63,7 +63,7 @@ public function process(ContainerBuilder $container)
         foreach ($container->findTaggedServiceIds($this->listenerTag) as $id => $events) {
             $def = $container->getDefinition($id);
             if ($def->isAbstract()) {
-                continue;
+                throw new InvalidArgumentException(sprintf('The service "%s" must not be abstract as event listeners are lazy-loaded.', $id));
             }
 
             foreach ($events as $event) {
@@ -90,7 +90,7 @@ public function process(ContainerBuilder $container)
         foreach ($container->findTaggedServiceIds($this->subscriberTag) as $id => $attributes) {
             $def = $container->getDefinition($id);
             if ($def->isAbstract()) {
-                continue;
+                throw new InvalidArgumentException(sprintf('The service "%s" must not be abstract as event subscribers are lazy-loaded.', $id));
             }
 
             // We must assume that the class value has been correctly filled, even if the service is created by a factory

src/Symfony/Component/EventDispatcher/Tests/DependencyInjection/RegisterListenersPassTest.php

@@ -87,6 +87,10 @@ public function testValidEventSubscriber()
         $registerListenersPass->process($builder);
     }
 
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage The service "foo" must not be abstract as event listeners are lazy-loaded.
+     */
     public function testAbstractEventListener()
     {
         $container = new ContainerBuilder();
@@ -95,20 +99,20 @@ public function testAbstractEventListener()
 
         $registerListenersPass = new RegisterListenersPass();
         $registerListenersPass->process($container);
-
-        $this->assertSame(array(), $container->getDefinition('event_dispatcher')->getMethodCalls());
     }
 
-    public function testAbstractEventSubscriberIsSkipped()
+    /**
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage The service "foo" must not be abstract as event subscribers are lazy-loaded.
+     */
+    public function testAbstractEventSubscriber()
     {
         $container = new ContainerBuilder();
         $container->register('foo', 'stdClass')->setAbstract(true)->addTag('kernel.event_subscriber', array());
         $container->register('event_dispatcher', 'stdClass');
 
         $registerListenersPass = new RegisterListenersPass();
         $registerListenersPass->process($container);
-
-        $this->assertSame(array(), $container->getDefinition('event_dispatcher')->getMethodCalls());
     }
 
     public function testEventSubscriberResolvableClassName()