fix(core,board,ecommerce): 조건부 검증 결함 패밀리 일괄 수정 및 실패 안내 노출
여러 화면에서 조건부(FormRequest) 검증이 우회되거나 과도하게 제한되던 결함을 한 계열로 묶어 수정한다. - 검증 규칙 확장 필터 이름 10개를 나머지 92개와 동일 규칙으로 통일 - 설명 길이를 바이트가 아닌 글자 수로 계산(한글 오검출 해소) - 비활성 언어팩 번역 보존한 채 수정 허용 - 알림 템플릿 제목 nullable 존중, 미지원 발송 수단 저장 차단 - 일정/실행 이력 목록 조회 검증 정합 - 게시판 첨부 미사용 시 빈 확장자 통과, new_display_hours/최소 길이 0 허용, 일괄 적용 override_values 범위 검증(SSoT) - 이커머스 클레임 사유 조회 검증, 용어 정정 - 원문 i18n 키 노출 다수 해소, 저장/삭제 실패 시 사유 화면 노출 - 아이콘 크기/다크모드 대비 회귀 수정 번들 확장(모듈3·플러그인6·템플릿2·언어팩5) 버전 new-bump 및 CHANGELOG 반영, ja 언어팩 동기.
This commit is contained in:
1 parent
c001b30a87
commit
7f8fb6870f
288 files changed
+5339
-1049
No files matched your search
@@ -3,11 +3,13 @@
|
||||
namespace Tests\Feature\Api\Admin;
|
||||
|
||||
use App\Enums\ExtensionOwnerType;
|
||||
use App\Enums\PermissionType;
|
||||
use App\Models\Permission;
|
||||
use App\Models\Role;
|
||||
use App\Models\User;
|
||||
use App\Services\ModuleService;
|
||||
use Illuminate\Foundation\Testing\RefreshDatabase;
|
||||
use Illuminate\Support\Facades\Route;
|
||||
use Illuminate\Validation\ValidationException;
|
||||
use Mockery;
|
||||
use Tests\TestCase;
|
||||
@@ -52,7 +54,7 @@ class ModuleUpdateTest extends TestCase
|
||||
'description' => json_encode(['ko' => $permIdentifier.' 권한', 'en' => $permIdentifier.' Permission']),
|
||||
'extension_type' => ExtensionOwnerType::Core,
|
||||
'extension_identifier' => 'core',
|
||||
'type' => \App\Enums\PermissionType::Admin,
|
||||
'type' => PermissionType::Admin,
|
||||
]
|
||||
);
|
||||
$permissionIds[] = $permission->id;
|
||||
@@ -229,7 +231,7 @@ class ModuleUpdateTest extends TestCase
|
||||
{
|
||||
$mockService = Mockery::mock(ModuleService::class);
|
||||
$mockService->shouldReceive('updateModule')
|
||||
->with('test-module', Mockery::any(), 'overwrite')
|
||||
->with('test-module', Mockery::any(), 'overwrite', false)
|
||||
->once()
|
||||
->andReturn([
|
||||
'success' => true,
|
||||
@@ -288,7 +290,7 @@ class ModuleUpdateTest extends TestCase
|
||||
{
|
||||
$mockService = Mockery::mock(ModuleService::class);
|
||||
$mockService->shouldReceive('updateModule')
|
||||
->with('test-module', Mockery::any(), 'overwrite')
|
||||
->with('test-module', Mockery::any(), 'overwrite', false)
|
||||
->once()
|
||||
->andReturn([
|
||||
'success' => true,
|
||||
@@ -332,7 +334,7 @@ class ModuleUpdateTest extends TestCase
|
||||
{
|
||||
$mockService = Mockery::mock(ModuleService::class);
|
||||
$mockService->shouldReceive('updateModule')
|
||||
->with('nonexistent-module', Mockery::any(), 'overwrite')
|
||||
->with('nonexistent-module', Mockery::any(), 'overwrite', false)
|
||||
->once()
|
||||
->andThrow(
|
||||
ValidationException::withMessages([
|
||||
@@ -353,7 +355,7 @@ class ModuleUpdateTest extends TestCase
|
||||
{
|
||||
$mockService = Mockery::mock(ModuleService::class);
|
||||
$mockService->shouldReceive('updateModule')
|
||||
->with('test-module', Mockery::any(), 'overwrite')
|
||||
->with('test-module', Mockery::any(), 'overwrite', false)
|
||||
->once()
|
||||
->andThrow(new \RuntimeException('Unexpected error'));
|
||||
$this->app->instance(ModuleService::class, $mockService);
|
||||
@@ -389,7 +391,7 @@ class ModuleUpdateTest extends TestCase
|
||||
public function test_check_updates_route_name_exists(): void
|
||||
{
|
||||
$this->assertTrue(
|
||||
\Illuminate\Support\Facades\Route::has('api.admin.modules.check-updates'),
|
||||
Route::has('api.admin.modules.check-updates'),
|
||||
'Route api.admin.modules.check-updates should exist'
|
||||
);
|
||||
}
|
||||
@@ -400,7 +402,7 @@ class ModuleUpdateTest extends TestCase
|
||||
public function test_update_route_name_exists(): void
|
||||
{
|
||||
$this->assertTrue(
|
||||
\Illuminate\Support\Facades\Route::has('api.admin.modules.update'),
|
||||
Route::has('api.admin.modules.update'),
|
||||
'Route api.admin.modules.update should exist'
|
||||
);
|
||||
}
|
||||
@@ -416,7 +418,7 @@ class ModuleUpdateTest extends TestCase
|
||||
{
|
||||
$mockService = Mockery::mock(ModuleService::class);
|
||||
$mockService->shouldReceive('updateModule')
|
||||
->with('test-module', Mockery::any(), 'keep')
|
||||
->with('test-module', Mockery::any(), 'keep', false)
|
||||
->once()
|
||||
->andReturn([
|
||||
'success' => true,
|
||||
@@ -492,7 +494,7 @@ class ModuleUpdateTest extends TestCase
|
||||
public function test_check_modified_layouts_route_name_exists(): void
|
||||
{
|
||||
$this->assertTrue(
|
||||
\Illuminate\Support\Facades\Route::has('api.admin.modules.check-modified-layouts'),
|
||||
Route::has('api.admin.modules.check-modified-layouts'),
|
||||
'Route api.admin.modules.check-modified-layouts should exist'
|
||||
);
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user