Move middleware tests to separate file

Author: ivanvermeyen

tests/Unit/Macros/LocalizedRoutesMacroTest.php

@@ -144,49 +144,6 @@ public function it_does_not_permanently_change_the_locale_without_middleware()
         $this->assertEquals($originalLocale, $response->original);
     }
 
-    /** @test */
-    public function it_sets_the_right_locale_when_accessing_localized_routes()
-    {
-        $this->setSupportedLocales(['en', 'nl']);
-        $this->setUseLocaleMiddleware(true);
-
-        Route::localized(function () {
-            Route::get('/', function () {
-                return App::getLocale();
-            });
-        });
-
-        $response = $this->call('GET', '/en');
-        $response->assertOk();
-        $this->assertEquals('en', $response->original);
-
-        $response = $this->call('GET', '/nl');
-        $response->assertOk();
-        $this->assertEquals('nl', $response->original);
-    }
-
-    /** @test */
-    public function it_sets_the_right_locale_when_accessing_localized_routes_with_omitted_prefix()
-    {
-        $this->setSupportedLocales(['en', 'nl']);
-        $this->setUseLocaleMiddleware(true);
-        $this->setOmitUrlPrefixForLocale('en');
-
-        Route::localized(function () {
-            Route::get('/', function () {
-                return App::getLocale();
-            });
-        });
-
-        $response = $this->call('GET', '/');
-        $response->assertOk();
-        $this->assertEquals('en', $response->original);
-
-        $response = $this->call('GET', '/nl');
-        $response->assertOk();
-        $this->assertEquals('nl', $response->original);
-    }
-
     /** @test */
     public function it_correctly_uses_scoped_config_options()
     {
@@ -273,31 +230,4 @@ public function it_creates_localized_routes_within_route_groups()
         $this->call('GET', '/admin/en/route')->assertOk();
         $this->call('GET', '/admin/nl/route')->assertOk();
     }
-
-    /** @test */
-    public function it_sets_the_locale_for_localized_routes_within_route_groups()
-    {
-        $this->setSupportedLocales(['en', 'nl']);
-        $this->setUseLocaleMiddleware(true);
-
-        Route::group([
-            'as' => 'admin.',
-            'prefix' => 'admin'
-        ], function () {
-            Route::localized(function () {
-                Route::get('route', function () {
-                    return App::getLocale();
-                })
-                    ->name('route.name');
-            });
-        });
-
-        $response = $this->call('GET', '/admin/en/route');
-        $response->assertOk();
-        $this->assertEquals('en', $response->original);
-
-        $response = $this->call('GET', '/admin/nl/route');
-        $response->assertOk();
-        $this->assertEquals('nl', $response->original);
-    }
 }

tests/Unit/Middleware/LocalizedRouteLocaleHandlerTest.php

@@ -0,0 +1,80 @@
+<?php
+
+namespace CodeZero\LocalizedRoutes\Tests\Unit\Macros;
+
+use CodeZero\LocalizedRoutes\Middleware\LocalizedRouteLocaleHandler;
+use CodeZero\LocalizedRoutes\Tests\Stubs\Model;
+use CodeZero\LocalizedRoutes\Tests\TestCase;
+use Illuminate\Support\Facades\App;
+use Illuminate\Support\Facades\Route;
+
+class LocalizedRouteLocaleHandlerTest extends TestCase
+{
+    /** @test */
+    public function it_sets_the_right_locale_when_accessing_localized_routes()
+    {
+        $this->setSupportedLocales(['en', 'nl']);
+        $this->setUseLocaleMiddleware(true);
+
+        Route::localized(function () {
+            Route::get('/', function () {
+                return App::getLocale();
+            });
+        });
+
+        $response = $this->call('GET', '/en');
+        $response->assertOk();
+        $this->assertEquals('en', $response->original);
+
+        $response = $this->call('GET', '/nl');
+        $response->assertOk();
+        $this->assertEquals('nl', $response->original);
+    }
+
+    /** @test */
+    public function it_sets_the_right_locale_when_accessing_localized_routes_with_omitted_prefix()
+    {
+        $this->setSupportedLocales(['en', 'nl']);
+        $this->setUseLocaleMiddleware(true);
+        $this->setOmitUrlPrefixForLocale('nl');
+
+        Route::localized(function () {
+            Route::get('/', function () {
+                return App::getLocale();
+            });
+        });
+
+        $response = $this->call('GET', '/');
+        $response->assertOk();
+        $this->assertEquals('nl', $response->original);
+
+        $response = $this->call('GET', '/en');
+        $response->assertOk();
+        $this->assertEquals('en', $response->original);
+    }
+
+    /** @test */
+    public function it_sets_the_locale_for_localized_routes_within_route_groups()
+    {
+        $this->setSupportedLocales(['en', 'nl']);
+
+        Route::group([
+            'as' => 'admin.',
+            'prefix' => 'admin',
+        ], function () {
+            Route::localized(function () {
+                Route::get('route', function () {
+                    return App::getLocale();
+                })->name('route.name')->middleware(['web', LocalizedRouteLocaleHandler::class]);
+            });
+        });
+
+        $response = $this->call('GET', '/admin/en/route');
+        $response->assertOk();
+        $this->assertEquals('en', $response->original);
+
+        $response = $this->call('GET', '/admin/nl/route');
+        $response->assertOk();
+        $this->assertEquals('nl', $response->original);
+    }
+}