o2o-site-AEO/solution/backend/tests/test_verify_candidates.py
Mina Choi 9d25ed613e 구조: 사장님(solution)과 내부 운영(admin)을 두 앱으로 가른다
최상단을 프로젝트 단위로 평평하게 둔다 — o2o-negosium 과 같은 규약이고, 이 레포만
다르게 갈 이유가 없다. negodata/{backend,front} 가 프로젝트 안에서 f/b 를 가르는 선례,
lps-admin/ 이 백엔드 없이 프론트만 가진 최상단 폴더의 선례다.

  backend/ frontend/{admin,site,shared}  →  solution/{backend,front,site,shared} + admin/

## 왜

내부 라우트(/local-content, /places/:id/seo)의 이름과 화면 코드가 사장님 번들에
그대로 실려 나가고 있었다. UserRole.DEVELOPER 주석의 "고객사에 존재를 노출하지 않는다"를
번들이 깨고 있었다 — 라우트 가드는 화면을 가리지 번들은 못 가린다.
번들을 갈라 확인했다: 사장님 dist 에서 local-content · /places · SeoAudit 이 전부 0건이다.

그 과정에서 두 곳이 더 새고 있었다.
- AppShell 의 NAV 배열이 내부 메뉴를 하드코딩하고 있었다. 앱을 가른 뒤에도 dist 에
  local-content 가 남아서 찾았다. 메뉴는 이제 앱이 prop 으로 들고 온다.
- EditorHeader·BuilderPage·LoginPage 가 /places 로 링크하고 있었다. 그 화면이 admin 으로
  나갔으니 사장님 앱에서는 404 다. 링크를 걷어내고 LoginPage 기본 도착지는 '/' 로 바꿨다
  (앱마다 홈이 다르고 각 라우터의 '/' 가 이미 그걸 안다).

## admin 에 백엔드를 두지 않았다

내부 화면이 부르는 훅이 전부 router/v1/{place,fact,local,validator} 에 이미 있다.
자체 백엔드를 두면 place·fact·link 를 같은 DB 에 대고 두 번 구현하게 된다.
대가는 solution/backend 가 죽으면 admin 도 멈추는 것 — 내부 도구라 감수한다.

## admin 의 `@` 는 solution/front/src 를 가리킨다

내부 화면이 쓰는 API 클라이언트·UI·수집 배선이 solution 에 한 벌만 있고 그 파일들끼리도
`@/...` 로 서로를 부른다. admin 에서 `@` 를 자기 src 로 잡으면 그 참조가 전부 깨진다
(실측 TS2307 14건). 복제하는 길도 있지만 RecollectPanel 주석이 금지한다 —
"수집 경로를 두 벌 만들면 확정 게이트"가 갈라진다.
admin 자기 파일만 `@admin` 이고, 의존 방향은 admin → solution 한 쪽뿐이다.

admin 이 여는 빌더는 다른 오리진이라 절대 URL + 새 탭이다(admin/src/lib/solutionUrl.ts).
react-router Link 로 두면 admin 안에서 라우트를 찾다 404 다.

## 그 밖

- npm 워크스페이스 루트를 레포 루트로 올렸다(admin 이 solution 밖이라).
- docker-compose 를 255→174줄로 줄이고 admin(:3002) 서비스를 넣었다. ADMIN_BIND 기본값은
  127.0.0.1 — 0.0.0.0 으로 열면 앱을 가른 의미가 없다.
- 발행 호스트를 프론트 .env 에 따로 적지 않는다. compose 가 루트의 SITE_PUBLIC_HOST 를
  VITE_PUBLISH_HOST 로 흘려보낸다 — 두 곳에 적으면 canonical 과 화면 주소가 조용히 갈라진다.
- nginx/site.conf 를 git 에서 빼고 .example 만 남겼다(.env·*.toml 과 같은 규약).
  compose 가 bind mount 하므로 클론 직후 복사해야 한다 — 없으면 Docker 가 그 자리에
  디렉토리를 만들어 nginx 가 설정 없이 뜬다.
- config.test.toml.example 을 추가했다. 없으면 클론한 사람이 pytest 를 아예 못 돌린다
  (conftest import 단계에서 죽는다). 외부 API 키는 전부 빈값이다 —
  APP_ENV=test 가 .env 를 안 읽는 이유를 여기서 우회하면 안 된다.
- 경로가 한 칸 깊어져 test_schema_ddl(parents[2]→[3]) 과 test_site_theme 을 고쳤다.

검증: front·admin·site 전부 lint 0 / build 0. 백엔드 514 passed.
남은 4건(test_build_publish 3 · test_snapshot 1)은 이 변경 전부터 실패하던 것으로,
손대지 않은 메인 체크아웃에서 같은 4건이 같게 실패하는 것을 확인했다.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019uYhHQdssRubirPirrdJJC
2026-08-31 15:12:09 +09:00

200 lines
9.8 KiB
Python

"""동일 업소 후보 조회 — ★ 서버가 자동 확정하지 않고 사람이 고르게 한다.
남의 가게를 붙이는 게 이 서비스에서 제일 비싼 실수다(체크인 시간이 틀리면 실제 예약 클레임).
그래서 자동 판정이 MATCHED 여도 후보를 전부 내려보내 UI 가 확인시킬 수 있게 한다.
"""
from common.enums import ErrorType
from services.external import kakao as kakao_client
from services.external import naver as naver_client
def _np(name, addr, url=None):
return naver_client.NaverPlace(
name=name, road_address=addr, address=None, phone=None,
latitude=38.0, longitude=128.7, category_name="숙박>펜션",
place_url=url, naver_place_id=None,
)
def _match(outcome, place=None, candidates=None, reason=None):
return naver_client.MatchResult(
outcome=outcome, place=place, candidates=candidates or [], reason=reason, detail=None,
)
async def _place(client, h, name="핑크비치펜션"):
return (await client.post("/v1/place", headers=h, json={"name": name, "category": 1})).json()["place"]["place_id"]
def _patch_naver(monkeypatch, result):
"""네이버 판정을 고정값으로 바꾼다. 호출 인자를 그대로 돌려줘 호출측 규약도 볼 수 있게 한다."""
monkeypatch.setattr(kakao_client.KakaoLocalClient, "enabled", property(lambda self: False))
monkeypatch.setattr(naver_client.NaverLocalClient, "enabled", property(lambda self: True))
seen: dict = {}
async def _verify(self, name, address_hint=None, *, search_query=None):
seen["name"] = name
seen["search_query"] = search_query
return result
async def _close(self):
return None
monkeypatch.setattr(naver_client.NaverLocalClient, "verify_place", _verify)
monkeypatch.setattr(naver_client.NaverLocalClient, "aclose", _close)
return seen
async def test_single_match_is_auto_selectable(auth_headers, client, monkeypatch):
"""검증: 정확히 1건 일치하는 상호명.
기대결과: auto_selectable=true 로 내려오되 **후보는 그대로 내려온다** — UI 가 한 번 확인시킬 수 있다."""
only = _np("핑크비치펜션", "강원특별자치도 양양군 현북면 하조대3길 11", "http://pinkbeach.kr")
_patch_naver(monkeypatch, _match(kakao_client.MatchOutcome.MATCHED, only, [only], "name_exact"))
h = await auth_headers("u1")
pid = await _place(client, h)
body = (await client.get(f"/v1/place/{pid}/verify/candidates", headers=h)).json()
assert body["result"]["success"] is True
assert body["outcome"] == "matched"
assert body["auto_selectable"] is True
assert len(body["candidates"]) == 1
assert body["candidates"][0]["place_url"] == "http://pinkbeach.kr"
async def test_ambiguous_returns_all_candidates_for_ui(auth_headers, client, monkeypatch):
"""검증: 동명 업소가 여러 건이라 자동 판정이 안 된다.
기대결과: ★ auto_selectable=false + 후보 전체 — UI 가 목록을 그려 사람이 고른다."""
rows = [
_np("하조대펜션", "강원특별자치도 양양군 현북면 하조대3길 11"),
_np("하조대펜션", "강원특별자치도 양양군 현북면 하조대2길 48"),
_np("하조대펜션", "강원특별자치도 양양군 손양면 동명로 5"),
]
_patch_naver(monkeypatch, _match(kakao_client.MatchOutcome.AMBIGUOUS, None, rows, "name_duplicate"))
h = await auth_headers("u1")
pid = await _place(client, h, "하조대펜션")
body = (await client.get(f"/v1/place/{pid}/verify/candidates", headers=h)).json()
assert body["outcome"] == "ambiguous"
assert body["auto_selectable"] is False
assert len(body["candidates"]) == 3
assert body["reason"] == "name_duplicate"
# 사람이 고를 수 있게 주소가 다 실려 있어야 한다
assert len({c["road_address"] for c in body["candidates"]}) == 3
async def test_no_candidate_is_reported(auth_headers, client, monkeypatch):
"""검증: 외부 장소 DB 에 후보가 하나도 없다.
기대결과: PLACE_VERIFY_NO_CANDIDATE — 사장님이 직접 입력하는 경로로 안내해야 한다."""
_patch_naver(monkeypatch, _match(kakao_client.MatchOutcome.NO_CANDIDATE, None, [], "empty"))
h = await auth_headers("u1")
pid = await _place(client, h, "존재하지않는가게")
body = (await client.get(f"/v1/place/{pid}/verify/candidates", headers=h)).json()
assert body["result"]["code"] == ErrorType.PLACE_VERIFY_NO_CANDIDATE.value
async def test_picked_candidate_can_be_confirmed(auth_headers, client, monkeypatch):
"""검증: 후보 목록에서 하나를 골라 확정한다(UI 흐름 전체).
기대결과: 확정되고, 후보가 준 홈페이지가 공식 채널로 자동 등록된다."""
rows = [
_np("하조대펜션", "강원특별자치도 양양군 현북면 하조대3길 11", "http://hajodae-a.kr"),
_np("하조대펜션", "강원특별자치도 양양군 손양면 동명로 5", "http://hajodae-b.kr"),
]
_patch_naver(monkeypatch, _match(kakao_client.MatchOutcome.AMBIGUOUS, None, rows, "name_duplicate"))
h = await auth_headers("u1")
pid = await _place(client, h, "하조대펜션")
cands = (await client.get(f"/v1/place/{pid}/verify/candidates", headers=h)).json()["candidates"]
picked = cands[1] # 사람이 두 번째를 골랐다
r = (await client.post(f"/v1/place/{pid}/verify", headers=h, json={
"source": 2, "road_address": picked["road_address"],
"latitude": picked["latitude"], "longitude": picked["longitude"],
"place_url": picked["place_url"],
})).json()
assert r["result"]["success"] is True
assert r["place"]["road_address"] == "강원특별자치도 양양군 손양면 동명로 5"
links = (await client.get(f"/v1/place/{pid}/link/list", headers=h)).json()["links"]
official = [l for l in links if l["url"] == "http://hajodae-b.kr"]
assert official, "고른 후보의 홈페이지가 공식 채널로 등록되지 않았다"
assert official[0].get("confirmed_at") is None, "등록만 하고 확정은 따로다"
async def test_candidates_require_configured_source(auth_headers, client, monkeypatch):
"""검증: 카카오도 네이버도 키가 없다.
기대결과: LOCAL_NOT_CONFIGURED — 조용히 빈 목록을 주지 않는다."""
monkeypatch.setattr(kakao_client.KakaoLocalClient, "enabled", property(lambda self: False))
monkeypatch.setattr(naver_client.NaverLocalClient, "enabled", property(lambda self: False))
h = await auth_headers("u1")
pid = await _place(client, h)
body = (await client.get(f"/v1/place/{pid}/verify/candidates", headers=h)).json()
assert body["result"]["code"] == ErrorType.LOCAL_NOT_CONFIGURED.value
async def test_candidates_scoped_to_company(auth_headers, client, other_company_id, monkeypatch):
"""검증: 다른 회사 계정으로 남의 사업장 후보를 조회한다.
기대결과: PLACE_NOT_FOUND."""
_patch_naver(monkeypatch, _match(kakao_client.MatchOutcome.MATCHED, _np("a", "b"), [], "x"))
h1 = await auth_headers("o1")
pid = await _place(client, h1)
h2 = await auth_headers("o2", other_company_id)
body = (await client.get(f"/v1/place/{pid}/verify/candidates", headers=h2)).json()
assert body["result"]["code"] == ErrorType.PLACE_NOT_FOUND.value
async def test_region_stays_in_query_and_out_of_the_match_name(auth_headers, client, monkeypatch):
"""검증: 화면이 '상호명 + 위치'를 합쳐 query 로 보낸다.
기대결과: 검색어에는 지역이 그대로 실리고, 판정용 상호명에는 **상호만** 간다.
★ 왜 이걸 못 박아 두나: 판정(pick_match)은 후보 상호명과 정확일치를 본다. 지역이 붙은
문자열을 판정에 그대로 넘기면 정확일치가 구조적으로 성립할 수 없어, 후보가 단 1건인
경우까지 AMBIGUOUS 로 떨어진다 — 사장님은 매번 "비슷한 가게가 여럿입니다"를 읽는다.
실측(2026-08-28) 10건 전부 그랬다. 검색은 넓게, 판정은 좁게.
"""
only = _np("보사노바 커피로스터스 강릉점", "강원 강릉시 창해로14번길 28")
seen = _patch_naver(
monkeypatch, _match(kakao_client.MatchOutcome.MATCHED, only, [only], "name_exact")
)
h = await auth_headers("u1")
pid = await _place(client, h, name="보사노바 커피로스터스 강릉점")
body = (
await client.get(
f"/v1/place/{pid}/verify/candidates",
headers=h,
params={"query": "보사노바 커피로스터스 강릉점 강릉 견소동"},
)
).json()
assert body["result"]["success"] is True
assert seen["search_query"] == "보사노바 커피로스터스 강릉점 강릉 견소동"
assert seen["name"] == "보사노바 커피로스터스 강릉점"
async def test_match_name_falls_back_to_query_when_place_has_no_name(auth_headers, client, monkeypatch):
"""검증: 사업장 상호가 비어 있다(직접 입력 등으로 이름 없이 만들어진 경우).
기대결과: 검색어를 판정용 상호명으로도 쓴다 — 조회 자체가 실패하지는 않는다."""
only = _np("핑크비치펜션", "강원특별자치도 양양군")
seen = _patch_naver(
monkeypatch, _match(kakao_client.MatchOutcome.MATCHED, only, [only], "name_exact")
)
h = await auth_headers("u1")
pid = await _place(client, h, name="핑크비치펜션")
# 상호를 비운다 — places.name 이 빈 문자열인 상태를 만든다.
await client.patch(f"/v1/place/{pid}", headers=h, json={"name": ""})
body = (
await client.get(
f"/v1/place/{pid}/verify/candidates", headers=h, params={"query": "핑크비치펜션 양양"}
)
).json()
assert body["result"]["success"] is True
assert seen["name"] == "핑크비치펜션 양양"