Fixed bug that FormRequest could't get the changed data from Context. (#3351)

* Added test cases

Co-authored-by: 李铭昕 <715557344@qq.com>

Author: wdaglb

src/Middleware/ValidationMiddleware.php

@@ -55,6 +55,8 @@ public function process(ServerRequestInterface $request, RequestHandlerInterface
             throw new ServerException(sprintf('The dispatched object is not a %s object.', Dispatched::class));
         }
 
+        Context::set(ServerRequestInterface::class, $request);
+
         if ($this->shouldHandle($dispatched)) {
             try {
                 [$requestHandler, $method] = $this->prepareHandler($dispatched->handler->callback);

tests/Cases/Stub/FooMiddleware.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+/**
+ * This file is part of Hyperf.
+ *
+ * @link     https://www.hyperf.io
+ * @document https://hyperf.wiki
+ * @contact  group@hyperf.io
+ * @license  https://github.com/hyperf/hyperf/blob/master/LICENSE
+ */
+namespace HyperfTest\Validation\Cases\Stub;
+
+use Psr\Http\Message\ResponseInterface;
+use Psr\Http\Message\ServerRequestInterface;
+use Psr\Http\Server\MiddlewareInterface;
+use Psr\Http\Server\RequestHandlerInterface;
+
+class FooMiddleware implements MiddlewareInterface
+{
+    public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
+    {
+        $parsedBody = $request->getParsedBody();
+        if (! isset($parsedBody['password'])) {
+            $parsedBody['password'] = 'Hyperf';
+        }
+        return $handler->handle($request->withParsedBody($parsedBody));
+    }
+}

tests/Cases/ValidationMiddlewareTest.php

@@ -34,6 +34,7 @@
 use Hyperf\Validation\ValidatorFactory;
 use HyperfTest\Validation\Cases\Stub\DemoController;
 use HyperfTest\Validation\Cases\Stub\DemoRequest;
+use HyperfTest\Validation\Cases\Stub\FooMiddleware;
 use Mockery;
 use PHPUnit\Framework\TestCase;
 use Psr\EventDispatcher\EventDispatcherInterface;
@@ -54,6 +55,31 @@ protected function tearDown(): void
         Context::set('http.request.parsedData', null);
     }
 
+    public function testValidationMiddlewareGetTheLatestInputData()
+    {
+        $container = $this->createContainer();
+        $factory = $container->get(DispatcherFactory::class);
+
+        $router = $factory->getRouter('http');
+        $router->addRoute('POST', '/sign-up', 'HyperfTest\Validation\Cases\Stub\DemoController@signUp');
+
+        $dispatcher = $factory->getDispatcher('http');
+        $fooMiddleware = new FooMiddleware();
+        $middleware = new ValidationMiddleware($container);
+        $coreMiddleware = new CoreMiddleware($container, 'http');
+        $handler = new HttpRequestHandler([$fooMiddleware, $middleware], $coreMiddleware, $container);
+        Context::set(ResponseInterface::class, new Response());
+
+        $request = (new Request('POST', new Uri('/sign-up')))
+            ->withParsedBody(['username' => 'Hyperf']);
+        $routes = $dispatcher->dispatch($request->getMethod(), $request->getUri()->getPath());
+        $request = Context::set(ServerRequestInterface::class, $request->withAttribute(Dispatched::class, new Dispatched($routes)));
+
+        $response = $handler->handle($request);
+
+        $this->assertEquals(200, $response->getStatusCode());
+    }
+
     public function testProcess()
     {
         $container = $this->createContainer();