o2o-site-AEO/solution/backend/tests/test_site_slug.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

141 lines
6.5 KiB
Python

"""사이트 주소(네임스페이스) 확인·예약.
이 경로가 절대 하면 안 되는 것:
- 한글·대문자 주소를 통과시키는 것 — 주소가 퍼센트 인코딩 덩어리가 되어 사장님이 불러줄 수 없다
- 확인은 통과시키고 저장에서 튕기는 것 — 규칙이 두 곳에 있으면 반드시 생긴다
- ★ 이미 발행돼 색인된 주소를 바꾸는 것 — AI 검색이 잡아 둔 페이지가 404 가 된다
"""
import uuid
import pytest
from sqlalchemy import text
from common.enums import ErrorType, SiteStatus
from services import site_slug
async def _place(client, headers, name="주소펜션"):
r = await client.post("/v1/place", headers=headers, json={"name": name, "category": 1})
return r.json()["place"]["place_id"]
async def _check(client, headers, pid, slug):
return (await client.get(f"/v1/place/{pid}/site/slug/check", headers=headers, params={"slug": slug})).json()
async def _reserve(client, headers, pid, slug):
return (await client.post(f"/v1/place/{pid}/site/slug", headers=headers, json={"slug": slug})).json()
@pytest.mark.parametrize(
"slug, reason",
[
("doflo", None),
("stay-mumum-2", None),
("도플로", site_slug.REASON_FORMAT), # 한글
("Doflo", site_slug.REASON_FORMAT), # 대문자
("my_site", site_slug.REASON_FORMAT), # 언더스코어
("a--b", site_slug.REASON_FORMAT), # 연속 하이픈(퓨니코드 접두 xn-- 와 헷갈린다)
("-doflo", site_slug.REASON_FORMAT),
("doflo-", site_slug.REASON_FORMAT),
("ab", site_slug.REASON_LENGTH), # 3자 미만
("a" * 51, site_slug.REASON_LENGTH), # 50자 초과
("admin", site_slug.REASON_RESERVED),
("api", site_slug.REASON_RESERVED),
("robots", site_slug.REASON_RESERVED),
("undefined", site_slug.REASON_RESERVED), # 프론트 버그가 그대로 주소가 되는 걸 막는다
],
)
def test_slug_rules(slug, reason):
"""검증: 형식·예약어 규칙(확인과 저장이 같이 쓰는 단 하나의 정의).
기대결과: 쓸 수 있으면 None, 아니면 사유 코드."""
assert site_slug.validate_slug(slug) == reason
async def test_check_then_reserve(auth_headers, client):
"""검증: 주소를 확인하고 예약한다. 사이트 행이 없어도 만들어진다.
기대결과: available → 저장 → 자기 주소이므로 다시 확인해도 available."""
h = await auth_headers("slug1")
pid = await _place(client, h)
assert (await _check(client, h, pid, "doflo"))["available"] is True
saved = await _reserve(client, h, pid, "doflo")
assert saved["result"]["code"] == ErrorType.SUCCESS.value
assert saved["site"]["domain"] == "doflo"
# ★ 자기 자신은 중복이 아니다 — 저장해 둔 화면을 다시 열었을 때 '중복'이라고 하면 안 된다.
again = await _check(client, h, pid, "doflo")
assert again["available"] is True and "reason" not in again
async def test_invalid_slug_is_refused_on_save_too(auth_headers, client):
"""검증: 확인에서 막힌 값은 저장에서도 막힌다(클라이언트 검증을 믿지 않는다).
기대결과: 두 경로가 같은 사유를 돌려준다."""
h = await auth_headers("slug2")
pid = await _place(client, h)
for slug, reason in (("도플로", site_slug.REASON_FORMAT), ("ab", site_slug.REASON_LENGTH),
("admin", site_slug.REASON_RESERVED)):
checked = await _check(client, h, pid, slug)
assert checked["available"] is False and checked["reason"] == reason
saved = await _reserve(client, h, pid, slug)
assert saved["result"]["success"] is False and saved["reason"] == reason
assert "site" not in saved
async def test_taken_by_other_place_offers_suggestion(auth_headers, client):
"""검증: 남이 쓰는 주소는 못 쓴다. 대신 쓸 수 있는 대안을 하나 준다.
기대결과: available=false / TAKEN / suggestion=doflo-2."""
h = await auth_headers("slug3")
mine = await _place(client, h, "도플로")
other = await _place(client, h, "도플로2호점")
await _reserve(client, h, mine, "doflo")
checked = await _check(client, h, other, "doflo")
assert checked["available"] is False
assert checked["reason"] == site_slug.REASON_TAKEN
assert checked["suggestion"] == "doflo-2"
saved = await _reserve(client, h, other, "doflo")
assert saved["result"]["code"] == ErrorType.DB_ALREADY_SAME_KEY.value
assert saved["suggestion"] == "doflo-2"
# 제안대로면 통과한다 — 제안이 실제로 쓸 수 있는 값이어야 의미가 있다.
assert (await _reserve(client, h, other, "doflo-2"))["site"]["domain"] == "doflo-2"
async def test_published_site_slug_is_locked(auth_headers, client, db_engine):
"""검증: ★ 이미 발행된 사이트의 주소는 바꿀 수 없다.
기대결과: SITE_SLUG_LOCKED. 같은 값 재전송은 변경이 아니므로 통과."""
h = await auth_headers("slug4")
pid = await _place(client, h)
await _reserve(client, h, pid, "published-stay")
# 발행 상태를 만든다(빌드 잡을 돌리는 대신 상태만) — 잠금은 published_at·status 가 근거다.
async with db_engine.begin() as conn:
await conn.execute(
text("UPDATE sites SET status = :st, published_at = now() WHERE place_id = :p"),
{"st": SiteStatus.PUBLISHED.value, "p": uuid.UUID(pid)},
)
locked = await _reserve(client, h, pid, "published-stay-new")
assert locked["result"]["code"] == ErrorType.SITE_SLUG_LOCKED.value
assert locked["reason"] == site_slug.REASON_LOCKED
# 같은 값 재전송은 변경이 아니다.
same = await _reserve(client, h, pid, "published-stay")
assert same["result"]["code"] == ErrorType.SUCCESS.value
assert same["site"]["domain"] == "published-stay"
async def test_other_company_place_is_blocked(auth_headers, client, other_company_id):
"""검증: 남의 회사 사업장 주소는 확인도 예약도 못 한다.
기대결과: PLACE_NOT_FOUND(존재 여부조차 알려주지 않는다)."""
h = await auth_headers("slug5")
intruder = await auth_headers("slug6", other_company_id)
pid = await _place(client, h)
assert (await _check(client, intruder, pid, "doflo"))["result"]["code"] == ErrorType.PLACE_NOT_FOUND.value
assert (await _reserve(client, intruder, pid, "doflo"))["result"]["code"] == ErrorType.PLACE_NOT_FOUND.value