bug #1324 Added test for access rule path array (georaldcamposano)

This PR was squashed before being merged into the 2.0.x-dev branch (closes #1324).

Discussion
----------

Added test for access rule path array

Related to PR #1310 Also fixed default value for 5th argument passed to RequestMatcher
constructor.

Commits
-------

5a9516ed Added test for access rule path array

src/Silex/Provider/SecurityServiceProvider.php

@@ -350,7 +350,7 @@ class SecurityServiceProvider implements ServiceProviderInterface, EventListener
                         'host' => null,
                         'methods' => null,
                         'ips' => null,
-                        'attributes' => null,
+                        'attributes' => array(),
                         'schemes' => null,
                     ];
                     $rule[0] = new RequestMatcher($rule[0]['path'], $rule[0]['host'], $rule[0]['methods'], $rule[0]['ips'], $rule[0]['attributes'], $rule[0]['schemes']);

tests/Silex/Tests/Provider/SecurityServiceProviderTest.php

@@ -270,6 +270,31 @@ class SecurityServiceProviderTest extends WebTestCase
         $this->assertNull($app['user']);
     }
 
+    public function testAccessRulePathArray()
+    {
+        $app = new Application();
+        $app->register(new SecurityServiceProvider(), array(
+            'security.firewalls' => array(
+                'default' => array(
+                    'http' => true,
+                ),
+            ),
+            'security.access_rules' => array(
+                array(array(
+                    'path' => '^/admin',
+                ), 'ROLE_ADMIN'),
+            ),
+        ));
+
+        $request = Request::create('/admin');
+        $app->boot();
+        $accessMap = $app['security.access_map'];
+        $this->assertEquals($accessMap->getPatterns($request), array(
+            array('ROLE_ADMIN'),
+            '',
+        ));
+    }
+
     public function createApplication($authenticationMethod = 'form')
     {
         $app = new Application();