목록 조회 성능 축(깊은 OFFSET·정렬·색인)과 그 검증 계층에서 계획서 전수검수와 3회에 걸친 감사가 지적한 항목을 처리했다. 세부 경위는 의 각 회차 문서에 있다. 검증 계층 — 컨트롤러가 base Request 를 직접 주입받던 확장 13곳을 전용 FormRequest 로 옮겼다. 옮기면서 기존 동작 계약은 그대로 두었다: 상한 초과 limit 을 거부하지 않고 상한까지 반환하던 공개 API, per_page 를 범위로 조정하던 관리자 목록, 미지원 period 를 year 로 해석하던 인기글 목록 모두 종전과 같은 응답을 낸다. 상한/폴백을 rules 로 승격시키면 200 이던 응답이 422 가 되어 기존 링크가 깨지므로, 규칙은 타입만 닫고 클램프·폴백은 접근자가 맡는다. period 는 접근자가 닫힌 집합만 반환해 캐시 키 공간도 함께 닫힌다. ckeditor5 이미지 업로드만 ResponseHelper 봉투를 쓰지 않는다. 응답을 파싱하는 주체가 CDN 으로 로드되는 상위 CKEditor5 43.3.1 의 SimpleUploadAdapter 라 규약을 바꿀 수 없어, 각 응답 지점에 사유를 명시한 면제를 부착하고 근거를 API 문서에 남겼다. 하네스 — 룰 5개의 대상 경로 패턴이 매처와 맞지 않아 번들 확장 컨트롤러가 검사 대상에서 통째로 빠져 있었다. 패턴을 고치자 확장 위반 17건이 드러나 전건 처리했다. severity 오타가 요약 집계 양쪽에 안 잡혀 "0 error" 로 보고되던 문제도 런너 사전 검증으로 막았다. 정렬 게이트 대조 하네스는 관계 정렬 변형만 쓰는 저장소를 탐지하지 못한 채 통과시키고 있었다. 탐지·제외·인자 파싱을 함께 고치고 단위 테스트를 신설했다.
209 lines
7.4 KiB
PHP
209 lines
7.4 KiB
PHP
<?php
|
|
|
|
namespace Modules\Sirsoft\Board\Tests\Feature;
|
|
|
|
// ModuleTestCase를 수동으로 require (autoload 전에 로드 필요)
|
|
require_once __DIR__.'/../../ModuleTestCase.php';
|
|
|
|
use App\Models\Permission;
|
|
use App\Models\Role;
|
|
use App\Models\User;
|
|
use Illuminate\Support\Facades\App;
|
|
use Illuminate\Support\Facades\DB;
|
|
use Modules\Sirsoft\Board\Enums\ReportStatus;
|
|
use Modules\Sirsoft\Board\Models\Board;
|
|
use Modules\Sirsoft\Board\Models\Report;
|
|
use Modules\Sirsoft\Board\Tests\ModuleTestCase;
|
|
|
|
/**
|
|
* 신고 관리 컨트롤러의 전용 FormRequest 검증 회귀 테스트 (#413-74 구조 정리)
|
|
*
|
|
* index/reporters/getStatusCounts 가 base Request 주입에서 전용 FormRequest 로 전환되며
|
|
* 입력 검증이 컨트롤러 인라인 검사에서 FormRequest 로 이관되었다. 이 전환이 기존 동작
|
|
* (빈 ids → 422, 잘못된 target_status → 422, 정상 조회 200)을 깨지 않음을 보장한다.
|
|
*/
|
|
class ReportRequestValidationTest extends ModuleTestCase
|
|
{
|
|
private User $admin;
|
|
|
|
private Board $board;
|
|
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
config(['telescope.enabled' => false]);
|
|
App::setLocale('ko');
|
|
|
|
DB::table('boards_report_logs')->delete();
|
|
DB::table('boards_reports')->delete();
|
|
DB::table('users')->where('is_super', false)->delete();
|
|
|
|
$this->admin = User::factory()->create();
|
|
|
|
$viewPermission = Permission::firstOrCreate(
|
|
['identifier' => 'sirsoft-board.reports.view'],
|
|
['name' => ['ko' => '신고 조회', 'en' => 'View Reports'], 'type' => 'admin']
|
|
);
|
|
$managePermission = Permission::firstOrCreate(
|
|
['identifier' => 'sirsoft-board.reports.manage'],
|
|
['name' => ['ko' => '신고 관리', 'en' => 'Manage Reports'], 'type' => 'admin']
|
|
);
|
|
|
|
$adminRole = Role::where('identifier', 'admin')->first();
|
|
$adminRole->permissions()->syncWithoutDetaching([$viewPermission->id, $managePermission->id]);
|
|
$this->admin->roles()->attach($adminRole->id);
|
|
|
|
$this->board = Board::updateOrCreate(
|
|
['slug' => 'report-request-validation-board'],
|
|
[
|
|
'name' => ['ko' => '신고 검증 테스트 게시판', 'en' => 'Report Validation Test Board'],
|
|
'slug' => 'report-request-validation-board',
|
|
'type' => 'list',
|
|
'per_page' => 20,
|
|
'per_page_mobile' => 10,
|
|
'order_by' => 'created_at',
|
|
'order_direction' => 'DESC',
|
|
'secret_mode' => 'disabled',
|
|
'use_comment' => true,
|
|
'use_reply' => false,
|
|
'use_file_upload' => false,
|
|
'use_report' => true,
|
|
'blocked_keywords' => [],
|
|
'notify_admin_on_post' => false,
|
|
'notify_author_on_comment' => false,
|
|
]
|
|
);
|
|
|
|
DB::table('boards_report_logs')->delete();
|
|
DB::table('boards_reports')->delete();
|
|
}
|
|
|
|
/**
|
|
* status-counts 요청에 ids 가 비어 있으면 422 로 거부됨을 검증합니다.
|
|
* (기존 컨트롤러 인라인 empty($ids) 검사 → StatusCountsRequest 이관)
|
|
*/
|
|
public function test_status_counts_with_empty_ids_is_rejected_with_422(): void
|
|
{
|
|
$response = $this->actingAs($this->admin)
|
|
->postJson('/api/modules/sirsoft-board/admin/reports/status-counts', [
|
|
'ids' => [],
|
|
]);
|
|
|
|
$response->assertStatus(422);
|
|
}
|
|
|
|
/**
|
|
* status-counts 요청에 ids 키 자체가 없어도 422 로 거부됨을 검증합니다.
|
|
*/
|
|
public function test_status_counts_without_ids_is_rejected_with_422(): void
|
|
{
|
|
$response = $this->actingAs($this->admin)
|
|
->postJson('/api/modules/sirsoft-board/admin/reports/status-counts', []);
|
|
|
|
$response->assertStatus(422);
|
|
}
|
|
|
|
/**
|
|
* status-counts 요청의 target_status 가 허용되지 않은 값이면 422 로 거부됨을 검증합니다.
|
|
*/
|
|
public function test_status_counts_with_invalid_target_status_is_rejected_with_422(): void
|
|
{
|
|
$report = Report::factory()->create([
|
|
'board_id' => $this->board->id,
|
|
'status' => ReportStatus::Pending,
|
|
]);
|
|
|
|
$response = $this->actingAs($this->admin)
|
|
->postJson('/api/modules/sirsoft-board/admin/reports/status-counts', [
|
|
'ids' => [$report->id],
|
|
'target_status' => 'not-a-real-status',
|
|
]);
|
|
|
|
$response->assertStatus(422);
|
|
}
|
|
|
|
/**
|
|
* 유효한 ids 로 status-counts 를 요청하면 200 으로 상태별 집계가 반환됨을 검증합니다.
|
|
* (전용 FormRequest 전환이 정상 경로를 깨지 않음)
|
|
*/
|
|
public function test_status_counts_with_valid_ids_succeeds(): void
|
|
{
|
|
$report = Report::factory()->create([
|
|
'board_id' => $this->board->id,
|
|
'status' => ReportStatus::Pending,
|
|
]);
|
|
|
|
$response = $this->actingAs($this->admin)
|
|
->postJson('/api/modules/sirsoft-board/admin/reports/status-counts', [
|
|
'ids' => [$report->id],
|
|
'target_status' => 'review',
|
|
]);
|
|
|
|
$response->assertStatus(200)
|
|
->assertJson(['success' => true]);
|
|
}
|
|
|
|
/**
|
|
* 전용 FormRequest 전환 후에도 신고 목록 조회(index)가 정상 200 임을 검증합니다.
|
|
*/
|
|
public function test_index_still_succeeds_after_request_refactor(): void
|
|
{
|
|
Report::factory()->create([
|
|
'board_id' => $this->board->id,
|
|
'status' => ReportStatus::Pending,
|
|
]);
|
|
|
|
$response = $this->actingAs($this->admin)
|
|
->getJson('/api/modules/sirsoft-board/admin/reports');
|
|
|
|
$response->assertStatus(200)
|
|
->assertJson(['success' => true]);
|
|
}
|
|
|
|
/**
|
|
* 전용 FormRequest 전환 후에도 신고자 목록 조회(reporters)가 정상 200 임을 검증합니다.
|
|
*/
|
|
public function test_reporters_still_succeeds_after_request_refactor(): void
|
|
{
|
|
$report = Report::factory()->create([
|
|
'board_id' => $this->board->id,
|
|
'status' => ReportStatus::Pending,
|
|
]);
|
|
|
|
$response = $this->actingAs($this->admin)
|
|
->getJson("/api/modules/sirsoft-board/admin/reports/{$report->id}/reporters");
|
|
|
|
$response->assertStatus(200)
|
|
->assertJson(['success' => true]);
|
|
}
|
|
|
|
/**
|
|
* 신고자 목록의 per_page 가 상·하한 밖 값을 거부하는지 검증합니다.
|
|
*
|
|
* 상한이 없으면 한 요청으로 신고자 전량을 끌어올 수 있어 페이지네이션이 무의미해진다.
|
|
*/
|
|
public function test_reporters_per_page_is_bounded(): void
|
|
{
|
|
$report = Report::factory()->create([
|
|
'board_id' => $this->board->id,
|
|
'status' => ReportStatus::Pending,
|
|
]);
|
|
|
|
$url = "/api/modules/sirsoft-board/admin/reports/{$report->id}/reporters";
|
|
|
|
foreach ([0, -3, 101, 100000] as $perPage) {
|
|
$this->actingAs($this->admin)
|
|
->getJson("{$url}?per_page={$perPage}")
|
|
->assertStatus(422);
|
|
}
|
|
|
|
// 경계 안쪽 값은 그대로 통과해야 한다
|
|
foreach ([1, 100] as $perPage) {
|
|
$this->actingAs($this->admin)
|
|
->getJson("{$url}?per_page={$perPage}")
|
|
->assertStatus(200);
|
|
}
|
|
}
|
|
}
|