Merge pull request #189 from cdwlighting/optimize_get_route_by_name

Created Unit tests for getRoutesByName with defined idPrefix.

Author: lsmith77

Doctrine/Phpcr/RouteProvider.php

@@ -119,8 +119,9 @@ protected function getCandidates($url)
      */
     public function getRouteByName($name, $parameters = array())
     {
+
         // $name is the route document path
-        if ($this->idPrefix && 0 === strpos($name, $this->idPrefix)) {
+        if ('' === $this->idPrefix || 0 === strpos($name, $this->idPrefix)) {
             $route = $this->getObjectManager()->find($this->className, $name);
         }
 
@@ -140,10 +141,10 @@ public function getRouteByName($name, $parameters = array())
      */
     public function getRoutesByNames($names, $parameters = array())
     {
-        if ($this->idPrefix) {
-            $routes = array();
+        $routes = array();
+        if ('' === $this->idPrefix) {
             foreach ($names as $name) {
-                if (0 === strpos($name, $this->idPrefix)) {
+                if ('' === $this->idPrefix || 0 === strpos($name, $this->idPrefix)) {
                     $routes[] = $name;
                 }
             }

Tests/Unit/Doctrine/Phpcr/RouteProviderTest.php

@@ -66,6 +66,7 @@ public function testGetRouteByName()
         $routeProvider = new RouteProvider($this->managerRegistry);
         $routeProvider->setManagerName('default');
 
+        $routeProvider->setPrefix('/cms/routes/');
         $foundRoute = $routeProvider->getRouteByName('/cms/routes/test-route');
 
         $this->assertInstanceOf('Symfony\Component\Routing\Route', $foundRoute);
@@ -92,7 +93,7 @@ public function testGetRouteByNameNotFound()
 
         $routeProvider = new RouteProvider($this->managerRegistry);
         $routeProvider->setManagerName('default');
-
+        $routeProvider->setPrefix('/cms/routes/');
         $routeProvider->getRouteByName('/cms/routes/test-route');
     }
 
@@ -116,6 +117,112 @@ public function testGetRouteByNameNoRoute()
 
         $routeProvider = new RouteProvider($this->managerRegistry);
         $routeProvider->setManagerName('default');
+        $routeProvider->setPrefix('/cms/routes/');
+        $routeProvider->getRouteByName('/cms/routes/test-route');
+    }
+
+    /**
+     * @expectedException  \Symfony\Component\Routing\Exception\RouteNotFoundException
+     */
+    public function testGetRouteByNameInvalidRoute()
+    {
+        $this->objectManager
+            ->expects($this->never())
+            ->method('find')
+        ;
+
+        $this->managerRegistry
+            ->expects($this->any())
+            ->method('getManager')
+            ->will($this->returnValue($this->objectManager))
+        ;
+
+        $routeProvider = new RouteProvider($this->managerRegistry);
+        $routeProvider->setManagerName('default');
+
+        $routeProvider->setPrefix('/cms/routes');
+
+        $routeProvider->getRouteByName('invalid_route');
+    }
+
+    public function testGetRouteByNameIdPrefixEmptyString()
+    {
+
+        $this->route
+            ->expects($this->any())
+            ->method('getPath')
+            ->will($this->returnValue('/cms/routes/test-route'))
+        ;
+
+        $this->objectManager
+            ->expects($this->any())
+            ->method('find')
+            ->with(null, '/cms/routes/test-route')
+            ->will($this->returnValue($this->route))
+        ;
+
+        $this->managerRegistry
+            ->expects($this->any())
+            ->method('getManager')
+            ->will($this->returnValue($this->objectManager))
+        ;
+
+        $routeProvider = new RouteProvider($this->managerRegistry);
+        $routeProvider->setManagerName('default');
+
+        $routeProvider->setPrefix('');
+        $foundRoute = $routeProvider->getRouteByName('/cms/routes/test-route');
+
+        $this->assertInstanceOf('Symfony\Component\Routing\Route', $foundRoute);
+        $this->assertEquals('/cms/routes/test-route', $foundRoute->getPath());
+    }
+
+
+    /**
+     * @expectedException \Symfony\Component\Routing\Exception\RouteNotFoundException
+     */
+    public function testGetRouteByNameNotFoundIdPrefixEmptyString()
+    {
+        $this->objectManager
+            ->expects($this->any())
+            ->method('find')
+            ->with(null, '/cms/routes/test-route')
+            ->will($this->returnValue(null))
+        ;
+
+        $this->managerRegistry
+            ->expects($this->any())
+            ->method('getManager')
+            ->will($this->returnValue($this->objectManager))
+        ;
+
+        $routeProvider = new RouteProvider($this->managerRegistry);
+        $routeProvider->setManagerName('default');
+        $routeProvider->setPrefix('');
+        $routeProvider->getRouteByName('/cms/routes/test-route');
+    }
+
+    /**
+     * @expectedException \Symfony\Component\Routing\Exception\RouteNotFoundException
+     */
+    public function testGetRouteByNameNoRoutePrefixEmptyString()
+    {
+        $this->objectManager
+            ->expects($this->any())
+            ->method('find')
+            ->with(null, '/cms/routes/test-route')
+            ->will($this->returnValue($this))
+        ;
+
+        $this->managerRegistry
+            ->expects($this->any())
+            ->method('getManager')
+            ->will($this->returnValue($this->objectManager))
+        ;
+
+        $routeProvider = new RouteProvider($this->managerRegistry);
+        $routeProvider->setManagerName('default');
+        $routeProvider->setPrefix('');
 
         $routeProvider->getRouteByName('/cms/routes/test-route');
     }
@@ -173,6 +280,8 @@ function ($name) use ($objectManagers) {
         $routeProvider = new RouteProvider($this->managerRegistry);
 
         $routeProvider->setManagerName('default');
+        $routeProvider->setPrefix('/cms/routes/');
+
         $foundRoute = $routeProvider->getRouteByName('/cms/routes/test-route');
         $this->assertInstanceOf('Symfony\Component\Routing\Route', $foundRoute);
         $this->assertEquals('/cms/routes/test-route', $foundRoute->getPath());