여러 화면에서 조건부(FormRequest) 검증이 우회되거나 과도하게 제한되던 결함을 한 계열로 묶어 수정한다. - 검증 규칙 확장 필터 이름 10개를 나머지 92개와 동일 규칙으로 통일 - 설명 길이를 바이트가 아닌 글자 수로 계산(한글 오검출 해소) - 비활성 언어팩 번역 보존한 채 수정 허용 - 알림 템플릿 제목 nullable 존중, 미지원 발송 수단 저장 차단 - 일정/실행 이력 목록 조회 검증 정합 - 게시판 첨부 미사용 시 빈 확장자 통과, new_display_hours/최소 길이 0 허용, 일괄 적용 override_values 범위 검증(SSoT) - 이커머스 클레임 사유 조회 검증, 용어 정정 - 원문 i18n 키 노출 다수 해소, 저장/삭제 실패 시 사유 화면 노출 - 아이콘 크기/다크모드 대비 회귀 수정 번들 확장(모듈3·플러그인6·템플릿2·언어팩5) 버전 new-bump 및 CHANGELOG 반영, ja 언어팩 동기.
190 lines
6.1 KiB
PHP
190 lines
6.1 KiB
PHP
<?php
|
|
|
|
namespace Modules\Sirsoft\Board\Tests\Feature\User;
|
|
|
|
// ModuleTestCase를 수동으로 require (autoload 전에 로드 필요)
|
|
require_once __DIR__.'/../../ModuleTestCase.php';
|
|
|
|
use App\Models\Role;
|
|
use App\Models\User;
|
|
use Illuminate\Http\UploadedFile;
|
|
use Illuminate\Routing\Route;
|
|
use Illuminate\Support\Facades\Storage;
|
|
use Modules\Sirsoft\Board\Http\Requests\UploadAttachmentRequest;
|
|
use Modules\Sirsoft\Board\Tests\BoardTestCase;
|
|
|
|
/**
|
|
* 허용 확장자 빈 값 게시판의 업로드 폴백 테스트
|
|
*
|
|
* allowed_extensions 가 빈 배열([]) 또는 NULL 인 레거시 게시판에서
|
|
* `implode(',', [])` 가 `mimes:` 빈 규칙을 만들어 전 파일 업로드가 거부되던 결함의 회귀 테스트.
|
|
*
|
|
* `??` 는 null 만 잡고 []는 통과시키므로 소비 시점에 `is_array() && !== []` 판정이 필요하다.
|
|
*
|
|
* @group board
|
|
* @group attachment
|
|
*/
|
|
class AttachmentAllowedExtensionsFallbackTest extends BoardTestCase
|
|
{
|
|
private User $memberUser;
|
|
|
|
/**
|
|
* 테스트 게시판 슬러그를 반환합니다.
|
|
*
|
|
* @return string 슬러그
|
|
*/
|
|
protected function getTestBoardSlug(): string
|
|
{
|
|
return 'attachment-ext-fallback';
|
|
}
|
|
|
|
/**
|
|
* 테스트 게시판 기본 속성을 반환합니다.
|
|
*
|
|
* @param string $slug 게시판 슬러그
|
|
* @return array<string, mixed> 게시판 속성
|
|
*/
|
|
protected function getDefaultBoardAttributes(string $slug): array
|
|
{
|
|
return [
|
|
'slug' => $slug,
|
|
'name' => ['ko' => '확장자 폴백 테스트 게시판', 'en' => 'Extension Fallback Test Board'],
|
|
'is_active' => true,
|
|
'use_file_upload' => true,
|
|
'allowed_extensions' => [],
|
|
];
|
|
}
|
|
|
|
/**
|
|
* 테스트 환경을 준비합니다.
|
|
*/
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
Storage::fake('public');
|
|
|
|
$this->grantUserRolePermissions(['posts.read', 'posts.write', 'attachments.upload']);
|
|
|
|
$this->memberUser = User::factory()->create(['email' => 'ext-fallback-member@test.com']);
|
|
|
|
$userRole = Role::where('identifier', 'user')->first();
|
|
if ($userRole) {
|
|
$this->memberUser->roles()->attach($userRole->id);
|
|
}
|
|
}
|
|
|
|
/**
|
|
* UploadAttachmentRequest 를 대상 게시판 슬러그로 생성합니다.
|
|
*
|
|
* @param string $slug 게시판 슬러그
|
|
* @return UploadAttachmentRequest 라우트 슬러그가 주입된 요청 객체
|
|
*/
|
|
private function makeUploadRequest(string $slug): UploadAttachmentRequest
|
|
{
|
|
$request = new UploadAttachmentRequest;
|
|
|
|
$request->setRouteResolver(function () use ($slug) {
|
|
$route = \Mockery::mock(Route::class);
|
|
$route->shouldReceive('parameter')->with('slug', null)->andReturn($slug);
|
|
|
|
return $route;
|
|
});
|
|
|
|
return $request;
|
|
}
|
|
|
|
/**
|
|
* file 규칙에서 mimes 파라미터를 추출합니다.
|
|
*
|
|
* @param array<string, mixed> $rules 검증 규칙
|
|
* @return string|null mimes 확장자 목록 (규칙 부재 시 null)
|
|
*/
|
|
private function extractMimes(array $rules): ?string
|
|
{
|
|
foreach ($rules['file'] ?? [] as $rule) {
|
|
if (is_string($rule) && str_starts_with($rule, 'mimes:')) {
|
|
return substr($rule, strlen('mimes:'));
|
|
}
|
|
}
|
|
|
|
return null;
|
|
}
|
|
|
|
/**
|
|
* allowed_extensions 가 빈 배열이면 mimes 규칙이 비어있지 않아야 한다 (회귀)
|
|
*/
|
|
public function test_empty_allowed_extensions_does_not_produce_empty_mimes_rule(): void
|
|
{
|
|
$rules = $this->makeUploadRequest($this->board->slug)->rules();
|
|
|
|
$mimes = $this->extractMimes($rules);
|
|
|
|
$this->assertNotSame('', $mimes, 'allowed_extensions=[] 에서 mimes: 빈 규칙이 생성되면 전 파일이 거부된다');
|
|
$this->assertNotEmpty($mimes, 'mimes 규칙은 기본 확장자 목록으로 폴백되어야 함');
|
|
}
|
|
|
|
/**
|
|
* allowed_extensions 가 NULL 이어도 mimes 규칙이 비어있지 않아야 한다
|
|
*/
|
|
public function test_null_allowed_extensions_does_not_produce_empty_mimes_rule(): void
|
|
{
|
|
$this->updateBoardSettings(['allowed_extensions' => null]);
|
|
|
|
$rules = $this->makeUploadRequest($this->board->slug)->rules();
|
|
|
|
$this->assertNotEmpty($this->extractMimes($rules), 'allowed_extensions=null 도 기본 확장자로 폴백되어야 함');
|
|
}
|
|
|
|
/**
|
|
* allowed_extensions 가 빈 배열인 게시판에서도 실제 업로드가 성공해야 한다 (회귀)
|
|
*
|
|
* @scenario case=upload_enabled_extensions_empty
|
|
*
|
|
* @effects empty_extensions_upload_falls_back_to_defaults
|
|
*/
|
|
public function test_upload_succeeds_when_allowed_extensions_is_empty(): void
|
|
{
|
|
$file = UploadedFile::fake()->create('document.pdf', 100, 'application/pdf');
|
|
|
|
$response = $this->actingAs($this->memberUser)
|
|
->postJson("/api/modules/sirsoft-board/boards/{$this->board->slug}/attachments", [
|
|
'file' => $file,
|
|
]);
|
|
|
|
$response->assertStatus(201)->assertJsonPath('success', true);
|
|
}
|
|
|
|
/**
|
|
* 허용 확장자가 지정된 게시판은 목록 밖 확장자를 계속 거부해야 한다 (회귀 방지)
|
|
*
|
|
* 폴백이 검증 자체를 무력화하지 않는지 확인한다.
|
|
*
|
|
* @scenario case=upload_enabled_extensions_nonempty
|
|
*
|
|
* @effects specified_extensions_still_reject_other_types
|
|
*/
|
|
public function test_specified_allowed_extensions_still_reject_other_types(): void
|
|
{
|
|
$this->updateBoardSettings(['allowed_extensions' => ['jpg', 'png']]);
|
|
|
|
$file = UploadedFile::fake()->create('document.pdf', 100, 'application/pdf');
|
|
|
|
$response = $this->actingAs($this->memberUser)
|
|
->postJson("/api/modules/sirsoft-board/boards/{$this->board->slug}/attachments", [
|
|
'file' => $file,
|
|
]);
|
|
|
|
$response->assertStatus(422);
|
|
}
|
|
|
|
/**
|
|
* 테스트 정리
|
|
*/
|
|
protected function tearDown(): void
|
|
{
|
|
\Mockery::close();
|
|
parent::tearDown();
|
|
}
|
|
}
|