move_detect/server/tests/DeviceScopeTest.php
유창욱 43a3d069a3 fix(audit): 1.1.0 3차 재감사 — 확정 결함 14건 전량 해소
4개 관점 병렬 감사 후 각 발견을 반증 시도해 확정. 반증된 4건은 감사 문서에
REFUTED 로 남겼다(히트맵 왜곡·에스컬레이션 전파·스트림 필터 고정·설정 미배선).

CRITICAL 2건 — 둘 다 이번 기능이 만든 회귀다:

R3-1 dispatch_alerts.php 의 장치 건전성 단계가 try 밖에 있었다. 이 단계만
move_devices·move_device_metrics 를 읽는데 통보 경로는 그 두 표를 건드리지
않는다 — 즉 마이그레이션 미적용·커넥션 한도·lock timeout 이 원래대로면 정상
발송됐을 실행을 통째로 죽인다. 원인이 지속형이라 매분 반복되고 콘솔은 멀쩡해
보인다. 1·2단계를 각각 격리했고, 3단계의 exit(1) 도 없앴다 — 거기 도달하는
예외는 저장소 장애이고 4단계 재시도가 바로 그것에서 회복하려고 존재한다.

R3-2 간헐 고장에 감지기가 양방향으로 무너졌다. edge/daemon.py:105 는 성공
프레임 하나에 streak 를 지워 cameras_erroring 를 0 으로 되돌린다. 정상 표본
하나로 저하 구간을 끊으니 (a) 두 시간 고장에도 sustain 을 못 넘어 아무것도
못 올리고 (b) 끊긴 구간마다 새 앵커가 생겨 6시간에 54건을 쏟아냈다. 엣지
낙상 감지기의 recover_grace_s 와 같은 규율로 짧은 정상을 흡수하고, uuid 에
재고지 버킷을 더해 주기당 1건으로 묶었다(같은 입력 ≤7건). 기존 16개 테스트가
전부 단조 시퀀스라 이 결함이 CI를 통과했다 — 그래서 플래핑·폭주·회복·미래시각
테스트를 추가했다.

HIGH 5건: 센티넬이 메일·웹훅으로 회사 밖에 나갔고(라벨을 app.js 에만 넣은 것이
원인), 사건 상세 링크로 존재하지 않는 카메라에 구역을 저장할 수 있었고, 인제스트가
센티넬을 예약값으로 안 막아 엣지가 그 camera_id 로 올린 진짜 낙상이 화면에서
더 안 보이게 됐고, BUILD-INFO 가 릴리스 이전 커밋을 가리켰고, 실제 자격증명
파일이 납품 폴더 안에 있었다.

값이 흩어진 것이 센티넬 결함의 근본 원인이라 단일 출처를 만들었다:
App\DeviceScope(서버) + assets/device-scope.js(프런트) + PdoEventRepository 의
NOT_DEVICE SQL 조각. 셋이 어긋나면 DeviceScopeTest 가 잡는다.

MEDIUM 7건: 카메라별 집계 3곳 제외, 설정 점검 오판정, 개선조치 대상 거부,
라벨 CSV 제외, 낡은 지표로 판정 금지, 미래 시각 표본 제거, 죽은 노드 재고지.

검증: phpunit 250 passed (신규 10) · pytest 는 dist 재생성 후 확인.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01EBjuR7DW6PNNMows4KnbYU
2026-09-05 00:06:35 +09:00

73 lines
3.2 KiB
PHP

<?php
declare(strict_types=1);
namespace App\Tests;
use App\DeviceScope;
use App\Ingest\EventValidator;
use PHPUnit\Framework\TestCase;
/**
* 센티넬 값이 흩어지는 것을 막는 테스트.
*
* 이 프로젝트에서 실제로 난 사고: 값을 DeviceHealthWatch 와 app.js 두 곳에만 적어 두었더니
* 콘솔 스트림에서만 "장치 전체" 로 보이고 상세·리포트·사고조사 패키지·CSV, 그리고 **메일과
* 웹훅** 에는 `__device__` 가 그대로 나갔다. 값은 PHP 상수 하나, JS 파일 하나, 그리고 SQL
* 리터럴 하나(상수를 넣을 수 없으므로) — 이 셋이 어긋나면 여기서 잡는다.
*/
final class DeviceScopeTest extends TestCase
{
public function testLabelIsUsedForTheSentinelAndPassesEverythingElseThrough(): void
{
self::assertSame('장치 전체', DeviceScope::label(DeviceScope::CAMERA));
self::assertSame('cam-01', DeviceScope::label('cam-01'));
self::assertSame('', DeviceScope::label(null));
self::assertTrue(DeviceScope::isDevice(DeviceScope::CAMERA));
self::assertFalse(DeviceScope::isDevice('cam-01'));
self::assertFalse(DeviceScope::isDevice(null));
}
public function testFrontEndSharedScriptCarriesTheSameValue(): void
{
$js = file_get_contents(__DIR__ . '/../public/assets/device-scope.js');
self::assertNotFalse($js);
self::assertStringContainsString("var CAMERA = '" . DeviceScope::CAMERA . "';", $js);
self::assertStringContainsString("var LABEL = '" . DeviceScope::LABEL . "';", $js);
}
public function testSqlExclusionFragmentCarriesTheSameValue(): void
{
// PdoEventRepository 는 카메라별 집계에서 센티넬을 빼는데, SQL 리터럴에는 PHP 상수를
// 넣을 수 없어 값이 한 번 더 적힌다. 그 사본이 진짜와 같은지 본다.
$php = file_get_contents(__DIR__ . '/../src/Repository/Pdo/PdoEventRepository.php');
self::assertNotFalse($php);
self::assertStringContainsString(
'private const NOT_DEVICE = "camera_id <> ' . "'" . DeviceScope::CAMERA . "'" . '";', $php);
}
public function testIngestRejectsTheReservedCameraId(): void
{
// 센티넬은 서버 소유다. 엣지가 이 값으로 낙상을 올리면 콘솔이 그것을 "장치 전체" 로
// 이름 붙이고 카메라 목록·구역·통계에서 빼버려, 진짜 사고가 화면에서 더 안 보이게 된다.
$res = (new EventValidator())->validate([
'device_id' => 'pi-01',
'camera_id' => DeviceScope::CAMERA,
'event_type' => 'fall',
'severity' => 'high',
'occurred_at' => 1_700_000_000,
]);
self::assertFalse($res['ok']);
self::assertContains('reserved:camera_id', $res['errors']);
}
public function testIngestStillAcceptsAnOrdinaryCameraId(): void
{
$res = (new EventValidator())->validate([
'device_id' => 'pi-01',
'camera_id' => 'cam-01',
'event_type' => 'fall',
'severity' => 'high',
'occurred_at' => 1_700_000_000,
]);
self::assertTrue($res['ok'], implode(',', $res['errors'] ?? []));
}
}