HeuJung
cc705c2f21
escapeEnvValue 재선언 fatal 이 통과한 사각지대를 구조적으로 봉인한다. 인스톨러는 Laravel 오토로드 밖의 플레인 PHP require 체인이라, 엔드포인트가 "어떤 파일을 어떤 순서로 조립하는가" 가 곧 함수 선언 순서다. 기존 단위 테스트는 함수를 직접 호출하므로 이 조립 계층을 전혀 검증하지 못했고, 그래서 재선언 fatal(빈 응답 → 프론트 JSON 파싱 에러)이 그대로 통과했다. 추가: 엔드포인트 5종(install-process / state-management / finalize-env / install-worker / index)의 실제 require 체인을 격리 프로세스로 재현하고, 그 엔드포인트가 호출하는 함수가 모두 정의되는지 + fatal 이 없는지 검증. 이로써 "Call to undefined function"(정의 파일 미포함) 과 "Cannot redeclare" (가드 없는 중복 선언) 두 fatal 유형이 모두 차단된다 — 증상은 빈 응답으로 동일하지만 원인이 달라 각각 잡아야 한다. 가드 제거 시 9건 중 3건이 red 로 전환됨을 확인 (거짓 통과 아님).
333 lines
14 KiB
PHP
333 lines
14 KiB
PHP
<?php
|
|
|
|
namespace Tests\Unit\Installer;
|
|
|
|
use PHPUnit\Framework\TestCase;
|
|
|
|
/**
|
|
* 인스톨러 include 파일의 로드 순서 독립성 회귀 가드 (이슈 #465 후속).
|
|
*
|
|
* 회귀: `install-process.php` 가 `installer-runtime.php` 를 `functions.php` 보다
|
|
* *먼저* require 하도록 바뀌면서 다음 fatal 이 발생했다.
|
|
*
|
|
* Fatal error: Cannot redeclare escapeEnvValue()
|
|
* (previously declared in installer-runtime.php) in functions.php
|
|
*
|
|
* 원인: `installer-runtime.php` 는 `escapeEnvValue()` 를 `function_exists` 가드와
|
|
* 함께 polyfill 로 선언하는데, `functions.php` 는 같은 함수를 **가드 없이 무조건**
|
|
* 선언한다. 따라서 runtime 이 먼저 로드되면 functions 로드 시 재선언 fatal.
|
|
*
|
|
* 증상: PHP fatal 이라 JSON 이 출력되지 않음 → 빈 응답 본문 →
|
|
* 브라우저의 `res.json()` 이 "Unexpected end of JSON input" 으로 실패
|
|
* → 설치 시작 즉시 "설치 실패" (SSE/폴링 양쪽 동일 — 두 모드 모두
|
|
* install-process.php 를 경유하므로)
|
|
*
|
|
* 본 테스트가 고정하는 계약:
|
|
* 인스톨러 include 파일들은 **어떤 순서로 로드되어도** 함수 재선언 fatal 을
|
|
* 일으키지 않는다. 각 엔드포인트가 require 순서를 바꿔도 안전해야 한다.
|
|
*
|
|
* 별도 프로세스로 실행하는 이유: 함수 선언은 프로세스 전역이라 PHPUnit 프로세스
|
|
* 안에서 두 파일을 로드하면 다른 Installer 테스트를 오염시킨다. 실제 fatal 여부를
|
|
* 관측하려면 격리된 PHP 프로세스에서 로드해야 한다.
|
|
*/
|
|
class InstallerIncludeOrderTest extends TestCase
|
|
{
|
|
private string $projectRoot = '';
|
|
|
|
protected function setUp(): void
|
|
{
|
|
parent::setUp();
|
|
|
|
$this->projectRoot = dirname(__DIR__, 3);
|
|
}
|
|
|
|
/**
|
|
* 인스톨러 include 파일 2개를 지정한 순서로 로드하는 격리 PHP 프로세스 실행.
|
|
*
|
|
* @param array<int, string> $relativeIncludes BASE_PATH 기준 상대 경로 목록
|
|
* @return array{exitCode:int, output:string}
|
|
*/
|
|
private function loadInIsolatedProcess(array $relativeIncludes): array
|
|
{
|
|
$root = str_replace('\\', '/', $this->projectRoot);
|
|
|
|
$lines = ['error_reporting(E_ALL);', "define('BASE_PATH', '{$root}');"];
|
|
foreach ($relativeIncludes as $include) {
|
|
$lines[] = "require_once '{$root}/{$include}';";
|
|
}
|
|
$lines[] = "echo 'LOAD_OK';";
|
|
|
|
$code = implode("\n", $lines);
|
|
|
|
$descriptors = [1 => ['pipe', 'w'], 2 => ['pipe', 'w']];
|
|
$process = proc_open([PHP_BINARY, '-r', $code], $descriptors, $pipes);
|
|
|
|
$this->assertIsResource($process, 'PHP 하위 프로세스를 띄우지 못함');
|
|
|
|
$stdout = stream_get_contents($pipes[1]);
|
|
$stderr = stream_get_contents($pipes[2]);
|
|
fclose($pipes[1]);
|
|
fclose($pipes[2]);
|
|
$exitCode = proc_close($process);
|
|
|
|
return ['exitCode' => $exitCode, 'output' => $stdout.$stderr];
|
|
}
|
|
|
|
/**
|
|
* 회귀 재현 — installer-runtime.php 를 functions.php 보다 먼저 로드.
|
|
*
|
|
* 이 순서가 곧 install-process.php 의 require 순서였고, escapeEnvValue()
|
|
* 재선언 fatal 로 빈 응답이 발생했다.
|
|
*/
|
|
public function test_runtime_before_functions_does_not_fatal(): void
|
|
{
|
|
$result = $this->loadInIsolatedProcess([
|
|
'public/install/includes/installer-runtime.php',
|
|
'public/install/includes/functions.php',
|
|
]);
|
|
|
|
$this->assertStringNotContainsString(
|
|
'Cannot redeclare',
|
|
$result['output'],
|
|
'installer-runtime.php 를 functions.php 보다 먼저 로드하면 함수 재선언 fatal 이 발생한다. '
|
|
.'중복 선언되는 함수에 function_exists 가드가 필요하다.',
|
|
);
|
|
$this->assertStringContainsString('LOAD_OK', $result['output'], '두 파일 로드가 완료되어야 함');
|
|
$this->assertSame(0, $result['exitCode'], 'fatal 없이 정상 종료해야 함');
|
|
}
|
|
|
|
/**
|
|
* 기존 순서(functions.php 먼저) 도 계속 동작해야 한다 — 회귀 방지.
|
|
*
|
|
* finalize-env.php / task-runner.php 가 이 순서를 사용한다.
|
|
*/
|
|
public function test_functions_before_runtime_does_not_fatal(): void
|
|
{
|
|
$result = $this->loadInIsolatedProcess([
|
|
'public/install/includes/functions.php',
|
|
'public/install/includes/installer-runtime.php',
|
|
]);
|
|
|
|
$this->assertStringNotContainsString('Cannot redeclare', $result['output']);
|
|
$this->assertStringContainsString('LOAD_OK', $result['output']);
|
|
$this->assertSame(0, $result['exitCode']);
|
|
}
|
|
|
|
/**
|
|
* install-process.php 의 실제 require 순서를 그대로 재현.
|
|
*
|
|
* 이 엔드포인트가 SSE / 폴링 양 모드의 공통 시작점이므로, 여기서 fatal 이 나면
|
|
* 두 모드 모두 설치 시작 즉시 실패한다.
|
|
*/
|
|
public function test_install_process_require_chain_does_not_fatal(): void
|
|
{
|
|
$result = $this->loadInIsolatedProcess([
|
|
'public/install/includes/config.php',
|
|
'public/install/includes/installer-state.php',
|
|
'public/install/includes/installer-runtime.php',
|
|
'public/install/includes/functions.php',
|
|
]);
|
|
|
|
$this->assertStringNotContainsString(
|
|
'Cannot redeclare',
|
|
$result['output'],
|
|
'install-process.php 의 require 체인에서 재선언 fatal 발생 — '
|
|
.'설치 시작 API 가 빈 응답을 반환하여 "Unexpected end of JSON input" 이 된다',
|
|
);
|
|
$this->assertSame(0, $result['exitCode']);
|
|
}
|
|
|
|
/**
|
|
* 각 인스톨러 엔드포인트의 실제 require 체인을 그대로 로드했을 때, 그 엔드포인트가
|
|
* 호출하는 함수들이 모두 정의되는지 검증한다.
|
|
*
|
|
* 인스톨러는 Laravel 오토로드 밖의 플레인 PHP require 체인이라, 어떤 파일에 함수
|
|
* 호출을 추가하면 그 함수의 정의 파일이 **그 엔드포인트의 체인에 실제로 포함되어
|
|
* 있는지** 를 사람이 직접 확인해야 한다. 빠뜨리면 "Call to undefined function"
|
|
* fatal → 빈 응답 → 프론트의 JSON 파싱 에러가 된다 (escapeEnvValue 재선언 fatal 과
|
|
* 증상이 동일하지만 원인은 다름).
|
|
*
|
|
* 단위 테스트는 함수를 직접 호출하므로 이 축을 잡지 못한다 — 엔드포인트의 조립
|
|
* 계층을 재현해야만 검출된다.
|
|
*
|
|
* @dataProvider endpointChainProvider
|
|
*
|
|
* @param array<int, string> $chain 엔드포인트의 top-level require 순서
|
|
* @param array<int, string> $requiredFunctions 그 엔드포인트가 호출하는 함수들
|
|
*/
|
|
public function test_endpoint_require_chain_defines_all_functions_it_calls(
|
|
string $endpoint,
|
|
array $chain,
|
|
array $requiredFunctions,
|
|
): void {
|
|
$root = str_replace('\\', '/', $this->projectRoot);
|
|
|
|
$lines = ['error_reporting(E_ALL);'];
|
|
foreach ($chain as $include) {
|
|
$lines[] = "require_once '{$root}/{$include}';";
|
|
}
|
|
foreach ($requiredFunctions as $fn) {
|
|
$lines[] = "if (! function_exists('{$fn}')) { echo 'MISSING:{$fn} '; }";
|
|
}
|
|
$lines[] = "echo 'CHECK_DONE';";
|
|
|
|
$descriptors = [1 => ['pipe', 'w'], 2 => ['pipe', 'w']];
|
|
$process = proc_open([PHP_BINARY, '-r', implode("\n", $lines)], $descriptors, $pipes);
|
|
$this->assertIsResource($process);
|
|
|
|
$output = stream_get_contents($pipes[1]).stream_get_contents($pipes[2]);
|
|
fclose($pipes[1]);
|
|
fclose($pipes[2]);
|
|
proc_close($process);
|
|
|
|
$this->assertStringNotContainsString(
|
|
'Fatal error',
|
|
$output,
|
|
"{$endpoint} 의 require 체인 로드 중 fatal 발생 — 이 엔드포인트는 빈 응답을 반환한다:\n".$output,
|
|
);
|
|
$this->assertStringNotContainsString(
|
|
'MISSING:',
|
|
$output,
|
|
"{$endpoint} 가 호출하는 함수가 그 require 체인에서 정의되지 않는다 "
|
|
.'— 실행 시 "Call to undefined function" fatal 이 발생한다:'."\n".$output,
|
|
);
|
|
$this->assertStringContainsString('CHECK_DONE', $output);
|
|
}
|
|
|
|
/**
|
|
* 엔드포인트별 (실제 require 체인, 그 엔드포인트가 호출하는 함수 목록).
|
|
*
|
|
* 인스톨러 엔드포인트에 새 함수 호출을 추가하면 여기에도 등록한다.
|
|
*
|
|
* @return array<string, array{0:string, 1:array<int,string>, 2:array<int,string>}>
|
|
*/
|
|
public static function endpointChainProvider(): array
|
|
{
|
|
return [
|
|
'install-process.php (SSE/폴링 공통 시작점)' => [
|
|
'install-process.php',
|
|
[
|
|
'public/install/includes/config.php',
|
|
'public/install/includes/session.php',
|
|
'public/install/includes/installer-state.php',
|
|
'public/install/includes/functions.php',
|
|
'public/install/includes/installer-runtime.php',
|
|
],
|
|
[
|
|
'sanitizeConfigForState',
|
|
'buildInstallerRuntimeFromState',
|
|
'writeInstallerRuntime',
|
|
'readInstallerRuntime',
|
|
'getInstallationState',
|
|
'saveInstallationState',
|
|
'applyExistingDbActionStateGuard',
|
|
'escapeEnvValue',
|
|
],
|
|
],
|
|
'state-management.php (reset / abort)' => [
|
|
'state-management.php',
|
|
[
|
|
'public/install/includes/config.php',
|
|
'public/install/includes/session.php',
|
|
'public/install/includes/functions.php',
|
|
'public/install/includes/installer-state.php',
|
|
'public/install/api/rollback-functions.php',
|
|
],
|
|
[
|
|
'redactInstallationStateSecrets',
|
|
'saveInstallationState',
|
|
'getInstallationState',
|
|
'rollbackCurrentTask',
|
|
'executeSeedTruncate',
|
|
],
|
|
],
|
|
'finalize-env.php' => [
|
|
'finalize-env.php',
|
|
[
|
|
'public/install/includes/config.php',
|
|
'public/install/includes/functions.php',
|
|
'public/install/includes/installer-runtime.php',
|
|
'public/install/includes/installer-state.php',
|
|
],
|
|
[
|
|
'redactInstallationStateSecrets',
|
|
'getInstallationState',
|
|
'saveInstallationState',
|
|
'readInstallerRuntime',
|
|
'mergeRuntimeIntoEnv',
|
|
'deleteInstallerRuntime',
|
|
],
|
|
],
|
|
'install-worker.php (SSE 워커 → task-runner)' => [
|
|
'install-worker.php',
|
|
[
|
|
'public/install/includes/config.php',
|
|
'public/install/includes/functions.php',
|
|
'public/install/includes/installer-state.php',
|
|
'public/install/includes/progress-emitter.php',
|
|
'public/install/includes/task-runner.php',
|
|
],
|
|
[
|
|
'purgeAdminPasswordAfterSeeding',
|
|
'hydrateDbSecretsFromRuntime',
|
|
'redactInstallationStateSecrets',
|
|
'readInstallerRuntime',
|
|
'writeInstallerRuntime',
|
|
'executeSeedTruncate',
|
|
],
|
|
],
|
|
'index.php (Step3 폼 / 완료 감지)' => [
|
|
'index.php',
|
|
[
|
|
'public/install/includes/config.php',
|
|
'public/install/includes/functions.php',
|
|
'public/install/includes/session.php',
|
|
'public/install/includes/installer-state.php',
|
|
'public/install/includes/request-handler.php',
|
|
],
|
|
[
|
|
'sanitizeConfigForState',
|
|
'isInstallationCompleted',
|
|
'installerSecretConfigKeys',
|
|
],
|
|
],
|
|
];
|
|
}
|
|
|
|
/**
|
|
* installer-runtime.php 가 polyfill 로 선언하는 함수는 functions.php 에서도
|
|
* function_exists 가드를 가져야 한다 — 정적 계약 검사.
|
|
*
|
|
* 앞으로 두 파일에 중복 함수가 새로 추가되어도 가드 없이는 통과하지 못한다.
|
|
*/
|
|
public function test_functions_shared_with_runtime_are_all_guarded(): void
|
|
{
|
|
$functionsPath = $this->projectRoot.'/public/install/includes/functions.php';
|
|
$runtimePath = $this->projectRoot.'/public/install/includes/installer-runtime.php';
|
|
|
|
$functionsSource = (string) file_get_contents($functionsPath);
|
|
$runtimeSource = (string) file_get_contents($runtimePath);
|
|
|
|
// installer-runtime.php 가 선언하는 함수 목록 추출
|
|
preg_match_all('/^\s*function\s+([a-zA-Z_][a-zA-Z0-9_]*)\s*\(/m', $runtimeSource, $m);
|
|
$runtimeFunctions = array_unique($m[1]);
|
|
|
|
$this->assertNotEmpty($runtimeFunctions, 'installer-runtime.php 에서 함수를 하나도 못 찾음 — 추출 정규식 점검 필요');
|
|
|
|
foreach ($runtimeFunctions as $fn) {
|
|
// functions.php 에도 같은 함수가 선언되어 있는가?
|
|
if (! preg_match('/^\s*function\s+'.preg_quote($fn, '/').'\s*\(/m', $functionsSource)) {
|
|
continue; // 중복 아님 — 안전
|
|
}
|
|
|
|
// 중복이라면 functions.php 쪽에 function_exists 가드가 있어야 한다
|
|
$this->assertMatchesRegularExpression(
|
|
"/function_exists\(\s*['\"]".preg_quote($fn, '/')."['\"]\s*\)/",
|
|
$functionsSource,
|
|
"functions.php 의 {$fn}() 이 installer-runtime.php 와 중복 선언되는데 "
|
|
."function_exists 가드가 없다. 로드 순서에 따라 'Cannot redeclare' fatal 이 발생한다.",
|
|
);
|
|
}
|
|
}
|
|
}
|