HeuJung
b5c0ec1178
관리자 상품목록을 한 페이지 여는 것만으로 그 페이지 모든 상품의 옵션이 응답에 실렸다. 같은 패턴을 저장소 전역에서 찾아 14개 목록 엔드포인트를 함께 정리했다. 근본 원인은 둘이다. Resource 가 whenLoaded 로 방어하는데 Repository 가 목록 쿼리에서 관계를 무조건 로드해 가드가 항상 참이 되는 가짜 가드, 그리고 toListArray 경량 표현을 정의해 두고도 컬렉션이 toArray 를 부르는 목록/상세 미분리다. 둘 다 응답만 보면 정상이라 오류도 경고도 없이 페이로드만 불어난다. 목록은 화면이 실제로 그리는 것만 싣는다. 개수·합계는 PHP 컬렉션 연산이 아니라 DB 집계로, 대표 1건이 필요한 곳은 관계 자체를 oldestOfMany 로 좁힌다. eager load 의 limit(1) 은 부모별이 아니라 배치 쿼리 전체에 걸려 첫 행만 값을 갖게 되므로 쓸 수 없다. 뺀 값에는 대체 경로를 먼저 만들었다. 상품 옵션은 행을 펼칠 때 배치로 불러오고(상품 수와 무관하게 쿼리 상수), 종전 동작이 필요한 호출자를 위해 ?with_options=1 등 opt-in 을 남겼다. 배송정책 국가설정과 리뷰 첨부 이미지는 소비처를 실측한 결과 화면이 실제로 그리고 있어 제거하지 않았다 — 그 소비 사실을 회귀 테스트로 고정했다. 재발 방지로 정적 검사 룰 4종과 규정 문서 항목을 함께 넣었다.
97 lines
3.3 KiB
PHP
97 lines
3.3 KiB
PHP
<?php
|
|
|
|
namespace Tests\Unit;
|
|
|
|
use App\Http\Requests\Admin\LayoutVersionListRequest;
|
|
use Tests\TestCase;
|
|
|
|
/**
|
|
* 버전 목록 조회 상한의 서버 ↔ 편집기 일치 검사
|
|
*
|
|
* 편집기의 '더 보기' 는 조회 상한을 한 묶음씩 넓히는 방식이라, 화면이 서버 상한을 모르면
|
|
* 넓히기를 멈추지 못하고 상한을 넘는 순간 목록 전체가 422 가 된다. 이미 보고 있던 이력까지
|
|
* 사라지므로 두 상수는 반드시 같아야 한다.
|
|
*
|
|
* 상수를 한쪽만 바꾸면 이 테스트가 깨진다.
|
|
*/
|
|
class LayoutVersionLimitParityTest extends TestCase
|
|
{
|
|
/** 편집기 훅 경로 (상수 SSoT 의 소비 측) */
|
|
private const HOOK_PATH = 'resources/js/core/template-engine/layout-editor/hooks/useLayoutVersions.ts';
|
|
|
|
/**
|
|
* 편집기의 VERSION_MAX_LIMIT 이 서버 MAX_LIMIT 과 같다.
|
|
*
|
|
* @return void
|
|
*/
|
|
public function test_editor_max_limit_matches_server_max_limit(): void
|
|
{
|
|
$this->assertSame(
|
|
LayoutVersionListRequest::MAX_LIMIT,
|
|
$this->readNumericConstant('VERSION_MAX_LIMIT'),
|
|
'편집기 VERSION_MAX_LIMIT 이 서버 LayoutVersionListRequest::MAX_LIMIT 과 다르다. '.
|
|
'화면이 서버 상한을 넘는 limit 을 보내면 버전 목록이 422 로 비어 버린다.'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* 편집기의 한 묶음 크기가 서버 기본 조회 건수와 같다.
|
|
*
|
|
* 첫 화면이 서버 기본값과 다른 수를 요청하면 '더 보기' 노출 판정("묶음이 가득 찼는가")이
|
|
* 어긋나, 남은 이력이 있는데도 버튼이 사라지거나 그 반대가 된다.
|
|
*
|
|
* @return void
|
|
*/
|
|
public function test_editor_page_size_matches_server_default_limit(): void
|
|
{
|
|
$this->assertSame(
|
|
LayoutVersionListRequest::DEFAULT_LIMIT,
|
|
$this->readNumericConstant('VERSION_PAGE_SIZE'),
|
|
'편집기 VERSION_PAGE_SIZE 가 서버 DEFAULT_LIMIT 과 다르다.'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* 상한이 한 묶음보다 크다 — 그렇지 않으면 '더 보기' 가 처음부터 무의미하다.
|
|
*
|
|
* @return void
|
|
*/
|
|
public function test_max_limit_leaves_room_for_at_least_one_more_batch(): void
|
|
{
|
|
$this->assertGreaterThan(
|
|
LayoutVersionListRequest::DEFAULT_LIMIT,
|
|
LayoutVersionListRequest::MAX_LIMIT,
|
|
'상한이 기본 조회 건수 이하면 더 보기로 넓힐 여지가 없다.'
|
|
);
|
|
}
|
|
|
|
/**
|
|
* 편집기 훅에서 숫자 상수 값을 읽습니다.
|
|
*
|
|
* @param string $name 상수 이름
|
|
* @return int 상수 값
|
|
*/
|
|
private function readNumericConstant(string $name): int
|
|
{
|
|
$path = base_path(self::HOOK_PATH);
|
|
|
|
$this->assertFileExists($path, '편집기 버전 훅 경로가 바뀌었다면 본 테스트의 경로도 함께 고쳐야 한다.');
|
|
|
|
$source = file_get_contents($path);
|
|
|
|
$matched = preg_match(
|
|
'/export\s+const\s+'.preg_quote($name, '/').'\s*=\s*(\d+)\s*;/',
|
|
$source,
|
|
$matches
|
|
);
|
|
|
|
$this->assertSame(
|
|
1,
|
|
$matched,
|
|
"편집기 훅에서 {$name} 선언을 찾지 못했다 (이름 변경 또는 선언 형태 변경)."
|
|
);
|
|
|
|
return (int) $matches[1];
|
|
}
|
|
}
|