test: regenerate only the five goldens this branch actually changed
Attribution was measured, not inferred: the visual suite was run at the merge-base `9e5fbd1` in a detached checkout (13 failed / 117 passed) and the `-actual.png` each run produced was compared by SHA-256 against HEAD's. For 13 of the 18 failures the rendered bytes at HEAD and at the merge-base are identical, so those failures belong to `main`, not here. Regenerated (all Case-editor surfaces, all grown by the Task 10 Asset panel; the +293px band was inspected in the new golden and is the "EVIDENCE / 본문에 Asset 삽입" panel): tech-log-studio-document-edit-360 360x3313 -> 360x3627 tech-log-studio-document-edit-1440 1440x2706 -> 1440x2999 tech-log-studio-case-editor-1440 1440x2706 -> 1440x2999 tech-log-studio-conflict-editor-1440 1440x2706 -> 1440x2999 tech-log-studio-dirty-leave-dialog-1440 1440x2706 -> 1440x2999 `--update-snapshots` was restricted to those five tests with `--grep`; a blanket update would have absorbed the 13 inherited failures and destroyed the distinction. `test:visual` now reports 13 failed / 117 passed, exactly the merge-base's set. Two earlier records are corrected in the spec: the five "pixel-only, not investigated" failures are all inherited, and `TECH_LOG_STUDIO_DOCUMENT_NEW` 360px was wrongly attributed to the Asset Picker -- it fails at the merge-base with the identical 28,413 differing pixels. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Opus 5
parent
a889cb5c00
commit
813f9e16cd
@@ -48,11 +48,11 @@
|
||||
|
||||
- `tests/unit/ci-artifact-contract.test.ts` 16개 테스트가 `bwrap: loopback: Failed RTM_NEWADDR: Operation not permitted` 샌드박스 제약으로 실패한다. 브랜치 분기점 `9e5fbd1`에서도 동일하게 재현되는 환경 문제이며 이번 작업과 무관하다.
|
||||
- `test:coverage`는 위 환경 실패 때문에 vitest가 non-zero로 종료해 `check-risk-coverage.ts`까지 도달하지 못한다(vitest 기본값 `coverage.reportOnFailure: false`). 그 파일만 제외한 진단 실행에서는 `src/application/policies/compatibility.ts`(re-export전용, 계측 가능한 statement 0개)와 `reference-http-gateway.ts`(statements 86.95%/branches 85%, 임계값 90%) 2건이 걸리는데, 둘 다 병합 지점(`9e5fbd1`) 이후 이 브랜치가 건드리지 않은 파일이다.
|
||||
- `test:visual`(chromium)에서 130개 중 18개가 실패하며, 원인은 두 가지로 서로 무관하다.
|
||||
- **원인 A — 이 브랜치 이전부터 존재(사전 존재)**: Public 테스트 2개(동일 라우트, 서로 다른 describe 블록)와 Studio preview/snapshot 계열 5개, 총 7개는 스냅샷 높이가 **줄었다**(예: 1440×6506 → 1440×6412, 1440×1904 → 1440×1811) — 커진 게 아니다. 원인은 `main`에 이미 있고 이 브랜치의 merge-base `9e5fbd1`의 조상인 `79e9aa8`("fix: align TechLog article content widths")로, `.evidence-figure`의 CSS 폭을 `min(61rem, calc(100% + 15rem))`(≈912px)에서 `min(var(--body-copy), 100%)`(672px, 비율 73.6%)로 바꿨다. `case-body-renderer.tsx`/`evidence-figure.tsx`는 `9e5fbd1..HEAD` diff가 비어 있고, golden PNG는 `79e9aa8`보다 앞선 `3a7c5de`에서 마지막으로 기록됐다 — 이 브랜치가 만든 회귀가 아니라 merge-base 시점에 이미 깨져 있던 기준선이다. 갱신은 이 브랜치가 아니라 `main`의 `79e9aa8`에 대해 기록해야 한다.
|
||||
- **원인 B — 이 브랜치의 승인된 작업**: Studio 편집 화면 6개는 실제로 스냅샷보다 **커졌다**(예: 1440×2706 → 1440×2999, 360×1000 → 360×1130) — Task 10이 Case 편집기에 추가한 Asset Picker·업로드 다이얼로그가 원인이다.
|
||||
- 나머지 5개는 크기 변화 없이 픽셀 단위 차이만 있다(추가 조사하지 않음).
|
||||
- 골든 스냅샷 갱신 여부는 두 원인 각각 별도의 리뷰 판단이 필요해 이번 Task에서 임의로 갱신하지 않았다.
|
||||
- `test:visual`(chromium): **최종 fix wave에서 130개 중 13개 실패로 정리했고, 남은 13개는 전부 merge-base `9e5fbd1`에서 동일하게 실패한다** — 이 브랜치가 만든 시각 회귀는 0건이다.
|
||||
- **판정 근거는 추론이 아니라 실측이다.** merge-base `9e5fbd1`을 detached checkout해 `test:visual`을 그대로 실행했고(13 failed / 117 passed), 두 실행이 남긴 `-actual.png`를 SHA-256으로 대조했다. 13개는 merge-base와 HEAD의 실제 렌더 결과가 **바이트 동일**했다 — 즉 이 브랜치의 코드와 무관하다.
|
||||
- **물려받은 13개(갱신하지 않음)**: `TECH_LOG_CASE` 1440, `known Public fixture /cases/collection-fetch-join-pagination` 1440, `TECH_LOG_STUDIO_DOCUMENT_PREVIEW` 1440, `TECH_LOG_STUDIO_PUBLICATION_PREVIEW` 1440, `Studio current-preview` 1440, `Studio publication-snapshot` 1440, `Studio immediate preview` 1440(이상 7개는 높이가 **줄었다**), `TECH_LOG_STUDIO_DOCUMENTS` 360/1440, `Studio document-list` 1440, `TECH_LOG_STUDIO_DOCUMENT_NEW` 1440, `Studio new-document` 1440(이상 5개는 크기 변화 없는 픽셀 차이), `TECH_LOG_STUDIO_DOCUMENT_NEW` 360(360×1000 → 360×1130). 높이가 줄어든 7개의 원인은 `main`에 이미 있고 merge-base `9e5fbd1`의 조상인 `79e9aa8`("fix: align TechLog article content widths")로, `.evidence-figure`의 CSS 폭을 `min(61rem, calc(100% + 15rem))`(≈912px)에서 `min(var(--body-copy), 100%)`(672px, 비율 73.6%)로 바꿨다. golden PNG는 `79e9aa8`보다 앞선 `3a7c5de`에서 마지막으로 기록됐다. 갱신은 이 브랜치가 아니라 `main`의 `79e9aa8`에 대해 기록해야 한다.
|
||||
- **이전 기록의 정정 2건**: (1) "나머지 5개는 크기 변화 없는 픽셀 차이(추가 조사하지 않음)"로 남겨뒀던 항목은 조사 결과 **전부 물려받은 것**이다. (2) `TECH_LOG_STUDIO_DOCUMENT_NEW` 360(360×1000 → 360×1130)을 Asset Picker 때문이라고 원인 B로 분류했었는데, merge-base에서 **동일한 픽셀 수(28,413)로 동일하게 실패**한다 — 물려받은 것이다. 따라서 이 브랜치가 만든 실패는 6개가 아니라 5개다.
|
||||
- **이 브랜치가 만들어 갱신한 5개**: `TECH_LOG_STUDIO_DOCUMENT_EDIT` 360(360×3313 → 360×3627), `TECH_LOG_STUDIO_DOCUMENT_EDIT` 1440·`Studio case-editor` 1440·`Studio conflict-editor` 1440·`Studio dirty-leave dialog` 1440(전부 1440×2706 → 1440×2999). 전부 Case 편집기 화면이고, 늘어난 293px 영역은 Task 10이 추가한 "EVIDENCE / 본문에 Asset 삽입" 패널(업로드 종류 select + `Asset 업로드` 버튼 + Picker 빈 상태)임을 갱신본에서 직접 확인했다. `--update-snapshots`는 이 5개 테스트에만 `--grep`으로 한정해 실행했다 — 일괄 갱신은 물려받은 13개까지 조용히 흡수해 이 구분을 없애기 때문이다.
|
||||
- `test:e2e`/`test:a11y`는 chromium에서 전부 통과하고, firefox/webkit 실패는 이 환경의 브라우저 의존성 문제다(firefox: Pretendard 폰트의 "name records not sorted" 경고를 strict 콘솔 검사가 실패로 잡음; webkit: 호스트에 필요한 시스템 라이브러리 없음 — `playwright install-deps` 필요).
|
||||
|
||||
전체 명령·원문 출력은 `.superpowers/sdd/2026-08-17-techlog-backend-alignment/task-12-report.md`에 기록했다.
|
||||
|
||||
Reference in New Issue
Block a user