Change test to a full change guard against additions as well

curry684 · curry684 · commit 6d5b815194db · 2018-05-30T10:51:34.000+02:00
diff --git a/src/Symfony/Bundle/FrameworkBundle/Tests/Controller/AbstractControllerTest.php b/src/Symfony/Bundle/FrameworkBundle/Tests/Controller/AbstractControllerTest.php
@@ -41,15 +41,13 @@ public function testSubscribedServices()
             'twig' => '?Twig\\Environment',
             'doctrine' => '?Doctrine\\Common\\Persistence\\ManagerRegistry',
             'form.factory' => '?Symfony\\Component\\Form\\FormFactoryInterface',
-            'security.token_storage' => '?Symfony\\Component\\Security\\Core\\Authentication\\Token\\Storage\\TokenStorageInterface',
-            'security.csrf.token_manager' => '?Symfony\\Component\\Security\\Csrf\\CsrfTokenManagerInterface',
             'parameter_bag' => '?Symfony\\Component\\DependencyInjection\\ParameterBag\\ContainerBagInterface',
             'message_bus' => '?Symfony\\Component\\Messenger\\MessageBusInterface',
+            'security.token_storage' => '?Symfony\\Component\\Security\\Core\\Authentication\\Token\\Storage\\TokenStorageInterface',
+            'security.csrf.token_manager' => '?Symfony\\Component\\Security\\Csrf\\CsrfTokenManagerInterface',
         );
 
-        foreach ($expectedServices as $id => $reference) {
-            $this->assertSame($reference, $subscribed[$id]);
-        }
+        $this->assertEquals($expectedServices, $subscribed, 'Subscribed core services in AbstractController have changed');
     }
 
     public function testGetParameter()
