Added test cases.

Author: limingxinleo

tests/Cases/FormRequestTest.php

@@ -11,18 +11,21 @@
  */
 namespace HyperfTest\Validation\Cases;
 
+use Hyperf\HttpMessage\Server\Response;
 use Hyperf\HttpMessage\Upload\UploadedFile;
 use Hyperf\Translation\ArrayLoader;
 use Hyperf\Translation\Translator;
 use Hyperf\Utils\Context;
 use Hyperf\Validation\Contract\ValidatorFactoryInterface;
 use Hyperf\Validation\ValidationException;
 use Hyperf\Validation\ValidatorFactory;
+use HyperfTest\Validation\Cases\Stub\BarSceneRequest;
 use HyperfTest\Validation\Cases\Stub\DemoRequest;
 use HyperfTest\Validation\Cases\Stub\FooSceneRequest;
 use Mockery;
 use PHPUnit\Framework\TestCase;
 use Psr\Container\ContainerInterface;
+use Psr\Http\Message\ResponseInterface;
 use Psr\Http\Message\ServerRequestInterface;
 
 /**
@@ -74,17 +77,46 @@ public function testRequestValidationDataWithSameKey()
         $this->assertEquals(['file' => ['Invalid File.', $file]], $request->getValidationData());
     }
 
+    public function testRewriteGetRules()
+    {
+        $psrRequest = Mockery::mock(ServerRequestInterface::class);
+        $psrRequest->shouldReceive('getQueryParams')->andReturn([]);
+        $psrRequest->shouldReceive('getUploadedFiles')->andReturn([]);
+        $psrRequest->shouldReceive('getParsedBody')->andReturn([
+            'name' => 'xxx',
+        ]);
+
+        Context::set(ServerRequestInterface::class, $psrRequest);
+        Context::set(ResponseInterface::class, new Response());
+        $container = Mockery::mock(ContainerInterface::class);
+        $translator = new Translator(new ArrayLoader(), 'en');
+        $container->shouldReceive('get')->with(ValidatorFactoryInterface::class)->andReturn(new ValidatorFactory($translator));
+
+        $request = new BarSceneRequest($container);
+        $res = $request->scene('required')->validated();
+        $this->assertSame(['name' => 'xxx'], $res);
+
+        try {
+            $request = new BarSceneRequest($container);
+            $request->validateResolved();
+            $this->assertTrue(false);
+        } catch (\Throwable $exception) {
+            $this->assertInstanceOf(ValidationException::class, $exception);
+            $this->assertSame('validation.integer', $exception->validator->errors()->first());
+        }
+    }
+
     public function testSceneForFormRequest()
     {
         $psrRequest = Mockery::mock(ServerRequestInterface::class);
         $psrRequest->shouldReceive('getQueryParams')->andReturn([]);
         $psrRequest->shouldReceive('getUploadedFiles')->andReturn([]);
         $psrRequest->shouldReceive('getParsedBody')->andReturn([
             'mobile' => '12345',
-            'name' => '',
         ]);
 
         Context::set(ServerRequestInterface::class, $psrRequest);
+        Context::set(ResponseInterface::class, new Response());
         $container = Mockery::mock(ContainerInterface::class);
         $translator = new Translator(new ArrayLoader(), 'en');
         $container->shouldReceive('get')->with(ValidatorFactoryInterface::class)->andReturn(new ValidatorFactory($translator));
@@ -95,8 +127,9 @@ public function testSceneForFormRequest()
 
         wait(function () use ($request, $psrRequest) {
             Context::set(ServerRequestInterface::class, $psrRequest);
+            Context::set(ResponseInterface::class, new Response());
             try {
-                $request->validated();
+                $request->validateResolved();
                 $this->assertTrue(false);
             } catch (\Throwable $exception) {
                 $this->assertInstanceOf(ValidationException::class, $exception);
@@ -105,7 +138,7 @@ public function testSceneForFormRequest()
 
         try {
             $request = new FooSceneRequest($container);
-            $request->validated();
+            $request->validateResolved();
             $this->assertTrue(false);
         } catch (\Throwable $exception) {
             $this->assertInstanceOf(ValidationException::class, $exception);

tests/Cases/Stub/BarSceneRequest.php

@@ -0,0 +1,38 @@
+<?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 Hyperf\Validation\Request\FormRequest;
+
+class BarSceneRequest extends FormRequest
+{
+    public function authorize(): bool
+    {
+        return true;
+    }
+
+    public function rules(): array
+    {
+        $scene = $this->getScene();
+
+        switch ($scene) {
+            case 'required':
+                return [
+                    'name' => 'required',
+                ];
+            default:
+                return [
+                    'name' => 'required|integer',
+                ];
+        }
+    }
+}