테스트가 개발자의 `.env` 를 삭제한 사고 이후 부트스트랩에 복원 안전망을 넣었지만, 그것은 프로세스 종료 시점 복원이라 같은 실행 안에서 뒤 테스트가 훼손된 파일을 읽는 구간이 남아 있었다. 복원이 아니라 애초에 건드리지 않는 구조로 바꾼다. 앱 루트를 임시 디렉토리로 돌리면 `base_path` 를 쓰는 제품 코드는 한 줄도 바뀌지 않은 채 격리된 파일만 다룬다. 공용 트레이트로 묶어 세 클래스가 같은 방식을 쓰게 했다. 격리 자체를 증명하기 위해 안전망을 임시로 끄고 돌렸다. 안전망 없이도 `.env` 가 바이트 단위로 그대로였다 — "green 이니 됐다" 로는 안전망이 뒤에서 복원해 준 것과 구분되지 않는다. 이 과정에서 격리로는 막을 수 없는 유형을 하나 더 찾았다. 별도 프로세스를 띄워 업그레이드 커맨드를 실행하는 테스트가 자식 쪽에서 `.env` 의 버전 값을 덮어쓰고 있었다. 부모의 앱 루트 전환도, 테스트 부트스트랩의 안전망도 자식에는 닿지 않는다. 테스트 환경 지정은 읽는 파일만 바꿀 뿐 쓰기 대상은 언제나 프로젝트 루트의 `.env` 다. 그 테스트가 검증하는 것은 자식이 방금 쓴 클래스를 읽어들이는지 뿐이므로, 커맨드가 이미 제공하는 부수효과 차단 옵션을 붙였다. 같은 구조의 문제가 업그레이드 스텝 임시 파일에도 있었다 — 실제 `upgrades/` 에 쓰고 정리를 tearDown 에 의존하므로 중단되면 개발 체크아웃에 정체불명의 스텝이 남는다. 실제로 두 번 발견해 지웠고, 부트스트랩 셧다운 훅이 그 패턴을 쓸어내도록 했다. 검증: 세 클래스 개별 green, 안전망을 끈 통합 실행 28건 green + `.env` 불변, 자식 프로세스 테스트 green, 전 구간 `.env` 해시 불변, audit 위반 0. 알려진 잔여 문제: 인스톨러와 업그레이드 스위트를 한 프로세스로 합쳐 돌리면 모듈 클래스 중복 선언 fatal 로 중단된다. 각각 따로 돌리면 297 / 301 건 모두 통과하며, 이번 변경 이전에도 같은 증상이었다. 별도 조사가 필요해 이번 범위에서 다루지 않았다.
120 lines
4.6 KiB
PHP
120 lines
4.6 KiB
PHP
<?php
|
|
|
|
namespace Tests\Concerns;
|
|
|
|
use Illuminate\Support\Facades\File;
|
|
|
|
/**
|
|
* 인스톨러/업그레이드 테스트용 앱 루트 격리.
|
|
*
|
|
* ## 왜 필요한가
|
|
*
|
|
* 인스톨러·업그레이드 코드는 `base_path('.env')`, `base_path('storage/installer/runtime.php')`
|
|
* 같은 실제 파일을 읽고 쓴다. 이를 테스트하려면 그 파일들이 있어야 하는데, 앱 루트를 그대로
|
|
* 두면 **개발자의 실제 `.env` 를 삭제·재생성하게 된다.**
|
|
*
|
|
* 각 테스트가 setUp 에서 원본을 메모리에 담고 tearDown 에서 되돌리는 방식은 안전해 보이지만
|
|
* 그렇지 않다 — 테스트가 fatal 로 죽으면 tearDown 이 실행되지 않아 `.env` 가 사라진 채 남는다.
|
|
* `.env` 는 Git 추적 대상이 아니라 복구 수단이 없고, APP_KEY 를 잃으면 기존 암호화 데이터와
|
|
* 세션이 전부 무효가 된다. 2026-07-21 에 실제로 그렇게 유실됐다.
|
|
*
|
|
* 그래서 "복원" 이 아니라 **애초에 건드리지 않는" 방식으로 바꾼다.** 앱 루트를 임시 디렉토리로
|
|
* 돌리면 `base_path()` 를 쓰는 프로덕션 코드는 한 줄도 바뀌지 않은 채 격리된 파일만 다룬다.
|
|
*
|
|
* ## 사용
|
|
*
|
|
* ```php
|
|
* protected function setUp(): void
|
|
* {
|
|
* parent::setUp();
|
|
*
|
|
* // 실제 파일이 필요하면 복사 대상으로 넘긴다 (.env.example 등)
|
|
* $this->isolateInstallerBasePath(['.env.example']);
|
|
*
|
|
* // 프로젝트 실제 파일을 읽어야 할 때만 realBasePath() 사용
|
|
* require_once $this->realBasePath('upgrades/data/7.0.4/migrations/01_Foo.php');
|
|
* }
|
|
*
|
|
* protected function tearDown(): void
|
|
* {
|
|
* $this->releaseInstallerBasePath();
|
|
* parent::tearDown();
|
|
* }
|
|
* ```
|
|
*
|
|
* @see CoreUpdateServiceTest::withIsolatedBasePath 콜백 단위 격리 변형
|
|
*/
|
|
trait IsolatesInstallerBasePath
|
|
{
|
|
/** 격리 이전의 실제 앱 루트 */
|
|
private ?string $realProjectBasePath = null;
|
|
|
|
/** 이번 테스트가 사용하는 임시 앱 루트 */
|
|
private ?string $isolatedBasePath = null;
|
|
|
|
/**
|
|
* 앱 루트를 임시 디렉토리로 전환합니다.
|
|
*
|
|
* 인스톨러 코드가 흔히 기대하는 `storage/`, `storage/app/` 을 미리 만들어 둔다.
|
|
*
|
|
* @param array<int, string> $copyFromProject 실제 프로젝트에서 복사해올 파일 (루트 기준 상대경로)
|
|
* @return string 생성된 임시 앱 루트 절대경로
|
|
*/
|
|
protected function isolateInstallerBasePath(array $copyFromProject = []): string
|
|
{
|
|
$this->realProjectBasePath = $this->app->basePath();
|
|
$this->isolatedBasePath = sys_get_temp_dir()
|
|
.DIRECTORY_SEPARATOR.'g7-installer-isolated-'.bin2hex(random_bytes(6));
|
|
|
|
File::ensureDirectoryExists($this->isolatedBasePath.'/storage/app');
|
|
|
|
foreach ($copyFromProject as $relative) {
|
|
$source = $this->realProjectBasePath.DIRECTORY_SEPARATOR.ltrim($relative, '/\\');
|
|
if (is_file($source)) {
|
|
$destination = $this->isolatedBasePath.DIRECTORY_SEPARATOR.ltrim($relative, '/\\');
|
|
File::ensureDirectoryExists(dirname($destination));
|
|
File::copy($source, $destination);
|
|
}
|
|
}
|
|
|
|
$this->app->setBasePath($this->isolatedBasePath);
|
|
|
|
return $this->isolatedBasePath;
|
|
}
|
|
|
|
/**
|
|
* 앱 루트를 원래대로 되돌리고 임시 디렉토리를 제거합니다.
|
|
*
|
|
* tearDown 에서 호출한다. 호출되지 않아도 실제 프로젝트 파일은 애초에 건드리지 않았으므로
|
|
* 남는 것은 임시 디렉토리뿐이다 — 이것이 복원 의존 방식과의 결정적 차이다.
|
|
*/
|
|
protected function releaseInstallerBasePath(): void
|
|
{
|
|
if ($this->realProjectBasePath !== null) {
|
|
$this->app->setBasePath($this->realProjectBasePath);
|
|
}
|
|
|
|
if ($this->isolatedBasePath !== null && is_dir($this->isolatedBasePath)) {
|
|
File::deleteDirectory($this->isolatedBasePath);
|
|
}
|
|
|
|
$this->realProjectBasePath = null;
|
|
$this->isolatedBasePath = null;
|
|
}
|
|
|
|
/**
|
|
* 실제 프로젝트 루트 기준 경로를 반환합니다 (격리와 무관).
|
|
*
|
|
* 업그레이드 마이그레이션 파일처럼 **실제 소스**를 읽어야 할 때만 사용한다.
|
|
*
|
|
* @param string $relative 프로젝트 루트 기준 상대경로
|
|
* @return string 절대경로
|
|
*/
|
|
protected function realBasePath(string $relative = ''): string
|
|
{
|
|
$root = $this->realProjectBasePath ?? $this->app->basePath();
|
|
|
|
return $relative === '' ? $root : $root.DIRECTORY_SEPARATOR.ltrim($relative, '/\\');
|
|
}
|
|
}
|