Skip to content

[Docs] behaviors.md 文档审查:测试命令引用错误与缺失测试 #72

@evan-zhang11

Description

@evan-zhang11

概述

每日文档审查发现 docs/dev/behaviors.md 中存在以下问题:

🔴 严重问题

1. STORE-001: 引用的测试不存在

  • 文档声明: cargo test test_storage_*
  • 实际情况: 仓库中不存在任何 test_storage_ 相关的测试
  • 影响: 无法验证文件存储行为的正确性
  • 建议:
    • 要么添加相应的测试
    • 要么更新验证标准为其他现有测试(如 test_persistence_across_restart

2. STORE-002 & STORE-003: 引用了不存在的 Python 测试

  • 文档声明:
    • STORE-002: pytest test_db_schema
    • STORE-003: pytest test_state_machine
  • 实际情况:
    • 仓库中没有 Python 测试文件
    • 数据库 schema 在 backend/src/db.rs 中定义,但没有专门的测试验证
    • 状态机行为实际上由 Rust 测试 test_startup_reconciliation_marks_processing_as_failed 验证
  • 影响: 文档误导开发者,无法找到正确的测试
  • 建议:
    • STORE-002: 更新验证命令为检查数据库初始化代码 + 集成测试
    • STORE-003: 更新验证命令为 cargo test test_startup_reconciliation

🟡 次要问题

3. OSM 测试数据特征数量不准确

  • 文档声明:
    • OSM-003: 31,715 建筑/土地利用特征
    • OSM-004: 10,000 简单多边形
  • 测试配置 (testdata/smoke/osm_tile_test_samples.json):
    • sf_polygons: z10 瓦片显示 30,063 特征
    • sf_simple_polygons: z10 瓦片显示 9,866 特征
  • 影响: 轻微的文档不准确,不影响测试正确性
  • 建议:
    • 要么更新文档为准确的特征数量
    • 要么说明这是约数值,精确值在测试配置中定义

✅ 验证通过的部分

以下文档内容经审查与实现一致:

  1. API 端点: 所有文档中的 API 端点都在 backend/src/routes.rs 中正确实现
  2. 返回格式: API-007 的返回格式 {url, slug, isPublic} 与代码实现一致
  3. 测试覆盖: 绝大多数文档中引用的 Rust 和 E2E 测试都存在
  4. 几何类型: OSM-001 到 OSM-006 的测试数据和测试都存在且正确

建议的修复优先级

  1. P0 (立即修复):

    • 修复 STORE-001, STORE-002, STORE-003 的测试引用(避免误导)
  2. P2 (低优先级):

    • 更新 OSM 特征数量为准确值或标注为近似值

相关文件

  • docs/dev/behaviors.md - 需要更新
  • backend/tests/api_tests.rs - 实际测试位置
  • backend/src/db.rs - 数据库 schema 定义
  • testdata/smoke/osm_tile_test_samples.json - OSM 测试配置

此问题由每日文档审查 cron job (2026-02-18) 自动创建。

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions