"""架构测试:守住抓取侧与交易侧的依赖方向 拆成两个进程之后,最容易悄悄退化的不是功能而是边界——某天为了省事在交易侧 `from app.scraping.parsers...` 一句,两个服务就重新长回一起:抓取的解析改动会 牵动下单链路,交易服务也被迫加载整套抓取依赖(包括 Playwright)。 允许的方向只有两条:scraping → shared、trading → shared。 交易侧要用抓取的能力,走抓取服务的 HTTP 接口(`purchase` 块本来就是它的对外契约), 不直接 import。 """ from __future__ import annotations import ast from pathlib import Path import pytest APP_DIR = Path(__file__).resolve().parent.parent / "app" def _imported_modules(path: Path) -> set[str]: """取一个源文件里 import 到的模块名(只看真实 import,不看注释与文档字符串)""" tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path)) modules: set[str] = set() for node in ast.walk(tree): if isinstance(node, ast.Import): modules.update(alias.name for alias in node.names) elif isinstance(node, ast.ImportFrom) and node.module and node.level == 0: modules.add(node.module) return modules def _python_files(package: str) -> list[Path]: return sorted((APP_DIR / package).rglob("*.py")) @pytest.mark.parametrize( ("package", "forbidden"), [ ("scraping", "app.trading"), ("trading", "app.scraping"), # shared 是两侧的共同底座,反向依赖任何一侧都会形成环 ("shared", "app.scraping"), ("shared", "app.trading"), ], ) def test_package_does_not_import(package: str, forbidden: str): offenders = [ f"{path.relative_to(APP_DIR)} -> {module}" for path in _python_files(package) for module in _imported_modules(path) if module == forbidden or module.startswith(f"{forbidden}.") ] assert not offenders, f"{package} 不应依赖 {forbidden}:{offenders}" def test_both_entrypoints_build(): """两个入口都要能独立创建应用——这是「两个部署单元」的最低验收""" from app.scraping.main import create_app as create_scraping_app from app.trading.main import create_app as create_trading_app # 用 OpenAPI 里的路径而不是 app.routes:新版 FastAPI 把 include_router 的结果 # 包成 _IncludedRouter 而非摊平,OpenAPI 反映的才是真正对外暴露的契约 scraping_paths = set(create_scraping_app().openapi()["paths"]) trading_paths = set(create_trading_app().openapi()["paths"]) assert "/api/search" in scraping_paths assert "/api/auth/status" in trading_paths # 登录态接口不该出现在抓取服务上:抓取实例可以多开,多份登录态就是重复下单的温床 assert not any(path.startswith("/api/auth") for path in scraping_paths) assert not any(path.startswith("/api/rakuma") for path in trading_paths) # /health 两边都有,各报各的 assert "/health" in scraping_paths and "/health" in trading_paths