0005 가 도메인 스키마를 걷어내고 표 이름을 옮겼는데, 문자열로 표 이름을 들고 있던 자리들이
따라오지 않았다. import 도 타입검사도 pyflakes 도 못 잡는 종류라 전부 **실행되는 순간에만**
터졌고, 그동안 pytest 는 569건이 통째로 죽어 있어 아무것도 못 잡고 있었다.
**init.sql 이 새 DB 를 옛 구조로 세우고 있었다**
64ce467 이 이 파일에 94줄을 더하기만 하고 삭제를 0줄 했다. 그래서 이 파일 한 벌로 세운 DB 는
`place.place_links`·`job.jobs` 를 갖고 ORM 은 `public.place_channels`·`public.jobs` 를 찾는다 —
기동은 정상이고 첫 쿼리에서 죽는다. "init.sql 은 새 DB 를 세우는 전체 DDL 이고 계속 최신을
유지한다"(migrations/README.md)는 계약이 깨져 있었다.
- public 한 벌 · 표 14개로 다시 썼다. 옛 스키마가 있는 DB 에서 다시 돌면 RAISE EXCEPTION 으로
멈춘다 — 그대로 두면 public 에 빈 표가 생기고 0005 가 "relation already exists" 로 실패해
데이터가 옛 스키마에 갇힌다
- 말미에 **마이그레이션 기준선**을 심는다. 없으면 새 DB 에서 migrate.py 가 0001 부터 다시 돌다가
`schema "local" does not exist` 로 죽는다
**운영 버그 둘** — 두 DB(새로 세운 것 · 마이그레이션으로 따라온 것)를 pg_dump 로 찍어 비교해 찾았다
- `upsert_weather` 의 ON CONFLICT 술어에 `kind IS NULL` 이 빠져 **날씨 캐시 저장이 계속 실패**하고
있었다(0007 이 인덱스에 그 조건을 더했다). 캐시라 화면이 안 죽고 로그에만 남았다.
포스트그레스는 술어가 인덱스 술어를 함의하는지 보고 아니면 "no unique or exclusion constraint
matching" 으로 거절한다 — 컬럼도 표도 멀쩡해서 눈으로는 원인이 안 보인다
- ORM 의 `area_contents` 인덱스 정의가 0004·0007·0008 을 하나도 안 따라왔다. 테스트 DB 는 이
모델로 세워지므로 **테스트가 운영과 다른 제약 아래에서 돌고 있었다**
**0009** — 두 DB 비교에서 나온 어긋남 셋(데이터는 안 건드린다)
- `idx_site_contents_site` 가 기존 DB 에만 없었다(0003 이 유니크만 걸었다) — 섹션 조회가 시퀀셜 스캔
- `places.external_place_id` VARCHAR(32) → (64). ORM 은 64 다 — 긴 id 가 잘리면 동일 업소 판정이 틀린다
- RENAME 이 안 따라간 PK 제약 이름 9개(`facts_pkey` → `place_facts_pkey` …)
**테스트를 살린다**
- conftest 의 TRUNCATE 가 표 이름을 **손으로 나열**하고 있었다. 0005 가 이름을 옮기자 전 테스트가
`relation "place_aliases" does not exist` 로 죽었다 — 이제 ORM 메타데이터에서 뽑아 다시 어긋날 수 없다
- `test_schema_ddl` 이 모델 표를 `"None.users"` 로 조회해 **한 표도 비교하지 않고 통과**하고 있었다.
init.sql 이 조용히 어긋난 동안 이 테스트는 초록이었다. 비교한 표 수를 세는 단언을 더한다
- 테스트 SQL 15곳의 옛 표 이름, `_run_worker` 1틱 문제(수집 뒤 따라오는 LOCAL_SYNC 를 집어 가
정작 기다리던 잡이 PENDING 으로 남았다), 지역 캐시 픽스처(읽는 코드가 옳게 거르는데 테스트가 빨개졌다)
**문서**
- `docs/DATA_MODEL.md` 신설 — 표 14개가 무엇을 담고 누가 쓰는지, 값 하나가 DB 에서 페이지까지
가는 길, 두 번 도는 게이트, **DB 에 없는 것**
- `SERVERS.md` DB 절을 마이그레이션 체계로. 배포에 `migrate.py` 를 넣는다 — 코드만 갈면 컨테이너는
정상으로 뜨고 가게 등록·수집·발행만 죽는다
- ARCHITECTURE 2절의 프리렌더 컨테이너가 `solution-frontend` 로 적혀 있었다. 굽는 건
`solution-prerender` 고 전자는 운영에서 뜨지도 않는다 — AGENTS.md 가 함정으로 적어 둔 그 혼동을
문서가 만들고 있었다
- 옛 표 이름 잔재(`place_links`·`local_contents`·`job.jobs`·`company.users`·`fact.facts`·`ai_check_results`)
검증: 빈 컨테이너에 init.sql 로 세운 DB ↔ 마이그레이션으로 따라온 DB 를 `pg_dump --schema-only`
로 비교 — 표·인덱스·제약·컬럼 전부 동일. pytest 583건 중 581 통과(남은 2건은 `.env` 누수·
레이트리밋 카운터로 환경 문제다). 구글 로그인 21건 포함.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
164 lines
8.1 KiB
Python
164 lines
8.1 KiB
Python
"""fact 테이블 제약 — '사이트에 나가는 값은 (사업장, 단위, key) 당 1건' 이 DB 레벨에서 지켜지는지.
|
|
|
|
유니크는 **노출 상태(VERIFIED·CORRECTED)에만** 걸린다.
|
|
- 걸어야 하는 이유: 안 걸면 체크인 시간이 15시/16시 두 값으로 동시에 노출된다.
|
|
- 활성 전체에 걸면 안 되는 이유: 재수집이 올 때마다 확인된 노출값을 밀어내야 하고,
|
|
그 순간 사이트에서 사실이 사라진다. 후보(UNVERIFIED·PENDING_OWNER)는 공존해야 한다.
|
|
"""
|
|
import uuid
|
|
|
|
import pytest
|
|
from sqlalchemy import text
|
|
from sqlalchemy.exc import IntegrityError
|
|
|
|
from common.enums import (
|
|
FACT_STATUS_TRANSITIONS,
|
|
LOCKED_FACT_STATUSES,
|
|
PUBLISHABLE_FACT_STATUSES,
|
|
FactStatus,
|
|
PlaceCategory,
|
|
PlaceStatus,
|
|
SourceType,
|
|
)
|
|
|
|
|
|
async def _seed_place(db_engine, owner_id) -> str:
|
|
"""검증까지 끝난 사업장 1개를 시드하고 place_id 를 돌려준다."""
|
|
pid = uuid.uuid4()
|
|
async with db_engine.begin() as conn:
|
|
await conn.execute(
|
|
text(
|
|
"INSERT INTO places (place_id, owner_user_id, name, category, status, external_place_id, verified_at) "
|
|
"VALUES (:pid, :cid, :name, :cat, :status, :kakao, now())"
|
|
),
|
|
{
|
|
"pid": pid, "cid": uuid.UUID(owner_id), "name": "테스트펜션",
|
|
"cat": PlaceCategory.LODGING.value, "status": PlaceStatus.DRAFT.value,
|
|
"kakao": "12345678",
|
|
},
|
|
)
|
|
return str(pid)
|
|
|
|
|
|
async def _insert_fact(db_engine, place_id, key, value, status, unit_id=None):
|
|
async with db_engine.begin() as conn:
|
|
await conn.execute(
|
|
text(
|
|
"INSERT INTO place_facts (fact_id, place_id, unit_id, key, value, source_type, status, collected_at) "
|
|
"VALUES (:fid, :pid, :uid, :key, :val, :src, :status, now())"
|
|
),
|
|
{
|
|
"fid": uuid.uuid4(), "pid": uuid.UUID(place_id), "uid": unit_id,
|
|
"key": key, "val": value, "src": SourceType.CRAWL.value, "status": status.value,
|
|
},
|
|
)
|
|
|
|
|
|
async def test_published_fact_is_unique_per_place_and_key(db_engine, owner_id):
|
|
"""검증: 같은 사업장·같은 key 로 노출 상태 fact 를 두 번 넣는다.
|
|
기대결과: 두 번째 INSERT 가 유니크 인덱스에 막힌다(체크인 시간이 두 값으로 갈라지지 않는다)."""
|
|
place_id = await _seed_place(db_engine, owner_id)
|
|
await _insert_fact(db_engine, place_id, "check_in_time", "15:00", FactStatus.VERIFIED)
|
|
|
|
with pytest.raises(IntegrityError):
|
|
await _insert_fact(db_engine, place_id, "check_in_time", "16:00", FactStatus.CORRECTED)
|
|
|
|
|
|
async def test_candidates_coexist_with_published_value(db_engine, owner_id):
|
|
"""검증: 노출값이 있는 상태에서 재수집 후보를 여러 건 넣는다.
|
|
기대결과: 전부 공존한다 — ★ 재수집이 노출 중인 사실을 밀어내지 않는다."""
|
|
place_id = await _seed_place(db_engine, owner_id)
|
|
await _insert_fact(db_engine, place_id, "check_in_time", "15:00", FactStatus.VERIFIED)
|
|
await _insert_fact(db_engine, place_id, "check_in_time", "16:00", FactStatus.PENDING_OWNER)
|
|
await _insert_fact(db_engine, place_id, "check_in_time", "14:00", FactStatus.UNVERIFIED)
|
|
|
|
async with db_engine.begin() as conn:
|
|
rows = (await conn.execute(
|
|
text("SELECT status FROM place_facts WHERE place_id = :pid AND key = 'check_in_time'"),
|
|
{"pid": uuid.UUID(place_id)},
|
|
)).all()
|
|
assert len(rows) == 3, "노출값 1건 + 후보 2건이 공존해야 한다"
|
|
|
|
|
|
async def test_rejected_fact_frees_the_key(db_engine, owner_id):
|
|
"""검증: 기존 값을 REJECTED 로 내린 뒤 같은 key 를 새로 노출한다.
|
|
기대결과: 통과 — 틀린 값은 이력으로 남고, 새 값이 노출 자리를 차지한다."""
|
|
place_id = await _seed_place(db_engine, owner_id)
|
|
await _insert_fact(db_engine, place_id, "check_in_time", "15:00", FactStatus.REJECTED)
|
|
await _insert_fact(db_engine, place_id, "check_in_time", "16:00", FactStatus.VERIFIED)
|
|
|
|
async with db_engine.begin() as conn:
|
|
rows = (await conn.execute(
|
|
text("SELECT value, status FROM place_facts WHERE place_id = :pid ORDER BY status"),
|
|
{"pid": uuid.UUID(place_id)},
|
|
)).all()
|
|
assert len(rows) == 2, "REJECTED 이력과 새 값이 함께 남아야 한다"
|
|
|
|
|
|
async def test_expired_fact_frees_the_key(db_engine, owner_id):
|
|
"""검증: 유효기간이 지나 EXPIRED 로 내린 값과 새 수집값의 공존.
|
|
기대결과: 통과 — EXPIRED 도 유니크에서 빠진다."""
|
|
place_id = await _seed_place(db_engine, owner_id)
|
|
await _insert_fact(db_engine, place_id, "cancel_policy", "구 규정", FactStatus.EXPIRED)
|
|
await _insert_fact(db_engine, place_id, "cancel_policy", "새 규정", FactStatus.VERIFIED)
|
|
|
|
|
|
async def test_same_key_allowed_across_units(db_engine, owner_id):
|
|
"""검증: 객실이 다르면 같은 key 를 각각 가질 수 있는지.
|
|
기대결과: 통과 — A동·B동이 각자의 기준 인원을 갖는다."""
|
|
place_id = await _seed_place(db_engine, owner_id)
|
|
unit_a, unit_b = uuid.uuid4(), uuid.uuid4()
|
|
async with db_engine.begin() as conn:
|
|
for uid, name in ((unit_a, "A동"), (unit_b, "B동")):
|
|
await conn.execute(
|
|
text("INSERT INTO place_units (unit_id, place_id, name) VALUES (:uid, :pid, :name)"),
|
|
{"uid": uid, "pid": uuid.UUID(place_id), "name": name},
|
|
)
|
|
|
|
await _insert_fact(db_engine, place_id, "standard_capacity", "4", FactStatus.VERIFIED, unit_id=unit_a)
|
|
await _insert_fact(db_engine, place_id, "standard_capacity", "2", FactStatus.VERIFIED, unit_id=unit_b)
|
|
|
|
with pytest.raises(IntegrityError): # 같은 객실 안에서 노출값은 여전히 1건
|
|
await _insert_fact(db_engine, place_id, "standard_capacity", "6", FactStatus.CORRECTED, unit_id=unit_a)
|
|
|
|
|
|
async def test_unit_fact_and_place_fact_are_separate(db_engine, owner_id):
|
|
"""검증: 같은 key 를 사업장 단위와 객실 단위로 동시에 갖는 경우.
|
|
기대결과: 통과 — 부분 인덱스가 unit_id NULL 여부로 갈라져 있다."""
|
|
place_id = await _seed_place(db_engine, owner_id)
|
|
unit_id = uuid.uuid4()
|
|
async with db_engine.begin() as conn:
|
|
await conn.execute(
|
|
text("INSERT INTO place_units (unit_id, place_id, name) VALUES (:uid, :pid, :name)"),
|
|
{"uid": unit_id, "pid": uuid.UUID(place_id), "name": "A동"},
|
|
)
|
|
await _insert_fact(db_engine, place_id, "has_kitchen", "false", FactStatus.VERIFIED)
|
|
await _insert_fact(db_engine, place_id, "has_kitchen", "true", FactStatus.VERIFIED, unit_id=unit_id)
|
|
|
|
|
|
def test_only_verified_and_corrected_are_publishable():
|
|
"""검증: 노출 가능 상태 집합(절대규칙 1).
|
|
기대결과: VERIFIED·CORRECTED 뿐. 미검증·반려·만료는 절대 사이트에 나가지 않는다."""
|
|
assert PUBLISHABLE_FACT_STATUSES == {FactStatus.VERIFIED, FactStatus.CORRECTED}
|
|
for status in (FactStatus.UNVERIFIED, FactStatus.PENDING_OWNER, FactStatus.REJECTED, FactStatus.EXPIRED):
|
|
assert status not in PUBLISHABLE_FACT_STATUSES
|
|
|
|
|
|
def test_corrected_is_locked_against_auto_update():
|
|
"""검증: 사람이 고친 값이 잠기는지(절대규칙 6).
|
|
기대결과: CORRECTED 는 잠금 상태이고, 전이표에서 자동 갱신 경로(UNVERIFIED 등)로 못 돌아간다."""
|
|
assert FactStatus.CORRECTED in LOCKED_FACT_STATUSES
|
|
allowed = FACT_STATUS_TRANSITIONS[FactStatus.CORRECTED]
|
|
assert FactStatus.UNVERIFIED not in allowed, "자동 수집이 사장님 수정본을 덮어쓸 수 있으면 안 된다"
|
|
assert FactStatus.VERIFIED not in allowed
|
|
|
|
|
|
def test_transition_table_covers_every_status():
|
|
"""검증: 전이표가 모든 상태를 다루는지.
|
|
기대결과: 6개 상태 전부 키로 존재하고, 목적지도 전부 유효한 FactStatus."""
|
|
assert set(FACT_STATUS_TRANSITIONS) == set(FactStatus)
|
|
for src, dests in FACT_STATUS_TRANSITIONS.items():
|
|
assert dests, f"{src.name}: 나갈 수 있는 상태가 없다"
|
|
for dest in dests:
|
|
assert isinstance(dest, FactStatus)
|