Parse scopes set on groups so they are available as an array.

Signed-off-by: Jason Lewis <[email protected]>

Author: jasonlewis

changes.md

@@ -5,6 +5,7 @@
 - Made the trace readable by exploding on the EOL character.
 - API filters are now applied before any other filters to ensure they are run first.
 - Request and response instances are correctly set on the formatter for exception responses.
+- Scopes set on groups are now parsed correctly.
 
 ### v0.8.2
 

src/Routing/Route.php

@@ -50,6 +50,16 @@ protected function parseAction($action)
         return $action;
     }
 
+    /**
+     * {@inheritDoc}
+     */
+    public function setAction(array $action)
+    {
+        $action = static::parseAction($action);
+
+        return parent::setAction($action);
+    }
+
     /**
      * Determine if the route is protected.
      *

tests/Routing/RouterTest.php

@@ -437,4 +437,22 @@ public function testRouterThrowsExceptionWhenInvalidAcceptHeaderWithStrict()
         $this->router->setStrict(true);
         $this->router->dispatch(Request::create('foo', 'GET'));
     }
+
+    public function testRouterParsesScopesFromStringsToArrays()
+    {
+        $this->router->api(['version' => 'v1'], function () {
+            $this->router->group(['scopes' => 'foo'], function () {
+                $this->router->get('foo', function () {
+                    return 'bar';
+                });
+            });
+
+            $this->router->get('bar', ['scopes' => 'bar', function () {
+                return 'baz';
+            }]);
+        });
+
+        $this->assertEquals(['foo'], $this->router->getApiGroups()->getByVersion('v1')->getRoutes()[0]->scopes());
+        $this->assertEquals(['bar'], $this->router->getApiGroups()->getByVersion('v1')->getRoutes()[1]->scopes());
+    }
 }