Files
Gnuboard7/modules/_bundled/sirsoft-board/tests/Feature/User/AttachmentAllowedExtensionsFallbackTest.php
HeuJung 7f8fb6870f fix(core,board,ecommerce): 조건부 검증 결함 패밀리 일괄 수정 및 실패 안내 노출
여러 화면에서 조건부(FormRequest) 검증이 우회되거나 과도하게 제한되던 결함을 한 계열로 묶어 수정한다.

- 검증 규칙 확장 필터 이름 10개를 나머지 92개와 동일 규칙으로 통일
- 설명 길이를 바이트가 아닌 글자 수로 계산(한글 오검출 해소)
- 비활성 언어팩 번역 보존한 채 수정 허용
- 알림 템플릿 제목 nullable 존중, 미지원 발송 수단 저장 차단
- 일정/실행 이력 목록 조회 검증 정합
- 게시판 첨부 미사용 시 빈 확장자 통과, new_display_hours/최소 길이 0 허용, 일괄 적용 override_values 범위 검증(SSoT)
- 이커머스 클레임 사유 조회 검증, 용어 정정
- 원문 i18n 키 노출 다수 해소, 저장/삭제 실패 시 사유 화면 노출
- 아이콘 크기/다크모드 대비 회귀 수정

번들 확장(모듈3·플러그인6·템플릿2·언어팩5) 버전 new-bump 및 CHANGELOG 반영, ja 언어팩 동기.
2026-07-23 10:54:17 +09:00

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();
}
}