fnsvr-01 的 PostgreSQL 在 8432 端口(不是 5432),实测 PostgreSQL 17.6 / Alpine。 已在该实例建好独立库与角色: - dadaotask(开发)/ dadaotask_test(测试),owner 都是 dadao_owner - dadao_owner 迁移用;dadao_app 应用用(NOSUPERUSER NOBYPASSRLS) - 两个库都已 ALTER SCHEMA public OWNER TO dadao_owner + REVOKE CREATE FROM PUBLIC + GRANT 给 dadao_app 代码改动: - config 增加 test_database_url / test_migration_database_url,并给出 resolved_database_url() / resolved_migration_database_url():ENV=test 且有测试库时优先用测试库, 否则退回开发库(CI 里直接给 DATABASE_URL 即可,行为不变)。 - db.py / alembic/env.py / tests/conftest.py 统一走这两个解析方法。 原先 conftest 清表用的是 settings.migration_database_url,而应用连的是测试库, 一旦测试库与开发库不同,TRUNCATE 就清到另一个库上——表现为"测试库脏数据越攒越多、 列表类断言随机失败"。这是本次真踩到的坑,不是假想。 - tests/conftest.py 保留 E402 豁免(环境变量必须在 import app 模块前写好)。 验证: - 远程 dadaotask_test 上 pytest 33 passed,跑完表内 0 行(确认清表打在同一个库) - 远程 dadaotask 上 alembic upgrade head 成功,/health 与 /version 正常,db_role 为 dadao_app 且三项全 false - 本机便携 PG 16.4 上同样 33 passed(两个 PG 大版本都过) - ruff check / format 干净 已知限制:该实例没有 pgvector(只有 pg_trgm / pgcrypto),M3 向量检索前必须先定扩展来源, 已写进 2.Design/DEVELOPMENT.md。
This commit is contained in:
@@ -32,6 +32,10 @@ class Settings(BaseSettings):
|
||||
migration_database_url: str = (
|
||||
"postgresql+asyncpg://dadao_owner:dadao_dev_pw@127.0.0.1:5433/dadaotask"
|
||||
)
|
||||
# 测试专用库:测试会 TRUNCATE 所有表,绝不能落到开发库上。
|
||||
# 留空则退回 database_url(CI 里直接给 DATABASE_URL 即可)。
|
||||
test_database_url: str | None = None
|
||||
test_migration_database_url: str | None = None
|
||||
db_echo: bool = False
|
||||
db_pool_size: int = 10
|
||||
|
||||
@@ -79,6 +83,18 @@ class Settings(BaseSettings):
|
||||
if problems:
|
||||
raise RuntimeError("配置校验失败:" + ";".join(problems))
|
||||
|
||||
def resolved_database_url(self) -> str:
|
||||
"""应用实际连的库:测试环境优先用独立测试库(测试会 TRUNCATE)。"""
|
||||
if self.env == "test" and self.test_database_url:
|
||||
return self.test_database_url
|
||||
return self.database_url
|
||||
|
||||
def resolved_migration_database_url(self) -> str:
|
||||
"""迁移/清表实际连的库:必须和应用同一个库,否则清表清到别处去了。"""
|
||||
if self.env == "test" and self.test_migration_database_url:
|
||||
return self.test_migration_database_url
|
||||
return self.migration_database_url
|
||||
|
||||
|
||||
@lru_cache
|
||||
def get_settings() -> Settings:
|
||||
|
||||
Reference in New Issue
Block a user