胡绍康andClaude Opus 4.6
f7a8817970
- 使用 Black 格式化 47 个 Python 文件 - 使用 isort 修复 import 排序 - 修复 Code Quality CI/CD 检查失败问题 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
74 lines
2.7 KiB
Python
74 lines
2.7 KiB
Python
import ast
|
||
import unittest
|
||
from pathlib import Path
|
||
|
||
|
||
class ModuleBoundaryTests(unittest.TestCase):
|
||
@staticmethod
|
||
def _collect_import_modules(path: Path) -> set[str]:
|
||
tree = ast.parse(path.read_text(encoding="utf-8"), filename=str(path))
|
||
modules: set[str] = set()
|
||
|
||
for node in ast.walk(tree):
|
||
if isinstance(node, ast.Import):
|
||
for alias in node.names:
|
||
modules.add(alias.name)
|
||
elif isinstance(node, ast.ImportFrom):
|
||
if node.module:
|
||
modules.add(node.module)
|
||
return modules
|
||
|
||
def _assert_forbidden_imports(self, *, path: Path, forbidden_prefixes: tuple[str, ...]):
|
||
imports = self._collect_import_modules(path)
|
||
for module in sorted(imports):
|
||
for prefix in forbidden_prefixes:
|
||
if module == prefix or module.startswith(prefix + "."):
|
||
self.fail(f"模块边界违规：{path} 导入了禁止依赖 {module!r}（匹配前缀 {prefix!r}）")
|
||
|
||
def test_routes_do_not_import_services_repos_or_legacy(self):
|
||
repo_root = Path(__file__).resolve().parents[1]
|
||
routes_dir = repo_root / "outlook_web" / "routes"
|
||
self.assertTrue(routes_dir.exists())
|
||
|
||
forbidden = (
|
||
"outlook_web.legacy",
|
||
"outlook_web.services",
|
||
"outlook_web.repositories",
|
||
"outlook_web.db",
|
||
)
|
||
for path in sorted(routes_dir.glob("*.py")):
|
||
if path.name == "__init__.py":
|
||
continue
|
||
self._assert_forbidden_imports(path=path, forbidden_prefixes=forbidden)
|
||
|
||
def test_repositories_do_not_depend_on_flask_routes_or_services(self):
|
||
repo_root = Path(__file__).resolve().parents[1]
|
||
repos_dir = repo_root / "outlook_web" / "repositories"
|
||
self.assertTrue(repos_dir.exists())
|
||
|
||
forbidden = (
|
||
"flask",
|
||
"outlook_web.routes",
|
||
"outlook_web.services",
|
||
"outlook_web.legacy",
|
||
)
|
||
for path in sorted(repos_dir.glob("*.py")):
|
||
if path.name == "__init__.py":
|
||
continue
|
||
self._assert_forbidden_imports(path=path, forbidden_prefixes=forbidden)
|
||
|
||
def test_services_do_not_depend_on_flask_routes_or_legacy(self):
|
||
repo_root = Path(__file__).resolve().parents[1]
|
||
services_dir = repo_root / "outlook_web" / "services"
|
||
self.assertTrue(services_dir.exists())
|
||
|
||
forbidden = (
|
||
"flask",
|
||
"outlook_web.routes",
|
||
"outlook_web.legacy",
|
||
)
|
||
for path in sorted(services_dir.glob("*.py")):
|
||
if path.name == "__init__.py":
|
||
continue
|
||
self._assert_forbidden_imports(path=path, forbidden_prefixes=forbidden)
|