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" 로 보고되던 문제도 런너 사전 검증으로 막았다. 정렬 게이트 대조 하네스는 관계 정렬 변형만 쓰는 저장소를 탐지하지 못한 채 통과시키고 있었다. 탐지·제외·인자 파싱을 함께 고치고 단위 테스트를 신설했다.
62 lines
2.5 KiB
PHP
62 lines
2.5 KiB
PHP
<?php
|
|
|
|
namespace Tests\Feature\Extension;
|
|
|
|
use App\Extension\ModuleManager;
|
|
use App\Extension\PluginManager;
|
|
use App\Extension\TemplateManager;
|
|
use Tests\TestCase;
|
|
|
|
/**
|
|
* `*:check-updates` 커맨드가 확인 결과를 사실대로 표시하는지 고정하는 테스트
|
|
*
|
|
* 매니저의 `checkAll*ForUpdates()` 는 `details` 에 **업데이트 가능한 항목만** 담고,
|
|
* 각 항목에 `update_available` 키를 넣지 않는다. 커맨드가 `$detail['update_available'] ?? false`
|
|
* 로 상태를 읽으면 업데이트가 있는 확장을 전부 "최신" 으로 표시하고 요약도 "0개" 가 된다.
|
|
*
|
|
* 운영자가 커맨드로 확인하면 "업데이트 없음" 을 보고 넘어가게 되므로, 표시 계약을
|
|
* 여기서 고정한다.
|
|
*/
|
|
class CheckUpdatesCommandReportingTest extends TestCase
|
|
{
|
|
/**
|
|
* 매니저가 돌려주는 details 항목이 표시에 필요한 키를 갖추는지 검증합니다.
|
|
*
|
|
* @return void
|
|
*/
|
|
public function test_details_항목이_업데이트_여부를_스스로_말한다(): void
|
|
{
|
|
foreach ([ModuleManager::class, PluginManager::class, TemplateManager::class] as $manager) {
|
|
$source = file_get_contents((new \ReflectionClass($manager))->getFileName());
|
|
|
|
$this->assertMatchesRegularExpression(
|
|
"/'update_available' => (true|\\\$result\['update_available'\]),\s*\n\s*'identifier'|'identifier' => \\\$identifier,\s*\n\s*'update_available'/",
|
|
$source,
|
|
class_basename($manager).'::details 항목에 update_available 이 없으면 커맨드가 상태를 판별할 수 없습니다'
|
|
);
|
|
}
|
|
}
|
|
|
|
/**
|
|
* 커맨드가 요약 건수를 "확인한 전체" 가 아닌 값으로 오기재하지 않는지 검증합니다.
|
|
*
|
|
* @return void
|
|
*/
|
|
public function test_커맨드가_업데이트_가능_건수를_details_로_센다(): void
|
|
{
|
|
foreach ([
|
|
'app/Console/Commands/Module/CheckModuleUpdatesCommand.php',
|
|
'app/Console/Commands/Plugin/CheckPluginUpdatesCommand.php',
|
|
'app/Console/Commands/Template/CheckTemplateUpdatesCommand.php',
|
|
] as $path) {
|
|
$source = file_get_contents(base_path($path));
|
|
|
|
$this->assertStringNotContainsString(
|
|
"\$detail['update_available'] ?? false",
|
|
$source,
|
|
basename($path).': 없는 키를 기본 false 로 읽으면 업데이트 가능 항목이 "최신" 으로 표시됩니다'
|
|
);
|
|
}
|
|
}
|
|
}
|