minor #23852 Made some SecurityBundle tests case-insensitive to prepare for future Symfony versions (javiereguiluz)

Robin Chalas · Robin Chalas · commit 1a5acc91bdce · 2017-08-14T10:34:21.000+02:00
This PR was squashed before being merged into the 3.4 branch (closes #23852). Discussion ---------- Made some SecurityBundle tests case-insensitive to prepare for future Symfony versions | Q | A | ------------- | --- | Branch? | 3.4 | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | - | License | MIT | Doc PR | - This is what @nicolas-grekas [told me to do](#23797 (comment)) to unlock #23797. Commits ------- 9b7750a Made some SecurityBundle tests case-insensitive to prepare for future Symfony versions
diff --git a/src/Symfony/Bundle/SecurityBundle/Tests/DependencyInjection/CompleteConfigurationTest.php b/src/Symfony/Bundle/SecurityBundle/Tests/DependencyInjection/CompleteConfigurationTest.php
@@ -81,6 +81,11 @@ public function testFirewalls()
             $configs[] = array_values($configDef->getArguments());
         }
 
+        // the IDs of the services are case sensitive or insensitive depending on
+        // the Symfony version. Transform them to lowercase to simplify tests.
+        $configs[0][2] = strtolower($configs[0][2]);
+        $configs[2][2] = strtolower($configs[2][2]);
+
         $this->assertEquals(array(
             array(
                 'simple',
