Merge pull request #102 from codeigniter4/routes-except

Ensure 'except' routes are not in routes at all. Fixes #72

Author: lonnieezell

src/Auth.php

@@ -100,7 +100,7 @@ public function routes(RouteCollection &$routes, array $config = [])
 
         $routes->group('/', ['namespace' => 'CodeIgniter\Shield\Controllers'], static function ($routes) use ($authRoutes, $config) {
             foreach ($authRoutes as $name => $row) {
-                if (! isset($config['except']) || (isset($config['except']) && ! array_key_exists($name, $config['except']))) {
+                if (! isset($config['except']) || (isset($config['except']) && ! in_array($name, $config['except'], true))) {
                     foreach ($row as $params) {
                         $options = isset($params[3])
                             ? ['as' => $params[3]]

tests/Unit/AuthRoutesTest.php

@@ -0,0 +1,43 @@
+<?php
+
+namespace Tests\Unit;
+
+use Tests\Support\TestCase;
+
+/**
+ * @internal
+ */
+final class AuthRoutesTest extends TestCase
+{
+    public function testRoutes()
+    {
+        $collection = single_service('routes');
+        $auth       = service('auth');
+
+        $auth->routes($collection);
+
+        $routes = $collection->getRoutes('get');
+
+        $this->assertArrayHasKey('register', $routes);
+        $this->assertArrayHasKey('login', $routes);
+        $this->assertArrayHasKey('login/magic-link', $routes);
+        $this->assertArrayHasKey('logout', $routes);
+        $this->assertArrayHasKey('auth/a/show', $routes);
+    }
+
+    public function testRoutesExcept()
+    {
+        $collection = single_service('routes');
+        $auth       = service('auth');
+
+        $auth->routes($collection, ['except' => ['login']]);
+
+        $routes = $collection->getRoutes('get');
+
+        $this->assertArrayNotHasKey('login', $routes);
+        $this->assertArrayHasKey('register', $routes);
+        $this->assertArrayHasKey('login/magic-link', $routes);
+        $this->assertArrayHasKey('logout', $routes);
+        $this->assertArrayHasKey('auth/a/show', $routes);
+    }
+}