Mina Choi
9f16c3224b
목록 줄이 아이콘·상호·배지·주소 넷뿐이었다. 서버는 이미 road_address·created_at· published_at 을 주는데 화면이 안 썼다. 실측(계정 test): 35줄 중 34줄이 발행 전이고 같은 상호 '버터브루' 가 4줄이라 어느 게 어느 건지 가릴 단서가 화면에 없었다. 리서치 — Wix My Sites 는 이름·URL·Premium·협업자만 두고 검색·그리드/리스트 전환·폴더가 있다. Sites API 문서가 권하는 조합은 displayName·thumbnail·viewUrl·editUrl 이다. 아임웹 내사이트는 **실제 화면을 열어 봤다**(imweb.me 가이드): 줄 왼쪽에 큰 가로형 썸네일, 상호 아래 도메인, 그리고 도메인/SSL·PG 신청처럼 **안 끝난 설정**을 줄 안에 배지로 늘어놓는다. 공통 원칙은 목록이 ① 구분 ② 상태 ③ 여는 길 셋만 한다는 것 — 통계는 사이트 안 대시보드다. - protocol·site_service: `MySiteData.thumbnail_url` 추가. 목록이 사이트 행을 이미 조인해 읽고 있어서 쿼리는 그대로다 - site_thumbnail: 공개 주소에 `?v=<발행 버전>`. 블롭 이름은 고정이고 내용만 덮어쓰므로 주소가 안 변하면 사진을 바꿔 재발행해도 **캐시에 남은 지난 그림**이 계속 보인다 (CACHE_CONTROL 60초로는 그 60초를 못 막는다). 이름에 버전을 넣지 않은 이유는 사이트당 블롭이 발행 횟수만큼 쌓이는데 지우는 코드가 없어서다 - site_thumbnail: 썸네일 전용 저장소 스위치(`THUMBNAIL_BLOB_*`). 예전엔 키 하나가 사이트 전체 업로드(azure_static)까지 같이 켰다 — 둘은 필요한 저장소가 다르다 (사이트는 정적 호스팅 `$web`, 썸네일은 이미지 버킷이면 된다) - SitesPage: 줄 → **카드 그리드**. 썸네일은 16:10(브라우저 창 비율 — 사이트 미리보기를 1:1 로 자르면 무슨 사이트인지 못 알아본다). 검색(상호·주소, 공백 무시) + 상태 칸 `전체/발행됨/발행 전` 에 건수. 판정은 `bucketOf` 하나가 소유한다(배지·필터·정렬이 갈라지면 건수가 어긋나 목록을 못 믿게 된다). 검색 0건 화면을 처음 온 사람의 빈 화면과 분리했다 — 35개 있는데 "아직 없습니다" 라고 말하던 자리다 - 카드 골격은 **상태와 무관하게 같다**. 발행 전 카드에만 줄이 하나 더 붙어 높이와 버튼 위치가 어긋났다(사장님 지적). 버튼 문구도 '편집' 하나로 — 하는 일이 같은데 글자만 달랐다 아직 그림이 없는 사이트가 대부분이다. 썸네일은 발행에 성공해야 생긴다. 검증: 백엔드 전체 통과. 목록 줄이 주소·생성일·썸네일을 들고 오는지, 발행 전 줄에 `thumbnail_url` 키가 아예 없는지, 재발행하면 `?v=1` → `?v=2` 로 주소가 바뀌는지 4건 추가. 프론트 tsc+eslint 통과. 실제 발행으로 블롭 업로드(232KB) → 공개 주소 200 → 목록 반영 확인. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QLWEFx4X3XRmKewUKjJWow
184 lines
8.7 KiB
Python
184 lines
8.7 KiB
Python
"""발행 썸네일 — 사이트의 대표 사진을 Azure Blob 으로 옮긴다.
|
|
|
|
★ 스크린샷이 아니다(헤드리스 브라우저는 영구 금지, docs/DECISIONS.md 1-1).
|
|
그래서 여기서 볼 것은 "무엇을 대표로 고르는가"와 "무엇을 받지 않는가" 두 가지다.
|
|
|
|
이 경로가 절대 하면 안 되는 것:
|
|
- 썸네일 실패로 발행을 되돌리는 것 — 정적 파일은 이미 올라갔다.
|
|
- 이미지가 아닌 응답을 그대로 올리는 것 — 카드가 깨진 그림이 된다.
|
|
- 사이트 디렉터리(`s/<slug>/`) 안에 두는 것 — 다음 발행이 통째로 지운다.
|
|
"""
|
|
from types import SimpleNamespace
|
|
|
|
import httpx
|
|
import pytest
|
|
|
|
from services import azure_static, site_payload, site_thumbnail
|
|
|
|
|
|
def _snapshot(*rows):
|
|
return {"media": list(rows)}
|
|
|
|
|
|
def _transport(monkeypatch, handler):
|
|
"""사진을 내려주는 CDN 대역. 테스트가 실제 네트워크를 부르지 않게 한다."""
|
|
real = httpx.AsyncClient
|
|
|
|
def factory(**kwargs):
|
|
return real(transport=httpx.MockTransport(handler), **kwargs)
|
|
|
|
monkeypatch.setattr(site_thumbnail.httpx, "AsyncClient", factory)
|
|
|
|
|
|
class FakeContainer:
|
|
def __init__(self):
|
|
self.uploads = {}
|
|
|
|
def upload_blob(self, name, data, **options):
|
|
self.uploads[name] = {"body": data, **options}
|
|
|
|
|
|
@pytest.fixture
|
|
def blob(monkeypatch):
|
|
container = FakeContainer()
|
|
service = SimpleNamespace(get_container_client=lambda name: container)
|
|
monkeypatch.setattr(
|
|
site_thumbnail.BlobServiceClient, "from_connection_string", lambda value: service
|
|
)
|
|
monkeypatch.setenv("AZURE_STORAGE_CONNECTION_STRING", "UseDevelopmentStorage=true")
|
|
monkeypatch.delenv("AZURE_STORAGE_PREFIX", raising=False)
|
|
return container
|
|
|
|
|
|
def test_대표사진은_객실전용이_아닌_첫장이다():
|
|
# og:image 선정(site_payload)과 같은 함수를 쓴다 — 두 벌이면 검색 결과와 카드가 갈린다.
|
|
snapshot = _snapshot(
|
|
{"media_id": "m1", "url": "https://cdn.test/room.jpg", "unit_id": "u1"},
|
|
{"media_id": "m2", "url": "https://cdn.test/front.jpg", "unit_id": None},
|
|
{"media_id": "m3", "url": "https://cdn.test/back.jpg", "unit_id": None},
|
|
)
|
|
assert site_payload.primary_media(snapshot)["media_id"] == "m2"
|
|
|
|
|
|
def test_설정이_없으면_아무것도_하지_않는다(monkeypatch):
|
|
monkeypatch.delenv("AZURE_STORAGE_CONNECTION_STRING", raising=False)
|
|
assert site_thumbnail.is_configured() is False
|
|
|
|
|
|
async def test_대표사진이_없으면_None_이고_발행은_그대로(blob):
|
|
"""검증: 승인 사진이 객실 전용뿐이라 대표가 없다.
|
|
기대결과: None 을 돌려주고 업로드하지 않는다(예외를 올리지 않는다)."""
|
|
snapshot = _snapshot({"media_id": "m1", "url": "https://cdn.test/room.jpg", "unit_id": "u1"})
|
|
assert await site_thumbnail.store("butter", snapshot) is None
|
|
assert blob.uploads == {}
|
|
|
|
|
|
async def test_이미지가_아니면_받지_않는다(blob, monkeypatch):
|
|
"""검증: 사진 URL 이 HTML(오류 페이지)을 돌려준다.
|
|
기대결과: 거부하고 None. 깨진 그림을 쇼케이스에 올리지 않는다."""
|
|
_transport(monkeypatch, lambda req: httpx.Response(200, headers={"content-type": "text/html"}, content=b"<html>"))
|
|
snapshot = _snapshot({"media_id": "m1", "url": "https://cdn.test/front.jpg", "unit_id": None})
|
|
|
|
assert await site_thumbnail.store("butter", snapshot) is None
|
|
assert blob.uploads == {}
|
|
|
|
|
|
async def test_상한을_넘는_사진은_받지_않는다(blob, monkeypatch):
|
|
body = b"x" * (site_thumbnail.MAX_BYTES + 1)
|
|
_transport(monkeypatch, lambda req: httpx.Response(200, headers={"content-type": "image/jpeg"}, content=body))
|
|
snapshot = _snapshot({"media_id": "m1", "url": "https://cdn.test/huge.jpg", "unit_id": None})
|
|
|
|
assert await site_thumbnail.store("butter", snapshot) is None
|
|
assert blob.uploads == {}
|
|
|
|
|
|
async def test_네트워크_실패는_삼킨다(blob, monkeypatch):
|
|
def _boom(request):
|
|
raise httpx.ConnectTimeout("timed out", request=request)
|
|
|
|
_transport(monkeypatch, _boom)
|
|
snapshot = _snapshot({"media_id": "m1", "url": "https://cdn.test/front.jpg", "unit_id": None})
|
|
|
|
assert await site_thumbnail.store("butter", snapshot) is None
|
|
|
|
|
|
async def test_사이트_디렉터리_밖의_thumbs_에_올린다(blob, monkeypatch):
|
|
"""검증: 대표 사진을 받아 업로드한다.
|
|
기대결과: `<prefix>/thumbs/<slug>.<ext>` — `s/<slug>/` 안이 아니다.
|
|
★ 사이트 경로는 매 발행마다 프리렌더 산출물로 통째로 교체된다
|
|
(azure_static._remove_stale_site_files) — 그 안에 두면 다음 발행에서 조용히 사라진다."""
|
|
_transport(monkeypatch, lambda req: httpx.Response(200, headers={"content-type": "image/jpeg"}, content=b"jpegbytes"))
|
|
snapshot = _snapshot({"media_id": "m1", "url": "https://cdn.test/front.jpg", "unit_id": None})
|
|
|
|
url = await site_thumbnail.store("butter", snapshot, 3)
|
|
|
|
# ★ `?v=` 는 캐시 무효화다. 블롭 이름은 그대로 덮어쓰므로 주소가 안 변하면
|
|
# 브라우저·CDN 이 지난 발행의 그림을 계속 보여준다(site_thumbnail.public_url).
|
|
assert url == "https://w4ai.o2o.kr/thumbs/butter.jpg?v=3"
|
|
name = f"{azure_static.DEFAULT_PREFIX}/thumbs/butter.jpg"
|
|
assert set(blob.uploads) == {name}
|
|
assert not name.startswith(f"{azure_static.DEFAULT_PREFIX}/{azure_static.SITE_ROOT_DIR}/")
|
|
|
|
uploaded = blob.uploads[name]
|
|
assert uploaded["body"] == b"jpegbytes"
|
|
settings = uploaded["content_settings"]
|
|
assert settings.content_type == "image/jpeg"
|
|
# 썸네일은 재발행마다 바뀌는데 주소는 그대로다 — 길게 캐시하면 옛 그림이 계속 뜬다.
|
|
assert settings.cache_control == site_thumbnail.CACHE_CONTROL
|
|
|
|
|
|
# ── 발행 경로와의 배선 ────────────────────────────────────────────────────
|
|
# 아래 두 건은 "발행이 썸네일에 매달리지 않는가" 를 본다. 썸네일 만들기 자체는 위에서 봤다.
|
|
async def test_발행하면_사이트에_썸네일_주소가_남는다(auth_headers, client, db_engine, monkeypatch):
|
|
from services import build_service
|
|
from tests.test_build_publish import _approved_media, _place, _run, _verified_facts
|
|
|
|
async def _store(slug, snapshot, version=None):
|
|
return f"https://w4ai.o2o.kr/thumbs/{slug}.jpg?v={version}"
|
|
|
|
monkeypatch.setattr(build_service.site_thumbnail, "store", _store)
|
|
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "썸네일펜션")
|
|
await _verified_facts(client, h, pid, {"intro": "하조대 해변 도보 3분 거리의 펜션입니다."})
|
|
await _approved_media(db_engine, pid)
|
|
|
|
await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})
|
|
await _run()
|
|
|
|
site = (await client.get(f"/v1/place/{pid}/site", headers=h)).json()["site"]
|
|
assert site["thumbnail_url"].startswith("https://w4ai.o2o.kr/thumbs/")
|
|
assert site["thumbnail_url"].endswith("?v=1")
|
|
|
|
# ★ 재발행하면 주소가 바뀌어야 한다. 블롭 이름은 그대로 덮어쓰므로, 주소가 그대로면
|
|
# 사장님은 사진을 바꾸고 다시 발행해도 캐시에 남은 **지난 그림**을 계속 본다.
|
|
await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})
|
|
await _run()
|
|
again = (await client.get(f"/v1/place/{pid}/site", headers=h)).json()["site"]
|
|
assert again["thumbnail_url"].endswith("?v=2")
|
|
assert again["thumbnail_url"] != site["thumbnail_url"]
|
|
|
|
|
|
async def test_썸네일을_못_만들어도_발행은_성공한다(auth_headers, client, db_engine, monkeypatch):
|
|
"""★ 썸네일은 발행의 부수 효과다 — 정적 파일은 이미 올라갔다. 여기서 되돌리면
|
|
사장님 사이트가 그림 하나 때문에 안 나간다."""
|
|
from services import build_service
|
|
from tests.test_build_publish import _approved_media, _place, _run, _verified_facts
|
|
|
|
async def _store(slug, snapshot, version=None):
|
|
return None
|
|
|
|
monkeypatch.setattr(build_service.site_thumbnail, "store", _store)
|
|
|
|
h = await auth_headers("u1")
|
|
pid = await _place(client, h, "그림없는펜션")
|
|
await _verified_facts(client, h, pid, {"intro": "하조대 해변 도보 3분 거리의 펜션입니다."})
|
|
await _approved_media(db_engine, pid)
|
|
|
|
job_id = (await client.post(f"/v1/place/{pid}/site/build", headers=h, json={"publish": True})).json()["job_id"]
|
|
await _run()
|
|
|
|
assert (await client.get(f"/v1/job/{job_id}", headers=h)).json()["job"]["result"]["published"] is True
|
|
site = (await client.get(f"/v1/place/{pid}/site", headers=h)).json()["site"]
|
|
assert "thumbnail_url" not in site
|