2603220525: 2603220459: 添加全面的测试套件覆盖核心功能。同时更新了CI工作流配置。

feat
  1. 2603220459: 添加全面的测试套件覆盖核心功能。同时更新了CI工作流配置。

Author: cimorn

.github/workflows/ci.yml

@@ -10,10 +10,10 @@ jobs:
 
     steps:
       - name: Check out repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Set up Python
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@v6
         with:
           python-version: "3.13"
 

.github/workflows/release.yml

@@ -14,10 +14,10 @@ jobs:
 
     steps:
       - name: Check out repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Set up Python
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@v6
         with:
           python-version: "3.13"
 
@@ -39,7 +39,7 @@ jobs:
           powershell -ExecutionPolicy Bypass -File ./.scripts/package_release.ps1
 
       - name: Upload release artifact
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v6
         with:
           name: weekflow-windows-release
           path: |
@@ -51,10 +51,10 @@ jobs:
 
     steps:
       - name: Check out repository
-        uses: actions/checkout@v4
+        uses: actions/checkout@v5
 
       - name: Set up Python
-        uses: actions/setup-python@v5
+        uses: actions/setup-python@v6
         with:
           python-version: "3.13"
 
@@ -81,7 +81,7 @@ jobs:
           ./.scripts/package_macos_release.sh
 
       - name: Upload release artifact
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v6
         with:
           name: weekflow-macos-release
           path: dist/*.zip
@@ -94,7 +94,7 @@ jobs:
 
     steps:
       - name: Download built artifacts
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@v5
         with:
           path: release-assets
           merge-multiple: true

.gitignore

@@ -10,7 +10,6 @@ build/
 dist/
 releases/
 WeekFlow.spec
-tests/
 # Local environment files
 .env
 

tests/__init__.py

@@ -0,0 +1 @@
+"""Test package for WeekFlow."""

tests/test_ai_config.py

@@ -0,0 +1,33 @@
+from WeekFlow.services.ai_config import (
+    DEFAULT_AI_CONFIGS,
+    normalize_ai_provider,
+)
+
+
+def test_normalize_ai_provider_prefers_operator_url_for_plan():
+    provider, defaults = normalize_ai_provider(
+        "volcengine_ark",
+        "https://operator.las.cn-beijing.volces.com/api/v1",
+    )
+
+    assert provider == "volcengine_plan"
+    assert defaults["base_url"] == "https://operator.las.cn-beijing.volces.com/api/v1"
+    assert defaults["model"] == DEFAULT_AI_CONFIGS["volcengine_plan"]["model"]
+
+
+def test_normalize_ai_provider_prefers_ark_url_for_ark():
+    provider, defaults = normalize_ai_provider(
+        "volcengine_plan",
+        "https://ark.cn-beijing.volces.com/api/v3",
+    )
+
+    assert provider == "volcengine_ark"
+    assert defaults["base_url"] == "https://ark.cn-beijing.volces.com/api/v3"
+    assert defaults["model"] == DEFAULT_AI_CONFIGS["volcengine_ark"]["model"]
+
+
+def test_normalize_ai_provider_maps_legacy_operator_name():
+    provider, defaults = normalize_ai_provider("volcengine_operator", "")
+
+    assert provider == "volcengine_plan"
+    assert defaults["base_url"] == DEFAULT_AI_CONFIGS["volcengine_plan"]["base_url"]

tests/test_ai_service.py

@@ -0,0 +1,139 @@
+import json
+from io import BytesIO
+from urllib.error import HTTPError
+
+import pytest
+
+from WeekFlow.models.report import ProjectItem, WeeklyReport
+from WeekFlow.services.ai import AIConfigError, AIService
+
+
+def _configured_report(**kwargs) -> WeeklyReport:
+    report = WeeklyReport(**kwargs)
+    report.ai["config"]["api_key"] = "test-key"
+    return report
+
+
+def test_ai_service_requires_configuration():
+    report = WeeklyReport()
+    report.ai["config"]["api_key"] = ""
+
+    service = AIService()
+
+    try:
+        service.test_connection(report)
+    except AIConfigError as exc:
+        assert "API Key" in str(exc)
+    else:  # pragma: no cover
+        raise AssertionError("Expected AIConfigError when API Key is missing")
+
+
+def test_ai_service_summarizes_basic_info_from_full_report(monkeypatch):
+    report = _configured_report(
+        topic="社区活动筹备",
+        achievements=["完成了志愿者分工表", "整理了活动物资清单"],
+        projects=[ProjectItem(name="项目 A", summary="完善报名说明", next_step="继续确认现场安排")],
+    )
+    service = AIService()
+    seen_messages = {}
+
+    def fake_chat(_config, messages):
+        seen_messages["messages"] = messages
+        return json.dumps({"one_line_summary": "集中整理了活动安排与资料信息。"}, ensure_ascii=False)
+
+    monkeypatch.setattr(service, "_chat", fake_chat)
+
+    polished = service.polish_current_section(report, section_key="basic_info")
+
+    assert polished.one_line_summary == "集中整理了活动安排与资料信息。"
+    prompt_text = seen_messages["messages"][1]["content"]
+    assert "项目 A" in prompt_text
+    assert "完成了志愿者分工表" in prompt_text
+
+
+def test_ai_service_polishes_weekly_results_with_stubbed_response(monkeypatch):
+    report = _configured_report(achievements=["旧成果"])
+    service = AIService()
+
+    def fake_chat(_config, _messages):
+        return json.dumps({"achievements": ["新成果", "补充说明"]}, ensure_ascii=False)
+
+    monkeypatch.setattr(service, "_chat", fake_chat)
+
+    polished = service.polish_current_section(report, section_key="overview")
+
+    assert polished.achievements == ["新成果", "补充说明"]
+
+
+def test_ai_service_polishes_current_project_with_stubbed_response(monkeypatch):
+    report = _configured_report(projects=[ProjectItem(name="项目 A", summary="旧内容", next_step="旧计划")])
+    service = AIService()
+
+    def fake_chat(_config, _messages):
+        return json.dumps({"summary": "新内容", "next_step": "新计划"}, ensure_ascii=False)
+
+    monkeypatch.setattr(service, "_chat", fake_chat)
+
+    polished = service.polish_current_section(report, section_key="projects", project_index=0)
+
+    assert polished.projects[0].name == "项目 A"
+    assert polished.projects[0].summary == "新内容"
+    assert polished.projects[0].next_step == "新计划"
+
+
+def test_ai_service_uses_volcengine_plan_defaults():
+    report = WeeklyReport()
+
+    assert report.ai["provider"] == "volcengine_plan"
+    assert report.ai["config"]["model"] == "doubao-seed-2-0-pro-260215"
+
+
+def test_ai_service_uses_ark_sdk_for_volcengine_ark(monkeypatch):
+    report = _configured_report()
+    report.ai["provider"] = "volcengine_ark"
+    report.ai["config"]["base_url"] = "https://ark.cn-beijing.volces.com/api/v3"
+    report.ai["config"]["model"] = "doubao-seed-2-0-lite-260215"
+    service = AIService()
+    called = {}
+
+    def fake_ark(self, config, messages):
+        called["provider"] = config.provider
+        called["message_count"] = len(messages)
+        return "连接成功"
+
+    def fail_openai(self, config, messages):  # pragma: no cover
+        pytest.fail("volcengine_ark provider should use Ark SDK path")
+
+    monkeypatch.setattr(AIService, "_chat_with_ark", fake_ark, raising=False)
+    monkeypatch.setattr(AIService, "_chat_with_openai_compatible", fail_openai, raising=False)
+
+    assert service.test_connection(report) == "连接成功"
+    assert called["provider"] == "volcengine_ark"
+    assert called["message_count"] == 2
+
+
+def test_ai_service_maps_http_401_to_clear_configuration_hint(monkeypatch):
+    report = _configured_report()
+    service = AIService()
+
+    def fake_urlopen(_req, timeout=60):
+        raise HTTPError(
+            url="https://operator.las.cn-beijing.volces.com/api/v1/chat/completions",
+            code=401,
+            msg="Unauthorized",
+            hdrs=None,
+            fp=BytesIO(b'{"path":"/api/v1/chat/completions","message":"Unauthorized"}'),
+        )
+
+    monkeypatch.setattr("WeekFlow.services.ai.request.urlopen", fake_urlopen)
+
+    try:
+        service.test_connection(report)
+    except AIConfigError as exc:
+        message = str(exc)
+        assert "HTTP 401" in message
+        assert "火山引擎plan" in message
+        assert "API Key" in message
+        assert "Base URL" in message
+    else:  # pragma: no cover
+        raise AssertionError("Expected AIConfigError for HTTP 401")

tests/test_build_icon.py

@@ -0,0 +1,13 @@
+from pathlib import Path
+
+
+ROOT = Path(__file__).resolve().parents[1]
+
+
+def test_build_script_uses_versioned_app_icon():
+    ico_icon_path = ROOT / "src" / "weekflow_logo.ico"
+    build_script = (ROOT / ".scripts" / "build_exe.ps1").read_text(encoding="utf-8")
+
+    assert ico_icon_path.exists()
+    assert "--icon" in build_script
+    assert "src\\weekflow_logo.ico" in build_script or "src/weekflow_logo.ico" in build_script

tests/test_demo_assets.py

@@ -0,0 +1,26 @@
+import json
+from pathlib import Path
+
+
+ROOT = Path(__file__).resolve().parents[1]
+OLD_DEMO_JSON = ROOT / "examples" / "demo-2611.data.json"
+NEW_DEMO_JSON = ROOT / "examples" / "demo-community.data.json"
+NEW_DEMO_MD = ROOT / "examples" / "demo-community.md"
+
+
+def test_old_demo_json_is_removed_and_new_demo_assets_exist():
+    assert not OLD_DEMO_JSON.exists()
+    assert NEW_DEMO_JSON.exists()
+    assert NEW_DEMO_MD.exists()
+
+
+def test_new_demo_assets_use_new_chinese_content():
+    payload = json.loads(NEW_DEMO_JSON.read_text(encoding="utf-8"))
+    markdown_text = NEW_DEMO_MD.read_text(encoding="utf-8")
+
+    assert payload["report_id"] == "第08周"
+    assert payload["topic"] == "社区活动筹备与资料整理"
+    assert payload["projects"][0]["name"] == "场地与物资确认"
+    assert payload["achievements"][0] == "整理了活动流程、物资清单和志愿者分工表。"
+    assert "Offline RL" not in markdown_text
+    assert "社区活动筹备与资料整理" in markdown_text

tests/test_editor_controller.py

@@ -0,0 +1,71 @@
+import json
+from pathlib import Path
+
+from WeekFlow.controllers.editor_controller import EditorController
+from WeekFlow.models.report import WeeklyReport
+
+
+def test_new_report_state_creation_uses_requested_identity(tmp_path: Path):
+    controller = EditorController(default_directory=tmp_path)
+
+    controller.create_new_report(
+        report_id="2611",
+        cycle="2026.03.12 - 2026.03.18",
+        topic="社区活动筹备",
+    )
+
+    assert controller.report.report_id == "2611"
+    assert controller.report.cycle == "2026.03.12 - 2026.03.18"
+    assert controller.current_json_path is None
+    assert controller.current_markdown_path is None
+    assert controller.report.one_line_summary == ""
+    assert controller.report.achievements == []
+    assert len(controller.report.projects) == 1
+    assert controller.report.projects[0].name == ""
+    assert controller.report.projects[0].records == []
+    assert controller.report.ai["provider"] == "volcengine_plan"
+    assert controller.is_dirty is True
+
+
+def test_load_existing_json_updates_controller_state(tmp_path: Path):
+    controller = EditorController(default_directory=tmp_path)
+    payload = WeeklyReport(report_id="2611", topic="社区活动筹备")
+    json_path = tmp_path / "2611.data.json"
+    json_path.write_text(
+        json.dumps(payload.to_dict(), ensure_ascii=False),
+        encoding="utf-8",
+    )
+
+    controller.load_from_json(json_path)
+
+    assert controller.report.report_id == payload.report_id
+    assert controller.report.topic == payload.topic
+    assert controller.current_json_path == json_path
+    assert controller.current_markdown_path == tmp_path / "2611.md"
+    assert controller.is_dirty is False
+
+
+def test_save_current_pair_updates_current_paths(tmp_path: Path):
+    controller = EditorController(default_directory=tmp_path)
+    controller.create_new_report(report_id="2611")
+
+    json_path, markdown_path = controller.save_current()
+
+    assert json_path == tmp_path / "2611.data.json"
+    assert markdown_path == tmp_path / "2611.md"
+    assert controller.current_json_path == json_path
+    assert controller.current_markdown_path == markdown_path
+    assert controller.is_dirty is False
+
+
+def test_save_as_named_version_uses_custom_stem(tmp_path: Path):
+    controller = EditorController(default_directory=tmp_path)
+    controller.create_new_report(report_id="2611", topic="社区活动筹备")
+
+    json_path, markdown_path = controller.save_as_named_version("2611-lab-note")
+
+    assert json_path == tmp_path / "2611-lab-note.data.json"
+    assert markdown_path == tmp_path / "2611-lab-note.md"
+    assert controller.current_json_path == json_path
+    assert controller.current_markdown_path == markdown_path
+    assert controller.current_stem == "2611-lab-note"

tests/test_editor_window.py

@@ -0,0 +1,36 @@
+import os
+from pathlib import Path
+
+os.environ.setdefault("QT_QPA_PLATFORM", "offscreen")
+
+from PySide6.QtWidgets import QApplication, QPushButton
+
+from WeekFlow.controllers.editor_controller import EditorController
+from WeekFlow.ui.editor_window import EditorWindow
+
+
+def _build_window(tmp_path: Path) -> EditorWindow:
+    app = QApplication.instance() or QApplication([])
+    controller = EditorController(default_directory=tmp_path)
+    controller.create_new_report(report_id="2611", topic="中文演示项目")
+    window = EditorWindow(controller)
+    QApplication.processEvents()
+    return window
+
+
+def test_editor_window_does_not_show_generate_report_button(tmp_path: Path):
+    window = _build_window(tmp_path)
+
+    button_texts = [button.text() for button in window.findChildren(QPushButton)]
+
+    assert "生成周报" not in button_texts
+
+    window.close()
+
+
+def test_editor_window_status_row_shows_only_rendered_title(tmp_path: Path):
+    window = _build_window(tmp_path)
+
+    assert window.report_label.text() == "Week 11"
+
+    window.close()