합의되지 않은 구조 변경을 되돌리고, 계획서 전수조사에서 드러난 결함을 처리했다. 쇼핑 화면 호출 통합(F4)은 API 쿼리 튜닝이 아니라 화면 구조 변경이라 원복했다. 레이아웃 9개를 develop 상태로 복원하고 전용 엔드포인트·컨트롤러·문서·테스트를 걷어냈다. 되돌리면 함께 사라지는 비-F4 수정 둘(페이저 has_more_pages 판정 3곳, Icon 크기 클래스 23줄)은 재적용했다. 성능 개선분은 전부 잔존한다. 통합검색 화면 상태를 전역 상태에서 URL 쿼리로 옮겼다. 종전에는 탭·필터·정렬·페이지가 주소에 없어 새로고침·뒤로가기에서 초기화되고 결과를 공유할 수도 없었다. 화면은 정상으로 보이고 콘솔 에러도 없어 코드만으로는 드러나지 않는 형태였다. 확장이 라우트를 추가해도 라우트 캐시가 걸린 사이트에서는 그 주소가 404 가 되던 구멍을 막았다. 훅 캐시와 달리 라우트 캐시에는 스캔 폴백이 없어 예외도 경고도 남지 않는다. 같은 문제를 이미 푼 ConfigCacheHelper 를 미러링해 갱신 지점을 한 곳으로 모았고, 코어 업데이트가 캐시를 비우기만 하고 되살리지 않던 문제도 함께 고쳤다. 상품·페이지 검색이 커서 판정에 페이지 번호를 넘기지 않아, 커서 없이 깊은 페이지를 지목한 딥링크가 조용히 1 페이지를 돌려주던 결함을 고쳤다. 호출 지점을 손으로 열거하는 대신 저장소를 훑는 패리티 테스트로 다음 도메인의 재발을 막는다. 환경설정 고급 탭의 목록 상한값이 저장되지 않던 결함을 고쳤다. 화면·검증·읽기는 모두 있었는데 저장 시 카테고리 분류표가 손으로 열거돼 있어 그 값만 버려지고 있었다. 분류표를 설정 정의에서 도출하도록 바꿔 다음 카테고리가 합류해도 같은 일이 생기지 않게 했다. 범위를 벗어난 값의 안내에 내부 식별자가 노출되던 것도 함께 정리했다. 언어팩 재설치가 활성 팩을 자기 자신과의 슬롯 충돌로 오인해 강등하던 회귀와, CLI 설치가 HTTP 권한 컨텍스트 없이 자동 활성화되는지를 고정하는 테스트를 함께 담았다.
212 lines
6.5 KiB
PHP
212 lines
6.5 KiB
PHP
<?php
|
|
|
|
namespace Tests\Unit\Search;
|
|
|
|
use App\Search\SearchPagePolicy;
|
|
use Tests\TestCase;
|
|
|
|
/**
|
|
* 커서 판정에 페이지 번호가 도달하는지 검사하는 패리티 테스트 (#519)
|
|
*
|
|
* {@see SearchPagePolicy::usesCursor()} 는 커서가 없을 때 `page <= 1` 인지로
|
|
* "시작점" 과 "깊은 페이지 딥링크" 를 가른다. 그래서 호출자가 페이지 번호를 넘기지 않으면
|
|
* 기본값 1 이 적용되어 **모든 요청이 시작점으로 판정**된다. 그 결과 `?page=3` 딥링크가
|
|
* 조용히 1 페이지를 돌려주고, 예외도 오류 응답도 남지 않는다.
|
|
*
|
|
* 호출 지점을 손으로 열거하면 검색에 새로 합류하는 도메인이 같은 함정을 다시 밟는다.
|
|
* 여기서는 저장소를 훑어 **모든 호출 지점**이 페이지 번호를 넘기는지 확인한다.
|
|
*
|
|
* @scenario case=search_cursor_page_propagation
|
|
*
|
|
* @effects every_uses_cursor_call_forwards_page
|
|
*/
|
|
class SearchCursorPagePropagationParityTest extends TestCase
|
|
{
|
|
/** 판정 함수 호출 형태 */
|
|
private const CALL_NEEDLE = 'SearchPagePolicy::usesCursor(';
|
|
|
|
/** 훑을 소스 루트 (선언 파일 자신은 제외) */
|
|
private const SOURCE_ROOTS = [
|
|
'app',
|
|
'modules/_bundled',
|
|
'plugins/_bundled',
|
|
];
|
|
|
|
/**
|
|
* 커서 판정을 호출하는 모든 곳이 페이지 번호를 함께 넘기는지 확인
|
|
*
|
|
* @effects every_uses_cursor_call_forwards_page
|
|
*/
|
|
public function test_every_uses_cursor_call_forwards_the_page_number(): void
|
|
{
|
|
$callSites = $this->collectCallSites();
|
|
|
|
// 스캐너가 아무것도 못 찾으면 초록은 아무 의미가 없다 — 모집단부터 단언한다.
|
|
$this->assertNotEmpty(
|
|
$callSites,
|
|
self::CALL_NEEDLE.' 호출을 한 건도 찾지 못했다. 스캔 대상 경로가 잘못됐을 수 있다'
|
|
);
|
|
|
|
$missing = [];
|
|
|
|
foreach ($callSites as $site) {
|
|
if ($this->countArguments($site['arguments']) < 4 && ! str_contains($site['arguments'], 'page:')) {
|
|
$missing[] = $site['file'].':'.$site['line'];
|
|
}
|
|
}
|
|
|
|
$this->assertSame(
|
|
[],
|
|
$missing,
|
|
"커서 판정에 페이지 번호를 넘기지 않는 호출이 있다. 커서 없이 깊은 페이지를 지목한 요청이\n".
|
|
"첫 페이지로 조용히 되돌아간다.\n".
|
|
implode("\n", $missing)
|
|
);
|
|
}
|
|
|
|
/**
|
|
* 저장소에서 커서 판정 호출 지점을 모읍니다.
|
|
*
|
|
* @return array<int, array{file: string, line: int, arguments: string}>
|
|
*/
|
|
private function collectCallSites(): array
|
|
{
|
|
$sites = [];
|
|
|
|
foreach ($this->phpFiles() as $file) {
|
|
$contents = file_get_contents($file);
|
|
|
|
if ($contents === false || ! str_contains($contents, self::CALL_NEEDLE)) {
|
|
continue;
|
|
}
|
|
|
|
$relative = str_replace('\\', '/', substr($file, strlen(base_path()) + 1));
|
|
|
|
// 선언 파일과 문서 주석의 참조는 호출이 아니다.
|
|
if (str_ends_with($relative, 'app/Search/SearchPagePolicy.php')) {
|
|
continue;
|
|
}
|
|
|
|
foreach (explode("\n", $contents) as $index => $line) {
|
|
$offset = strpos($line, self::CALL_NEEDLE);
|
|
|
|
if ($offset === false || $this->isComment($line)) {
|
|
continue;
|
|
}
|
|
|
|
$sites[] = [
|
|
'file' => $relative,
|
|
'line' => $index + 1,
|
|
'arguments' => $this->extractArguments($line, $offset + strlen(self::CALL_NEEDLE) - 1),
|
|
];
|
|
}
|
|
}
|
|
|
|
return $sites;
|
|
}
|
|
|
|
/**
|
|
* 스캔 대상 PHP 파일 목록을 만듭니다.
|
|
*
|
|
* @return array<int, string>
|
|
*/
|
|
private function phpFiles(): array
|
|
{
|
|
$files = [];
|
|
|
|
foreach (self::SOURCE_ROOTS as $root) {
|
|
$path = base_path($root);
|
|
|
|
if (! is_dir($path)) {
|
|
continue;
|
|
}
|
|
|
|
$iterator = new \RecursiveIteratorIterator(
|
|
new \RecursiveDirectoryIterator($path, \FilesystemIterator::SKIP_DOTS)
|
|
);
|
|
|
|
foreach ($iterator as $entry) {
|
|
if ($entry->isFile() && $entry->getExtension() === 'php') {
|
|
$files[] = $entry->getPathname();
|
|
}
|
|
}
|
|
}
|
|
|
|
return $files;
|
|
}
|
|
|
|
/**
|
|
* 여는 괄호 위치에서 시작해 짝이 맞는 닫는 괄호까지의 인자 문자열을 잘라냅니다.
|
|
*
|
|
* @param string $line 호출이 있는 소스 한 줄
|
|
* @param int $openParen 여는 괄호의 위치
|
|
* @return string 괄호 안 인자 문자열
|
|
*/
|
|
private function extractArguments(string $line, int $openParen): string
|
|
{
|
|
$depth = 0;
|
|
$length = strlen($line);
|
|
|
|
for ($i = $openParen; $i < $length; $i++) {
|
|
if ($line[$i] === '(') {
|
|
$depth++;
|
|
} elseif ($line[$i] === ')') {
|
|
$depth--;
|
|
|
|
if ($depth === 0) {
|
|
return substr($line, $openParen + 1, $i - $openParen - 1);
|
|
}
|
|
}
|
|
}
|
|
|
|
// 여러 줄로 끊긴 호출은 그 줄에서 닫히지 않는다 — 남은 부분을 그대로 돌려준다.
|
|
return substr($line, $openParen + 1);
|
|
}
|
|
|
|
/**
|
|
* 괄호 중첩을 고려해 최상위 쉼표 기준으로 인자 수를 셉니다.
|
|
*
|
|
* @param string $arguments 괄호 안 인자 문자열
|
|
* @return int 인자 수
|
|
*/
|
|
private function countArguments(string $arguments): int
|
|
{
|
|
if (trim($arguments) === '') {
|
|
return 0;
|
|
}
|
|
|
|
$depth = 0;
|
|
$count = 1;
|
|
$length = strlen($arguments);
|
|
|
|
for ($i = 0; $i < $length; $i++) {
|
|
$char = $arguments[$i];
|
|
|
|
if ($char === '(' || $char === '[') {
|
|
$depth++;
|
|
} elseif ($char === ')' || $char === ']') {
|
|
$depth--;
|
|
} elseif ($char === ',' && $depth === 0) {
|
|
$count++;
|
|
}
|
|
}
|
|
|
|
return $count;
|
|
}
|
|
|
|
/**
|
|
* 그 줄이 주석인지 판정합니다.
|
|
*
|
|
* @param string $line 소스 한 줄
|
|
* @return bool 주석이면 true
|
|
*/
|
|
private function isComment(string $line): bool
|
|
{
|
|
$trimmed = ltrim($line);
|
|
|
|
return str_starts_with($trimmed, '*')
|
|
|| str_starts_with($trimmed, '//')
|
|
|| str_starts_with($trimmed, '/*');
|
|
}
|
|
}
|