Files
Gnuboard7/tests/TestCase.php
T
HeuJung 7f8fb6870f fix(core,board,ecommerce): 조건부 검증 결함 패밀리 일괄 수정 및 실패 안내 노출
여러 화면에서 조건부(FormRequest) 검증이 우회되거나 과도하게 제한되던 결함을 한 계열로 묶어 수정한다.

- 검증 규칙 확장 필터 이름 10개를 나머지 92개와 동일 규칙으로 통일
- 설명 길이를 바이트가 아닌 글자 수로 계산(한글 오검출 해소)
- 비활성 언어팩 번역 보존한 채 수정 허용
- 알림 템플릿 제목 nullable 존중, 미지원 발송 수단 저장 차단
- 일정/실행 이력 목록 조회 검증 정합
- 게시판 첨부 미사용 시 빈 확장자 통과, new_display_hours/최소 길이 0 허용, 일괄 적용 override_values 범위 검증(SSoT)
- 이커머스 클레임 사유 조회 검증, 용어 정정
- 원문 i18n 키 노출 다수 해소, 저장/삭제 실패 시 사유 화면 노출
- 아이콘 크기/다크모드 대비 회귀 수정

번들 확장(모듈3·플러그인6·템플릿2·언어팩5) 버전 new-bump 및 CHANGELOG 반영, ja 언어팩 동기.
2026-07-23 10:54:17 +09:00

289 lines
12 KiB
PHP

<?php
namespace Tests;
use App\Contracts\Notifications\ChannelReadinessCheckerInterface;
use App\Contracts\Repositories\ConfigRepositoryInterface;
use App\Extension\Testing\ExtensionTestAllowlist;
use App\Listeners\Identity\EnforceIdentityPolicyListener;
use Illuminate\Foundation\Testing\TestCase as BaseTestCase;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Facades\Storage;
abstract class TestCase extends BaseTestCase
{
/**
* 좀비 커넥션 정리 실행 여부 (프로세스당 1회)
*/
private static bool $staleConnectionsCleaned = false;
/**
* 테스트에 필요한 확장 목록
*
* 각 테스트 클래스에서 오버라이드하여 필요한 확장의 마이그레이션을 로드합니다.
* 경로는 프로젝트 루트 기준 상대 경로입니다.
*
* 예: ['plugins/sirsoft-marketing', 'modules/sirsoft-ecommerce']
*
* @var array<string>
*/
protected array $requiredExtensions = [];
/**
* 테스트 환경 설정
*
* RefreshDatabase 트레이트가 migrate:fresh를 실행하기 전에
* g7_testing DB의 좀비 커넥션을 정리합니다.
*
* 근본 원인: 테스트 프로세스 강제 종료 시 MySQL 커넥션이
* 트랜잭션 락을 유지한 채 남아있어 DROP TABLE이 metadata lock 대기에 빠짐
*
* 확장 로딩 allowlist 도 여기서 주입합니다. parent::setUp() 의
* createApplication() 단계에서 모든 ServiceProvider 의 register()/boot() 가
* 실행되므로, allowlist 는 그 이전(= parent::setUp() 호출 전)에 설정되어야
* provider 자동 등록 가드가 올바른 시점에 적용됩니다.
*
* settings 디스크는 앱 부팅 직후 페이크로 대체합니다 (아래 fakeSettingsDisk).
*/
protected function setUp(): void
{
// 앱 생성 전에 확장 allowlist 설정 (provider register 가드 적용 시점 보장)
ExtensionTestAllowlist::set($this->resolveAllowedExtensions());
$this->afterApplicationCreated(function () {
$this->fakeSettingsDisk();
});
if (! self::$staleConnectionsCleaned) {
// Laravel 앱 부팅 전이므로 부트스트랩 후 콜백으로 등록
$this->afterApplicationCreated(function () {
$this->killStaleTestingConnections();
self::$staleConnectionsCleaned = true;
});
}
parent::setUp();
}
/**
* settings 디스크를 페이크로 대체하여 실제 설정 파일을 보호합니다.
*
* `settings` 디스크의 root 는 `storage/app/settings` 로, 개발/운영 환경이 실제로
* 사용하는 설정 파일 그 자체입니다. 페이크로 대체하지 않으면 설정 저장 경로를 타는
* 테스트(설정 API 호출, ConfigRepository::saveCategory 등)가 실제 설정을 덮어쓰고,
* RefreshDatabase 는 DB 만 되돌리므로 그 오염이 그대로 남습니다.
*
* 실제 피해 사례: 설정 저장 테스트가 `general.json` 의 사이트명/언어를 테스트 값으로
* 덮어써 개발 사이트 언어가 ja 로 바뀌어 있었고, SEO/캐시 TTL 도 테스트 값으로 대체됨.
*
* 설정 *읽기* 는 부팅 시 Config(`g7_settings.*`)로 이미 적재된 뒤이므로 이 페이크의
* 영향을 받지 않습니다. 저장소를 직접 읽어야 하는 테스트는 페이크 디스크에 직접
* 값을 넣고 검증합니다.
*/
private function fakeSettingsDisk(): void
{
Storage::fake('settings');
}
/**
* mail 채널을 발송 준비 완료 상태로 만듭니다.
*
* 테스트 환경의 mail 설정은 from_address 가 플레이스홀더(noreply@example.com)라
* ChannelReadinessService::checkMail() 이 not-ready 로 판정하고, GenericNotification::via()
* 가 mail 채널을 제외합니다. 즉 메일 발송을 단언하는 테스트는 이 헬퍼로 명시적으로
* "메일이 설정된 사이트" 를 만들어야 합니다.
*
* (설정 미비 시 발송하지 않는 것은 의도된 제품 동작이므로 기본값은 그대로 둡니다)
*/
protected function enableMailChannelReadiness(): void
{
app(ConfigRepositoryInterface::class)->saveCategory('mail', [
'mailer' => 'smtp',
'host' => 'smtp.test.local',
'port' => 587,
'encryption' => 'tls',
'from_address' => 'test@g7.test',
'from_name' => 'G7 Test',
]);
app(ChannelReadinessCheckerInterface::class)->check('mail');
}
/**
* 테스트 종료 시 확장 allowlist 를 초기화합니다.
*
* 프로세스 내 다음 테스트 클래스가 stale allowlist 를 물려받지 않도록
* 보장합니다.
*/
protected function tearDown(): void
{
ExtensionTestAllowlist::reset();
// IDV 동적 hook 구독 정적 상태를 초기화한다. 프로세스 내 다음 테스트가
// 이전 테스트에서 등록된 모듈 hook 콜백을 물려받아 의도치 않게 발화되는
// 누수를 차단한다 (프로덕션은 단일 부팅이라 무관 — 테스트 격리 전용).
EnforceIdentityPolicyListener::resetDynamicSubscriptions();
parent::tearDown();
}
/**
* 이 테스트가 허용하는 확장 목록을 계산합니다.
*
* requiredExtensions 선언에 더해, 확장 자체 테스트 베이스 클래스가
* selfExtension() 으로 반환한 자기 확장을 자동으로 포함합니다.
* → 확장 테스트 클래스는 자기 확장을 명시하지 않아도 격리에서 허용됩니다.
*
* @return array<string>
*/
protected function resolveAllowedExtensions(): array
{
$extensions = $this->requiredExtensions;
$self = $this->selfExtension();
if ($self !== null) {
$extensions[] = $self;
}
return array_values(array_unique($extensions));
}
/**
* 이 테스트 클래스가 속한 확장의 경로를 반환합니다.
*
* 테스트 클래스 파일의 물리 경로를 검사하여 `modules/<id>` 또는
* `plugins/<id>` (활성 디렉토리) / `modules/_bundled/<id>` 등을
* 자동 탐지합니다. 확장 자체 테스트는 이 메서드 덕분에 자기 확장을
* 명시하지 않아도 격리 allowlist 에 자동 포함됩니다.
*
* 코어 테스트(tests/ 하위)는 패턴에 매칭되지 않으므로 null 을 반환합니다.
*
* @return string|null 'modules/<id>' / 'plugins/<id>' 또는 null
*/
protected function selfExtension(): ?string
{
try {
$file = (new \ReflectionClass($this))->getFileName();
} catch (\ReflectionException) {
return null;
}
if ($file === false) {
return null;
}
// 경로 구분자 정규화 (Windows 백슬래시 → 슬래시)
$normalized = str_replace('\\', '/', $file);
// modules/<id>/... 또는 modules/_bundled/<id>/... 에서 확장 식별자 추출
if (preg_match('#/(modules|plugins)/(?:_bundled/)?([^/]+)/#', $normalized, $m)) {
return $m[1].'/'.$m[2];
}
return null;
}
/**
* RefreshDatabase 등 트레이트 초기화 전에 확장 마이그레이션을 등록합니다.
*
* Laravel의 setUpTraits()는 RefreshDatabase::refreshDatabase()를 호출하여
* migrate:fresh를 실행합니다. 확장 마이그레이션 경로는 그 전에 등록되어야 합니다.
*
* PHP 메서드 해석 순서: 클래스 > 트레이트 > 부모 클래스
* → beforeRefreshingDatabase()는 RefreshDatabase 트레이트에 의해 가려지므로
* setUpTraits() 오버라이드로 마이그레이션 경로를 먼저 등록합니다.
*
* @return void
*/
protected function setUpTraits()
{
// RefreshDatabase가 migrate:fresh를 실행하기 전에 확장 마이그레이션 경로 등록.
// ① 이 클래스가 선언한 requiredExtensions ② 모든 번들 확장(_bundled) 을 함께 등록한다.
// RefreshDatabase 는 프로세스당 migrate:fresh 를 1회만 실행하므로, 그 1회를
// 트리거하는 첫 테스트 클래스가 어떤 확장을 선언했는지에 따라 다른 클래스가
// 쓰는 확장 테이블이 누락될 수 있다(테스트 순서 의존 — RefreshDatabase 공유 상태).
// 번들 확장 마이그레이션을 항상 등록해 어떤 순서로 실행되든 모든 번들 테이블이
// 생성되게 한다(테이블 생성은 provider 무관 — Schema 빌더만 실행, 격리 무영향).
$this->loadExtensionMigrations();
return parent::setUpTraits();
}
/**
* 확장 마이그레이션 경로를 migrator 에 등록합니다.
*
* RefreshDatabase의 migrate:fresh 실행 시 확장 테이블도 함께 생성되도록
* 마이그레이션 경로를 등록한다. 대상:
* - 이 클래스가 선언한 `requiredExtensions` (활성 디렉토리 경로)
* - 모든 번들 확장(`modules|plugins|templates/_bundled/*`) — 테스트 순서에 무관하게
* 번들 테이블이 항상 생성되도록(공유 RefreshDatabase 상태 누락 방지).
*
* `$migrator->path()` 는 동일 경로 중복 등록을 무시하므로 안전하다. selfExtension()
* (확장 자체 테스트) 은 ModuleTestCase / PluginTestCase 가 migrateFreshUsing() 으로
* 자기 마이그레이션을 직접 처리하므로 여기서 별도 추가하지 않는다.
*/
private function loadExtensionMigrations(): void
{
$migrator = app('migrator');
$paths = [];
// ① 명시 선언한 확장(활성 디렉토리 경로)
foreach ($this->requiredExtensions as $extension) {
$paths[] = base_path($extension.'/database/migrations');
}
// ② 모든 번들 확장(_bundled) — 테스트 순서 무관 테이블 보장
foreach (['modules', 'plugins', 'templates'] as $kind) {
$bundledRoot = base_path($kind.'/_bundled');
if (! is_dir($bundledRoot)) {
continue;
}
foreach (glob($bundledRoot.'/*', GLOB_ONLYDIR) ?: [] as $extDir) {
$paths[] = $extDir.'/database/migrations';
}
}
foreach (array_unique($paths) as $migrationPath) {
if (is_dir($migrationPath)) {
$migrator->path($migrationPath);
}
}
}
/**
* g7_testing DB의 좀비 커넥션을 정리합니다.
*
* 현재 프로세스의 커넥션은 제외하고,
* g7_testing DB에 연결된 다른 모든 커넥션을 KILL합니다.
*/
private function killStaleTestingConnections(): void
{
// phpunit.xml에서 DB_DATABASE=g7_testing으로 설정됨
$testingDb = config('database.connections.mysql.database');
try {
$currentId = DB::selectOne('SELECT CONNECTION_ID() as id')->id;
$processes = DB::select('SHOW PROCESSLIST');
$killed = 0;
foreach ($processes as $process) {
if (($process->db ?? '') === $testingDb && $process->Id !== $currentId) {
try {
DB::statement('KILL '.$process->Id);
$killed++;
} catch (\Throwable) {
// 이미 종료된 커넥션은 무시
}
}
}
if ($killed > 0) {
fwrite(STDERR, "\n[TestCase] Killed {$killed} stale g7_testing connection(s)\n");
}
} catch (\Throwable) {
// DB 연결 실패 시 무시 (첫 마이그레이션에서 처리됨)
}
}
}