fix: convert 302 to 303

Author: NeoIsRecursive

src/Http/Middleware.php

@@ -49,13 +49,7 @@ public function __invoke(Request $request, HttpMiddlewareCallable $next): Respon
             $response->status === Status::FOUND &&
                 in_array($request->method, [Method::POST, Method::PUT, Method::PATCH], strict: true)
         ) {
-            // TODO: set status to 303
-            // return new GenericResponse(
-            //     status: Status::SEE_OTHER,
-            //     headers: [
-            //         'Location' => $response->getHeader('Location'),
-            //     ]
-            // );
+            $response->setStatus(Status::SEE_OTHER);
         }
 
         return $response;

tests/Fixtures/TestController.php

@@ -10,14 +10,17 @@
 use NeoIsRecursive\Inertia\Props\AlwaysProp;
 use Tempest\Http\Responses\Ok;
 use Tempest\Http\Responses\Redirect;
-use Tempest\Http\Session\Session;
 use Tempest\Router\Get;
+use Tempest\Router\Patch;
+use Tempest\Router\Post;
+use Tempest\Router\Put;
 
 use function NeoIsRecursive\Inertia\inertia;
 use function Tempest\uri;
 
 final readonly class TestController
 {
+    #[Get(uri: '/')]
     public function index(): InertiaResponse
     {
         return inertia(component: 'Index');
@@ -84,4 +87,22 @@ public function testRedirectWithClearHistory(Inertia $inertia): Redirect
         $inertia->clearHistory();
         return new Redirect(to: uri([self::class, 'testCanSharePropsFromAnyWhere']));
     }
+
+    #[Post(uri: '/test-post-with-redirect')]
+    public function testPostWithRedirect(): Redirect
+    {
+        return new Redirect(to: uri([static::class, 'index']));
+    }
+
+    #[Patch(uri: '/test-patch-with-redirect')]
+    public function testPatchWithRedirect(): Redirect
+    {
+        return new Redirect(to: uri([static::class, 'index']));
+    }
+
+    #[Put(uri: '/test-put-with-redirect')]
+    public function testPutWithRedirect(): Redirect
+    {
+        return new Redirect(to: uri([static::class, 'index']));
+    }
 }

tests/Integration/HistoryTest.php

@@ -66,23 +66,6 @@ public function test_the_history_can_be_encrypted_via_middleware(): void
         //     ]);
     }
 
-    public function test_the_history_can_be_encrypted_via_middleware_alias(): void
-    {
-        static::markTestIncomplete(message: 'This test is incomplete and needs to be fixed.');
-
-        // Route::middleware([StartSession::class, ExampleMiddleware::class, 'inertia.encrypt'])->get('/', function () {
-        //     return Inertia::render('User/Edit');
-        // });
-        // $response = $this->withoutExceptionHandling()->get('/', [
-        //     'X-Inertia' => 'true',
-        // ]);
-        // $response->assertSuccessful();
-        // $response->assertJson([
-        //     'component' => 'User/Edit',
-        //     'encryptHistory' => true,
-        // ]);
-    }
-
     public function test_the_history_can_be_encrypted_globally(): void
     {
         static::markTestIncomplete(message: 'This test is incomplete and needs to be fixed.');

tests/Integration/MiddlewareTest.php

@@ -0,0 +1,44 @@
+<?php
+
+declare(strict_types=1);
+
+namespace NeoIsRecursive\Inertia\Tests\Integration;
+
+use NeoIsRecursive\Inertia\Tests\Fixtures\TestController;
+use NeoIsRecursive\Inertia\Tests\TestCase;
+use PHPUnit\Framework\Attributes\TestWith;
+use Tempest\Http\Status;
+
+use function Tempest\uri;
+
+final class MiddlewareTest extends TestCase
+{
+    public function test_middleware_is_registered(): void
+    {
+        $middleware = $this->container->get(\NeoIsRecursive\Inertia\Http\Middleware::class);
+
+        static::assertInstanceOf(\NeoIsRecursive\Inertia\Http\Middleware::class, $middleware);
+    }
+
+    #[TestWith([
+        'testPostWithRedirect',
+        'post',
+    ], name: 'post')]
+    #[TestWith([
+        'testPutWithRedirect',
+        'put',
+    ], name: 'put')]
+    #[TestWith([
+        'testPatchWithRedirect',
+        'patch',
+    ], name: 'patch')]
+    public function test_middleware_converts_post_put_or_patch_302_to_303(string $action, string $method): void
+    {
+        $response = $this->http->{$method}(uri([TestController::class, $action]), headers: [
+            'X-Inertia' => 'true',
+            'X-Version' => '1.0',
+        ]);
+
+        $response->assertStatus(Status::SEE_OTHER);
+    }
+}