목록 조회 성능 축(깊은 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" 로 보고되던 문제도 런너 사전 검증으로 막았다. 정렬 게이트 대조 하네스는 관계 정렬 변형만 쓰는 저장소를 탐지하지 못한 채 통과시키고 있었다. 탐지·제외·인자 파싱을 함께 고치고 단위 테스트를 신설했다.
137 lines
5.4 KiB
PHP
137 lines
5.4 KiB
PHP
<?php
|
|
|
|
namespace Tests\Support\Concerns;
|
|
|
|
use App\Benchmark\BenchmarkProfileRegistry;
|
|
use App\Benchmark\ListIndexAdvisor;
|
|
use App\Enums\BenchmarkAxis;
|
|
|
|
/**
|
|
* 목록 프로파일이 요구하는 색인이 스키마에 있는지 검사하는 공통 로직.
|
|
*
|
|
* 코어 스위트는 코어 프로파일만, 각 모듈 스위트는 자기 프로파일만 검사한다. 모듈 테이블은
|
|
* 그 모듈 스위트에서만 마이그레이션되므로, 코어 스위트에 모두 몰면 테이블이 없어 건너뛰고
|
|
* **초록인 채 미검사**가 된다.
|
|
*
|
|
* 같은 이유로 "테이블이 없으면 skip" 도 하지 않는다. 검사 대상이 자기 스위트에 있는데
|
|
* 테이블이 없다는 것은 마이그레이션이 빠졌다는 뜻이므로 실패로 보고한다.
|
|
*/
|
|
trait AssertsListIndexCoverage
|
|
{
|
|
/**
|
|
* 지정한 출처의 목록 프로파일이 요구하는 색인을 전수 검사합니다.
|
|
*
|
|
* @param string $sourceIdentifier 프로파일 출처 (코어는 'core', 모듈은 모듈 식별자)
|
|
*/
|
|
protected function assertListIndexCoverage(string $sourceIdentifier): void
|
|
{
|
|
$registry = app(BenchmarkProfileRegistry::class);
|
|
|
|
$profiles = array_filter(
|
|
$registry->byAxis(BenchmarkAxis::ListQuery),
|
|
fn ($profile) => $profile->sourceIdentifier === $sourceIdentifier
|
|
);
|
|
|
|
$this->assertNotEmpty(
|
|
$profiles,
|
|
"출처 '{$sourceIdentifier}' 의 목록 프로파일을 하나도 찾지 못했다 — "
|
|
.'검사 모집단이 비어 이 테스트가 아무것도 검증하지 않는다'
|
|
);
|
|
|
|
$this->assertIndexCoverageForOptions(
|
|
array_map(fn ($profile) => $profile->options ?? [], $profiles)
|
|
);
|
|
}
|
|
|
|
/**
|
|
* 확장 클래스의 선언을 직접 읽어 검사합니다.
|
|
*
|
|
* 확장 프로파일은 레지스트리에서 "활성 모듈" 을 거쳐 수집되는데, 테스트 DB 에는 확장이
|
|
* 활성으로 등록돼 있지 않아 모집단이 비게 된다. 검사 대상은 DB 상태가 아니라 **코드에
|
|
* 선언된 것**이므로 선언을 직접 읽는다.
|
|
*
|
|
* @param object $extension 모듈/플러그인 인스턴스
|
|
*/
|
|
protected function assertListIndexCoverageForExtension(object $extension): void
|
|
{
|
|
$this->assertTrue(
|
|
method_exists($extension, 'getBenchmarkProfiles'),
|
|
get_class($extension).' 에 getBenchmarkProfiles() 가 없다 — 계측 대상 선언이 사라졌는지 확인할 것'
|
|
);
|
|
|
|
$declared = $extension->getBenchmarkProfiles();
|
|
|
|
$lists = array_filter(
|
|
$declared,
|
|
fn ($options) => is_array($options) && ($options['type'] ?? null) === 'list'
|
|
);
|
|
|
|
$this->assertNotEmpty(
|
|
$lists,
|
|
get_class($extension).' 의 목록(list) 프로파일이 하나도 없다 — '
|
|
.'검사 모집단이 비어 이 테스트가 아무것도 검증하지 않는다'
|
|
);
|
|
|
|
$this->assertIndexCoverageForOptions($lists);
|
|
}
|
|
|
|
/**
|
|
* 프로파일 옵션 배열을 검사합니다.
|
|
*
|
|
* @param array<string, array<string, mixed>> $profileOptions 프로파일 키 → 옵션
|
|
*/
|
|
private function assertIndexCoverageForOptions(array $profileOptions): void
|
|
{
|
|
$advisor = app(ListIndexAdvisor::class);
|
|
$failures = [];
|
|
|
|
foreach ($profileOptions as $key => $options) {
|
|
$row = array_merge(
|
|
['profile' => (string) $key],
|
|
$advisor->inspect($options),
|
|
['exemption' => $advisor->exemptionReason($options)]
|
|
);
|
|
if ($row['exemption'] !== null) {
|
|
// 면제는 사유와 함께 선언된 경우에만 인정한다
|
|
continue;
|
|
}
|
|
|
|
$required = implode(', ', $row['required']);
|
|
|
|
$failures[] = match ($row['status']) {
|
|
'satisfied' => null,
|
|
'tiebreak_missing' => sprintf(
|
|
"%s (%s): 색인 '%s' 가 정렬을 덮지만 끝에 기본키가 없어 동률 구간에서 filesort 가 남는다. "
|
|
.'필요: (%s). 기존 색인에 기본키를 덧붙여 교체할 것',
|
|
$row['profile'],
|
|
$row['table'],
|
|
$row['partial_by'],
|
|
$required
|
|
),
|
|
'missing' => sprintf(
|
|
'%s (%s): 정렬을 덮는 색인이 없어 뒤쪽 페이지가 테이블 전체를 훑는다. 필요: (%s)',
|
|
$row['profile'],
|
|
$row['table'],
|
|
$required
|
|
),
|
|
'table_absent' => sprintf(
|
|
'%s (%s): 테이블이 없다 — 이 스위트에서 마이그레이션되지 않았다 (건너뛰면 미검사가 초록으로 보인다)',
|
|
$row['profile'],
|
|
$row['table']
|
|
),
|
|
default => sprintf('%s: 대상 테이블 선언이 없다', $row['profile']),
|
|
};
|
|
}
|
|
|
|
$failures = array_values(array_filter($failures));
|
|
|
|
$this->assertSame(
|
|
[],
|
|
$failures,
|
|
'목록 색인 미충족 '.count($failures)."건:\n - ".implode("\n - ", $failures)
|
|
."\n\n색인이 불필요한 목록(행 수 고정·상한이 작음)은 프로파일에 "
|
|
."'index_exempt' => '사유' 를 선언해 면제한다."
|
|
);
|
|
}
|
|
}
|