HeuJung
b10e89bef0
목록 조회 성능 축(깊은 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" 로 보고되던 문제도 런너 사전 검증으로 막았다. 정렬 게이트 대조 하네스는 관계 정렬 변형만 쓰는 저장소를 탐지하지 못한 채 통과시키고 있었다. 탐지·제외·인자 파싱을 함께 고치고 단위 테스트를 신설했다.
88 lines
3.3 KiB
PHP
88 lines
3.3 KiB
PHP
<?php
|
|
|
|
namespace Tests\Unit\Benchmark;
|
|
|
|
use App\Benchmark\ListIndexAdvisor;
|
|
use Illuminate\Foundation\Testing\RefreshDatabase;
|
|
use Tests\Support\Concerns\AssertsListIndexCoverage;
|
|
use Tests\TestCase;
|
|
|
|
/**
|
|
* 코어 목록 프로파일의 색인 커버리지 검사
|
|
*
|
|
* 지연 조인은 inner 가 키 컬럼만 읽게 만들지만, 그 inner 가 인덱스 순서 그대로 끝나야
|
|
* 깊은 OFFSET 이 싸진다. 정렬을 덮는 색인이 없으면 inner 도 filesort 로 전체를 훑어
|
|
* 개선 폭이 사라지는데, 이 실패는 화면에서 "뒤로 갈수록 느리다" 로만 드러난다.
|
|
*
|
|
* 필요한 색인은 프로파일 선언(filters / order / soft_delete)에서 도출한다 — 새 목록이
|
|
* 프로파일을 선언하면 필요한 색인이 자동으로 검사 대상이 되므로, 색인 설계를 매번 손으로
|
|
* 반복하지 않아도 된다.
|
|
*
|
|
* @scenario case=list_index_coverage
|
|
*
|
|
* @effects list_profiles_declare_matching_index
|
|
*/
|
|
class ListIndexCoverageTest extends TestCase
|
|
{
|
|
use AssertsListIndexCoverage;
|
|
use RefreshDatabase;
|
|
|
|
public function test_core_list_profiles_have_matching_indexes(): void
|
|
{
|
|
$this->assertListIndexCoverage('core');
|
|
}
|
|
|
|
/**
|
|
* 도출 규칙 자체가 맞는지 검사합니다.
|
|
*
|
|
* 커버리지 검사가 초록이어도 도출이 틀렸으면 엉뚱한 색인을 요구하거나 요구하지 않는다.
|
|
* 판정기를 판정한다.
|
|
*/
|
|
public function test_required_columns_are_derived_from_declaration(): void
|
|
{
|
|
$advisor = new ListIndexAdvisor;
|
|
|
|
$this->assertSame(
|
|
['board_id', 'is_notice', 'deleted_at', 'created_at', 'id'],
|
|
$advisor->requiredColumns([
|
|
'filters' => ['board_id' => 1, 'is_notice' => 0],
|
|
'soft_delete' => true,
|
|
'order' => [['created_at', 'desc'], ['id', 'desc']],
|
|
]),
|
|
'등치 필터 → deleted_at → 정렬 → 기본키 순서로 도출되어야 한다'
|
|
);
|
|
|
|
$this->assertSame(
|
|
['deleted_at', 'ordered_at', 'id'],
|
|
$advisor->requiredColumns([
|
|
// in / not in 은 선행 컬럼이 등치로 고정되지 않아 뒤의 정렬을 인덱스로 쓸 수 없다
|
|
'filters' => ['order_status' => ['not in', ['draft', 'hidden']]],
|
|
'soft_delete' => true,
|
|
'order' => [['ordered_at', 'desc'], ['id', 'desc']],
|
|
]),
|
|
'비-등치 필터는 선행 컬럼에서 제외되어야 한다'
|
|
);
|
|
|
|
$this->assertSame(
|
|
['created_at', 'id'],
|
|
$advisor->requiredColumns([
|
|
'order' => [['created_at', 'desc']],
|
|
]),
|
|
'정렬 끝에 기본키가 없으면 자동으로 덧붙여야 한다'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* 면제는 사유가 있을 때만 인정되는지 검사합니다.
|
|
*/
|
|
public function test_exemption_requires_a_reason(): void
|
|
{
|
|
$advisor = new ListIndexAdvisor;
|
|
|
|
$this->assertNull($advisor->exemptionReason([]));
|
|
$this->assertNull($advisor->exemptionReason(['index_exempt' => ' ']));
|
|
$this->assertNull($advisor->exemptionReason(['index_exempt' => true]));
|
|
$this->assertSame('행 수 고정', $advisor->exemptionReason(['index_exempt' => '행 수 고정']));
|
|
}
|
|
}
|