Files
group_fqcd_jr/tests/unit/test_advisor_migration_contract.py
T
wangjianlong_0626 4766e3bd98 feat(benefit): 客户权益功能(T010)+ 修投顾迁移契约里写死 head 的脆弱断言
## 1. 新增客户权益(用户端)

`GET /api/v1/users/me/entitlements`(T010,权限 `benefit:read:self`):

- **层级**由 `fin_customer_profile.total_asset` **实时判定**
  (门槛来自 `knowledge/product/高净值客户服务规范.md`:
  金卡 50 万 / 白金 200 万 / 钻石 600 万 / 私行 1000 万;低于 50 万为普通客户);
- **权益按层级累积展开**(文档原文"含全部下级权益,新增以下"):
  金卡 9 条 / 白金 20 / 钻石 33 / 私行 54,各档已逐档实测;
- 返回**升级提示**(`next_tier`:下一层级与门槛),前端可直接渲染"再投 X 元升级"。

### 新增表 `fin_customer_benefit`(1 张)

层级 → 权益目录,54 条种子数据(`tools/seed_customer_benefits.py`,按 `benefit_code` 幂等)。

**基线合规证明**(规则 1/3/4):只新增这一张表;**未**重命名/删除任何已有表;
**未**重命名/删除/复用任何已有字段,**未**改任何已有字段的类型、可空性或业务含义;
未改 `docs/00`。
复核:`tools/audit_schema.py` → `90 business tables, no missing or unexpected tables`。

### 两条设计取舍

1. **不落"某客户享有哪些权益"**:层级可算,权益由层级推出,两者都不落库。
   与 `docs/00` L159(不保留 `net_worth_flag`,因为可算)同一取向。
2. **权益只存各层新增条目**,累积由服务层 `tier_chain()` 展开 ——
   否则改一条权益要改四处,漏一处就出现"白金没有金卡权益"。

### 数据来源与一处刻意省略

逐条照抄知识文档,不新增文档里没有的权益。**私行那条
「7×24小时私人银行专线:400-XXX-XXXX 转 8」不写号码** ——
文档里是占位符,而对客号码的唯一来源是 `customer_service_rules.CONTACT_PHONE`
(本线此前修过"同一客服给客户两个不同号码"的缺陷)。把占位符抄进库等于再造一份假号码。

## 2. 修投顾迁移契约里写死的断言

`tests/unit/test_advisor_migration_contract.py` 原先断言

```python
assert script.get_heads()[0] == "20260911_merge_adv_risk_heads"
```

那是"投顾迁移刚加完那一刻"的快照 —— 本 PR 一新增迁移(`20260912_customer_benefit`)
它就变红,**而红的原因与投顾链的对错无关**:断言测到的是时间,不是契约。

原意是"投顾链接在这条主链上、没另起分支"。改为断言**投顾链尾是当前 head 的祖先**
(链尾从 `ADVISOR_FILES[-1]` 派生,不写死),既保住原意又不受后续迁移影响。
`len(script.get_heads()) == 1`(链不分叉)与"投顾文件首尾相接"两条原样保留。

## 3. 顺带发现的既有缺口(**不在本次改动范围**)

`app/api/controllers/trading.py` 的 **T001–T009 未调用 `AuthorizationService.require`**:
`docs/05` §19 为它们登记了权限码(`account:read:self` / `trade:order:*` / `holding:read:self`),
但代码只做认证 + 开户测评门槛,**没有执行 RBAC 权限检查**。
对照:仓库里 **26 个 service** 都调了 `require`,`trade_service` 不在其中。

本线的 T010 **按正确做法实现**:`CustomerBenefitService.entitlements_for` 先鉴权再读数据,
且**鉴权在读取客户资产之前**(有测试断言"拒绝时未查库")。
T001–T009 如何补,需架构师定口径后另行处理。

## 4. 文档

- 新增 `docs/41-客户权益功能说明.md`:表登记 + 基线合规证明 + 分层口径 + 累积规则 +
  数据来源 + 权限 + 与仪表盘的关系 + 上述缺口
- `docs/05` §19 登记 T010,并**单独注明它引入了新表**(避免被误读为
  "T 段数据库零变更"的一部分)
- `AGENTS.md` 表数 89 → **90** 张业务表

## 验证

- `pytest tests/unit/service/test_customer_benefit_service.py` → **20 passed**
  (含边界:499999.99 不是金卡、500000 整是金卡、1000 万整是私行;累积条数;升级提示;
  鉴权先于读数据)
- 全量 `pytest tests` → `2 failed, 1469 passed, 1 skipped`
  (2 个失败为既有环境项:httpx 把中文序列化成 `\uXXXX`,非本次引入)
- `ruff check app tests tools alembic` → `All checks passed`
- `mypy app` → **0 错 / 252 文件**
- 真机:`GET /users/me/entitlements` → `200`;各档分层与累积条数逐档实测通过
- `audit_schema.py` → 90 张业务表无缺失/意外;文档守卫 55 份无编号冲突;
  端点编号无重复;RBAC 种子一致性通过
2026-09-12 17:24:37 +08:00

81 lines
3.5 KiB
Python

from __future__ import annotations
import re
from pathlib import Path
from alembic.config import Config
from alembic.script import ScriptDirectory
ROOT = Path(__file__).resolve().parents[2]
VERSIONS = ROOT / "alembic" / "versions"
BASELINE = ROOT / "alembic" / "baseline_generated.sql"
ADVISOR_FILES = (
"20260910_advisor_investment_goal.py",
"20260910_advisor_product_industry_exposure.py",
"20260910_advisor_portfolio_projection_checkpoint.py",
"20260910_advisor_product_metric_snapshot.py",
"20260910_advisor_product_asset_classification.py",
"20260910_advisor_product_reference_snapshot.py",
"20260910_advisor_offsite_fund_reference.py",
"20260910_advisor_product_price_history.py",
"20260910_advisor_product_governance_reference.py",
"20260910_advisor_governance_monitor_and_quotes.py",
"20260910_advisor_market_quote_resilience.py",
"20260910_advisor_data_quality_backtest.py",
"20260911_advisor_goal_conversation.py",
"20260911_advisor_profile_tag_governance.py",
)
def created_tables(path: Path) -> set[str]:
content = path.read_text(encoding="utf-8")
return set(re.findall(r"CREATE TABLE(?: IF NOT EXISTS)?\s+`?([A-Za-z0-9_]+)`?", content))
def test_advisor_migrations_form_one_chain_from_qyqy_head() -> None:
script = ScriptDirectory.from_config(Config(str(ROOT / "alembic.ini")))
assert len(script.get_heads()) == 1
# ⚠️ 这里**不写死 head 名**。原先断言
# `script.get_heads()[0] == "20260911_merge_adv_risk_heads"`
# 那是"投顾迁移刚加完那一刻"的快照;之后任何人新增迁移(2026-09-12 的
# 客户权益迁移 `20260912_customer_benefit` 即是一例)都会让本用例变红,
# 而变红的原因与投顾链的对错**无关** —— 断言测到的是时间,不是契约。
#
# 原意是"投顾链确实接在这条主链上、没另起一条分支"。改为断言
# **投顾链尾是当前 head 的祖先**:既保住这个意思,又不受后续迁移影响。
head = script.get_heads()[0]
ancestors = {rev.revision for rev in script.iterate_revisions(head, "base")}
advisor_tail = re.search(
r'revision = "([^"]+)"',
(VERSIONS / ADVISOR_FILES[-1]).read_text(encoding="utf-8"),
)
assert advisor_tail is not None
assert advisor_tail.group(1) in ancestors
first = (VERSIONS / ADVISOR_FILES[0]).read_text(encoding="utf-8")
assert 'down_revision = "20260910_drop_review_separation"' in first
for previous, current in zip(ADVISOR_FILES, ADVISOR_FILES[1:], strict=False):
previous_content = (VERSIONS / previous).read_text(encoding="utf-8")
current_content = (VERSIONS / current).read_text(encoding="utf-8")
previous_revision = re.search(r'revision = "([^"]+)"', previous_content)
assert previous_revision is not None
assert f'down_revision = "{previous_revision.group(1)}"' in current_content
def test_advisor_migrations_only_create_additive_tables() -> None:
baseline_tables = created_tables(BASELINE)
advisor_tables: set[str] = set()
for filename in ADVISOR_FILES:
content = (VERSIONS / filename).read_text(encoding="utf-8")
assert "ALTER TABLE" not in content
assert "DROP TABLE" not in content
advisor_tables.update(created_tables(VERSIONS / filename))
assert advisor_tables
assert not advisor_tables & baseline_tables
assert "fin_sim_order" not in advisor_tables
assert "fin_transaction" not in advisor_tables
assert "fin_holding" not in advisor_tables