Merge pull request #165 from zds-s/added-test

Optimise request->all() to clear the number subscripts when fetching

app/Setting/Controller/Settings/SystemConfigController.php

@@ -79,7 +79,7 @@ public function update(SettingConfigRequest $request): ResponseInterface
     #[PostMapping('updateByKeys'), Permission('setting:config:update'), OperationLog]
     public function updateByKeys(): ResponseInterface
     {
-        return $this->service->updatedByKeys($this->request->all()) ? $this->success() : $this->error();
+        return $this->service->updatedByKeys($this->request->post()) ? $this->success() : $this->error();
     }
 
     /**

tests/ModelControllerTest/Setting/Modules/ModuleTest.php

@@ -11,7 +11,7 @@
  */
 use Nette\Utils\FileSystem;
 
-/**
+/*
  * This file is part of MineAdmin.
  *
  * @see     https://www.mineadmin.com

tests/Pest.php

@@ -13,7 +13,6 @@
 use Hyperf\Context\ApplicationContext;
 use Hyperf\DbConnection\Db;
 use Hyperf\Di\Aop\ProceedingJoinPoint;
-use Hyperf\Stringable\Str;
 use HyperfTests\HttpTestCase;
 use HyperfTests\MineControllerTestCase;
 use Mine\Aspect\OperationLogAspect;
@@ -49,8 +48,8 @@ function testFailResponse(mixed $result)
     ->beforeEach(function () {
         // Create Super Administrator
         Db::table('system_user')->truncate();
-        $this->password = Str::random(8);
-        $this->username = Str::random(10);
+        $this->password = '123456';
+        $this->username = 'admin';
         SystemUser::whereKey(env('SUPER_ADMIN', 1))->delete();
         $this->mock = SystemUser::create([
             'id' => env('SUPER_ADMIN', 1),