Elaina/ephron-ren-prd

Fork 0

Files

Ubuntu 09111dc96f docs: add home admin redirect test follow-up PRD

2026-05-18 10:43:45 +08:00

4.8 KiB

Raw Blame History

Follow-up PRD：收敛 Home Admin 未登录重定向测试的实现耦合

背景

在验收 Home 相关改动时，存在一个与本次功能无关的旧测试失败：

home/tests/test_admin_permissions.py
test_admin_redirect_uses_public_home_url_when_unauthenticated

该测试失败并不表明 Home Admin 未登录跳转功能失效，而是因为测试对 redirect URL 的完整字符串做了过度刚性的断言，和当前共享登录守卫 / 共享服务 URL 逻辑产生了耦合漂移。

问题定义

当前测试断言类似：

assert query["redirect"][0] == encode_redirect("http://127.0.0.1:8000/admin?tab=drafts")
assert decode_redirect(query["redirect"][0]) == "http://127.0.0.1:8000/admin?tab=drafts"

这类断言的问题在于：

把具体实现细节写死了
- host 必须是 127.0.0.1
- scheme 必须是 http
- 端口必须是 8000
- 完整 URL 字符串必须完全一致
与共享 URL 生成逻辑耦合过深
- 当前服务 URL 由共享模块生成
- 未来若 SERVICE_DEV_HOST、SERVICE_DEV_SCHEME、base_url 或 admin guard 逻辑调整，测试会失败
- 但真实业务行为可能仍然正确
测试的重点偏了
- 它本应验证“未登录会跳到登录页，并保留回跳目标”
- 现在却变成了验证“当前内部 URL 拼接实现恰好等于某个历史字符串”

目标

将该测试从“验证固定实现细节”调整为“验证稳定行为契约”，使其：

能正确覆盖未登录重定向语义
不再因共享 URL 生成细节微调而误报失败
不改变现有业务逻辑，只修测试

方案

一、保留要测的核心行为

该测试真正应该验证的是：

未登录访问 /admin?tab=drafts
返回 302
重定向目标是登录页
登录页 URL 中包含 redirect 参数
redirect 解码后仍然指向 Home Admin
原始查询参数 tab=drafts 被保留

这才是稳定契约。

二、去掉对完整绝对 URL 字符串的硬编码

不建议继续这样断言

assert decode_redirect(query["redirect"][0]) == "http://127.0.0.1:8000/admin?tab=drafts"

原因：

这会把测试绑死在当前某个局部实现上
不利于 shared admin guard / shared ports 的后续演进

建议改为结构化断言

先解码：

redirect_target = decode_redirect(query["redirect"][0])
parsed_target = urlparse(redirect_target)
parsed_query = parse_qs(parsed_target.query)

然后断言：

assert parsed_target.path == "/admin"
assert parsed_query.get("tab") == ["drafts"]

登录页断言仍然保留

assert response.status_code == 302
parsed = urlparse(response.headers["location"])
assert parsed.path == "/login"
assert "redirect" in parse_qs(parsed.query)

三、如果确实要校验 host / scheme，必须改成从共享配置推导

如果团队认为还需要校验回跳目标的 host/scheme，也不要手写：

http://127.0.0.1:8000

而应从共享配置推导，例如：

get_service_url("home", is_dev=...)

这样至少保证：

测试与当前规范保持一致
不会因重复硬编码造成漂移

但从测试稳定性角度看，本次建议优先只验证 path + query，不要把 host 作为必须项。

修改范围

必改文件

home/tests/test_admin_permissions.py

修改对象

test_admin_redirect_uses_public_home_url_when_unauthenticated

不改范围

本次 follow-up 不修改：

home/src/routes/admin.py
shared/core/ports.py
shared admin guard
任何业务重定向逻辑

这是一个纯测试修正 PRD。

验收标准

修正后，该测试应满足：

未登录访问 /admin?tab=drafts 返回 302
location 指向 /login
query 中包含 redirect
redirect 解码后：
- path 是 /admin
- query 中 tab == drafts
不再对完整固定 URL 字符串做硬编码断言

风险控制

为什么不能直接删掉测试

因为这个测试本身验证的场景仍然有价值：

未登录状态下，admin 页面必须正确跳转到登录页
登录后回跳目标必须保留

问题不在“要不要测”，而在“测法不对”。

为什么不建议顺手改业务实现

当前没有证据表明业务重定向逻辑是坏的。失败来自测试与实现耦合过深，而不是功能错误。

因此本次 follow-up 应坚持：

只修测试，不修业务逻辑

结论

这个 follow-up 的本质，是把一个脆弱测试改成一个验证真实契约的稳定测试。

目标不是让测试“勉强通过”，而是让它以后在共享登录守卫、共享 URL 配置继续演进时，仍然只对真正的行为回归报警。

4.8 KiB Raw Blame History Unescape Escape