Allow ChainRouter::match() to use the correct exception message when a RequestMatcherInterface router is used.

Author: timplunkett

ChainRouter.php

@@ -170,10 +170,13 @@ private function doMatch($url, Request $request = null)
                 // matching requests is more powerful than matching URLs only, so try that first
                 if ($router instanceof RequestMatcherInterface) {
                     if (null === $request) {
-                        $request = Request::create($url);
+                        $request_for_matching = Request::create($url);
+                    }
+                    else {
+                        $request_for_matching = $request;
                     }
 
-                    return $router->matchRequest($request);
+                    return $router->matchRequest($request_for_matching);
                 }
                 // every router implements the match method
                 return $router->match($url);

Tests/Routing/ChainRouterTest.php

@@ -385,6 +385,32 @@ public function testMatchRequestNotFound()
         $this->router->matchRequest(Request::create('/test'));
     }
 
+    /**
+     * Call match on ChainRouter that has RequestMatcher in the chain.
+     *
+     * @expectedException \Symfony\Component\Routing\Exception\ResourceNotFoundException
+     * @expectedExceptionMessage None of the routers in the chain matched url '/test'
+     */
+    public function testMatchWithRequestMatchersNotFound()
+    {
+        $url = '/test';
+        $request = Request::create('/test');
+
+        $high = $this->getMock('Symfony\Cmf\Component\Routing\Tests\Routing\RequestMatcher');
+
+        $high
+            ->expects($this->once())
+            ->method('matchRequest')
+            ->with($request)
+            ->will($this->throwException(new \Symfony\Component\Routing\Exception\ResourceNotFoundException))
+        ;
+
+        $this->router->add($high, 20);
+
+        $result = $this->router->match($url);
+        $this->assertEquals(array('test'), $result);
+    }
+
     /**
      * If any of the routers throws a not allowed exception and no other matches, we need to see this
      *