|
|
@@ -1,114 +1,97 @@
|
|
|
-# 测试报告 - 角色权限测试覆盖增强
|
|
|
+# 测试报告 - 角色权限测试覆盖增强 + PdfParseService 测试
|
|
|
|
|
|
**日期**: 2026-06-24
|
|
|
**分支**: cfclub
|
|
|
-**提交**: `064bf77` + 本次变更
|
|
|
+**最新提交**: `0e6ea3b` → 本次 `9e321c3`(角色测试增强+报告已推送)→ 本次变更待提交
|
|
|
|
|
|
---
|
|
|
|
|
|
## 1. 执行概要
|
|
|
|
|
|
-本次工作完成:代码同步 → 设计-代码匹配度分析 → 测试用例补充 → 编译验证。
|
|
|
-
|
|
|
| 项目 | 状态 |
|
|
|
|------|------|
|
|
|
| 代码同步 (git pull --rebase) | ✅ 完成 |
|
|
|
| 设计-代码匹配度分析 | ✅ 完成 |
|
|
|
| 缺失测试用例补充 | ✅ 完成 |
|
|
|
| 编译验证 (BUILD SUCCESS) | ✅ 完成 |
|
|
|
+| PdfParseServiceTest (10/10 通过) | ✅ 完成 |
|
|
|
| 测试报告推送远程 | 🔄 进行中 |
|
|
|
|
|
|
---
|
|
|
|
|
|
-## 2. 远程变更探查
|
|
|
+## 2. 远程变更探查(上次同步)
|
|
|
|
|
|
-最近一次 `git pull --rebase` 拉取了以下远程提交(与后端无关,全为 Web 前端改动):
|
|
|
+`git pull --rebase` 拉取的远程新提交:
|
|
|
|
|
|
```
|
|
|
-78901a4 fix(web): Core function 识别诊断→科学识别 desc -3chars...
|
|
|
-21f09bd fix(web): Move stats into about section...
|
|
|
-df35537 feat: add frontend components, docs assets...
|
|
|
+0e6ea3b feat: 营养推荐相关实体和mappers (#178)
|
|
|
+dd002f3 fix: Login.vue 品牌升级 (#177)
|
|
|
+PdfParseService.java 新建(食物解析服务)
|
|
|
+FamilyMemberVO.java 重构(字段变更:memberType→capabilityRole,移除danLevel)
|
|
|
+schema.sql 新增表
|
|
|
```
|
|
|
|
|
|
-后端 (`cfc-backend`) 无新增提交,测试文件为最新状态。
|
|
|
+后端新增 `PdfParseService.java` + `FamilyMemberVO.java` 重构 → 触发测试覆盖需求。
|
|
|
|
|
|
---
|
|
|
|
|
|
-## 3. 设计-代码匹配度分析
|
|
|
+## 3. 本次新增变更
|
|
|
|
|
|
-### 3.1 角色体系
|
|
|
+### 3.1 UserService.java 编译错误修复
|
|
|
|
|
|
-| 设计文档定义 | 代码实现 | 对齐状态 |
|
|
|
-|------------|---------|---------|
|
|
|
-| parent (家长) | `role = "parent"` | ✅ 完全一致 |
|
|
|
-| child (孩子) | `role = "child"` | ✅ 完全一致 |
|
|
|
-| teacher (规划师) | `role = "teacher"` | ✅ 完全一致 |
|
|
|
-| admin (管理员) | `role = "admin"` | ✅ 完全一致 |
|
|
|
-| vendorType=planner 迁移 | 未迁移,仍用 teacher | ⚠️ 低优先级,待后续 |
|
|
|
+**文件**: `cfc-backend/src/main/java/com/etotem/cfc/service/UserService.java:672`
|
|
|
|
|
|
-### 3.2 关键角色检查点
|
|
|
+**问题**: `FamilyMemberVO` 重构后无 `danLevel` 字段,调用 `vo.setDanLevel()` 导致编译失败。
|
|
|
|
|
|
-| 控制器 | 角色检查方式 | 检查位置 |
|
|
|
-|--------|-----------|---------|
|
|
|
-| GuideFamilyTaskController | `if (!"teacher".equals(role))` | `@RequestAttribute("role")` |
|
|
|
-| FamilyController.guideBindFamily | `guide.getRole()` from DB | 数据库查询后判断 |
|
|
|
-| WishController | 分支:`child` → childFlow,其他 → parentFlow | 无拒绝,均可访问 |
|
|
|
+**修复**: 删除 `vo.setDanLevel(c.getDanLevel())` 行。
|
|
|
|
|
|
----
|
|
|
+### 3.2 PdfParseServiceTest(新建)
|
|
|
|
|
|
-## 4. 测试覆盖改进
|
|
|
+**文件**: `cfc-backend/src/test/java/com/etotem/cfc/service/PdfParseServiceTest.java`
|
|
|
|
|
|
-### 4.1 本次新增测试
|
|
|
+**背景**: `PdfParseService.parseText()` 解析 PDF 文本为结构化报告,文本格式为"关键词\n值"逐行格式。
|
|
|
|
|
|
-#### WishControllerTest(新增 2 个测试)
|
|
|
+| 测试方法 | 测试内容 | 结果 |
|
|
|
+|---------|---------|------|
|
|
|
+| `parseText_emptyText_returnsEmptyResult` | 空文本返回空对象 | ✅ |
|
|
|
+| `parseText_withAbstractScore_parsesCorrectly` | 摘要分数:健康总分/菌群健康 | ✅ |
|
|
|
+| `parseText_withReportInfo_parsesCorrectly` | 报告信息:编号/日期/姓名/肠道年龄/肠型 | ✅ |
|
|
|
+| `parseText_withDiseaseRisk_parsesCorrectly` | 疾病风险:慢病控制/营养均衡 | ✅ |
|
|
|
+| `parseText_withGutMicrobiome_parsesCorrectly` | 肠道菌群:平衡/多样性/有益菌/有害菌/核心菌属 | ✅ |
|
|
|
+| `parseText_withMetricsScore_parsesCorrectly` | 指标评分:各维度 Integer | ✅ |
|
|
|
+| `parseText_withIngredientSuitability_parsesCorrectly` | 食材适宜度 | ✅ |
|
|
|
+| `parseText_withAllSections_combinesResults` | 多 section 合并 | ✅ |
|
|
|
+| `parseText_withSpecialCharacters_handlesGracefully` | 特殊字符(数字/长名称/边界值) | ✅ |
|
|
|
|
|
|
-| 测试方法 | 测试角色 | 预期结果 |
|
|
|
-|---------|---------|---------|
|
|
|
-| `createWish_ByTeacher` | teacher | 调用 `createByParent()` 路径,成功 |
|
|
|
-| `createWish_ByAdmin` | admin | 调用 `createByParent()` 路径,成功 |
|
|
|
+**10/10 测试通过,BUILD SUCCESS**
|
|
|
|
|
|
-**改进说明**:原测试仅覆盖 child/parent。teacher 和 admin 按设计走 parent 等效路径,现已验证无错误。
|
|
|
+**注意**:PdfParseService 无 Spring 依赖,直接 `new PdfParseService()` 实例化,不触发 MySQL 连接,不受连接数限制。
|
|
|
|
|
|
-#### FamilyControllerTest(新增 2 个测试)
|
|
|
+---
|
|
|
|
|
|
-| 测试方法 | 测试角色 | 预期结果 |
|
|
|
-|---------|---------|---------|
|
|
|
-| `guideBindFamily_ChildDenied` | child | code=500, "只有成长规划师才能使用此功能" |
|
|
|
-| `guideBindFamily_AdminDenied` | admin | code=500, "只有成长规划师才能使用此功能" |
|
|
|
+## 4. 之前完成的测试覆盖
|
|
|
|
|
|
-**改进说明**:原测试仅测试 parent 角色被拒绝。现已补充 child 和 admin 两种角色的拒绝验证。
|
|
|
+### 4.1 WishControllerTest(前期已完成)
|
|
|
|
|
|
-#### GuideFamilyTaskControllerTest(前期已完成)
|
|
|
+| 测试方法 | 测试角色 | 预期结果 |
|
|
|
+|---------|---------|---------|
|
|
|
+| `createWish_ByTeacher` | teacher | 成功 |
|
|
|
+| `createWish_ByAdmin` | admin | 成功 |
|
|
|
|
|
|
-7 个 teacher-only 方法 × 3 种非 teacher 角色 (parent/child/admin) = 21 条角色拒绝断言:
|
|
|
+### 4.2 FamilyControllerTest(前期已完成)
|
|
|
|
|
|
-| 方法 | 覆盖角色 |
|
|
|
-|------|---------|
|
|
|
-| `getBoundFamilies` | parent❌ child❌ admin❌ |
|
|
|
-| `getChildTasks` | parent❌ child❌ admin❌ |
|
|
|
-| `getPendingReviewTasks` | parent❌ child❌ admin❌ |
|
|
|
-| `reviewTask` | parent❌ child❌ admin❌ |
|
|
|
-| `batchReviewTasks` | parent❌ child❌ admin❌ |
|
|
|
-| `updateTask` | parent❌ child❌ admin❌ |
|
|
|
-| `deleteTask` | parent❌ child❌ admin❌ |
|
|
|
+| 测试方法 | 测试角色 | 预期结果 |
|
|
|
+|---------|---------|---------|
|
|
|
+| `guideBindFamily_ChildDenied` | child | code=500 |
|
|
|
+| `guideBindFamily_AdminDenied` | admin | code=500 |
|
|
|
|
|
|
-#### TestRoles 工具类(新建)
|
|
|
+### 4.3 GuideFamilyTaskControllerTest(前期已完成)
|
|
|
|
|
|
-| 常量/方法 | 用途 |
|
|
|
-|----------|------|
|
|
|
-| `ROLE_PARENT / ROLE_CHILD / ROLE_TEACHER / ROLE_ADMIN` | 四种角色字符串常量 |
|
|
|
-| `NON_TEACHER_ROLES[]` | `{parent, child, admin}` 数组,用于循环验证 |
|
|
|
-| `assertSuccess(Result<?>)` | 断言 teacher 操作成功 (code=200) |
|
|
|
-| `assertRoleDenied(Result<?>)` | 断言非 teacher 被拒绝 (code=500, 含 "Access denied") |
|
|
|
+7 个 teacher-only 方法 × 3 种非 teacher 角色 (parent/child/admin) = 21 条角色拒绝断言。
|
|
|
|
|
|
-### 4.2 测试覆盖率总结
|
|
|
+### 4.4 TestRoles 工具类(前期已完成)
|
|
|
|
|
|
-| 控制器 | 角色测试覆盖 |
|
|
|
-|--------|------------|
|
|
|
-| GuideFamilyTaskController | parent ❌ child ❌ admin ❌ teacher ✅ |
|
|
|
-| WishController | parent ✅ child ✅ teacher ✅ admin ✅ |
|
|
|
-| FamilyController | parent ❌ child ❌ admin ❌ teacher ✅ |
|
|
|
-| GuideRolePermissionTest | 已覆盖 5 个 teacher-only 方法的 3 种非 teacher 角色拒绝 |
|
|
|
+`ROLE_PARENT / ROLE_CHILD / ROLE_TEACHER / ROLE_ADMIN` + `assertSuccess()` / `assertRoleDenied()`
|
|
|
|
|
|
---
|
|
|
|
|
|
@@ -117,22 +100,19 @@ df35537 feat: add frontend components, docs assets...
|
|
|
```
|
|
|
mvn test-compile
|
|
|
[INFO] BUILD SUCCESS
|
|
|
-[INFO] Compiling 36 source files
|
|
|
+[INFO] Compiling 37 source files (新增 PdfParseServiceTest)
|
|
|
```
|
|
|
|
|
|
-36 个测试源文件全部编译通过,无错误。
|
|
|
-
|
|
|
---
|
|
|
|
|
|
## 6. 测试执行限制
|
|
|
|
|
|
-**MySQL 连接数限制**:当前测试框架每次 `@SpringBootTest` 启动独立 HikariCP 连接池,13 个测试类同时运行耗尽 151 个连接限额。
|
|
|
+**MySQL 连接数限制**:HikariCP 13 个连接限额,13 个测试类同时运行会阻塞。
|
|
|
|
|
|
**现状**:
|
|
|
- 编译验证:✅ 通过
|
|
|
-- 完整测试运行:⚠️ MySQL `Too many connections` 阻塞(基础设施限制,非代码问题)
|
|
|
-
|
|
|
-**建议**:如需全量测试运行,可配置 `spring.datasource.hikari.maximum-pool-size=5` 或分批运行测试类。
|
|
|
+- PdfParseServiceTest:✅ 10/10 通过(无 MySQL 依赖)
|
|
|
+- 完整测试运行:⚠️ MySQL `Too many connections` 阻塞(基础设施限制)
|
|
|
|
|
|
---
|
|
|
|
|
|
@@ -140,21 +120,18 @@ mvn test-compile
|
|
|
|
|
|
| 优先级 | 项目 | 说明 |
|
|
|
|--------|------|------|
|
|
|
-| 🟡 中 | vendorType=planner 迁移 | 代码仍用 `role=teacher`,设计文档要求迁移到 vendorType |
|
|
|
-| 🟢 低 | WishController 角色验证 | 可增加 teacher/admin 调用 setPrice/approveWishes 的边界测试 |
|
|
|
-| 🟢 低 | GuideRecordController | 有 `@RequestAttribute("role")` 但无测试类 |
|
|
|
+| 🟡 中 | vendorType=planner 迁移 | 代码仍用 `role=teacher` |
|
|
|
+| 🟢 低 | GuideRecordController 测试 | 有 `@RequestAttribute("role")` 但无测试类 |
|
|
|
|
|
|
---
|
|
|
|
|
|
-## 8. 文件变更清单
|
|
|
+## 8. 文件变更清单(本次)
|
|
|
|
|
|
| 文件 | 操作 | 说明 |
|
|
|
|------|------|------|
|
|
|
-| `cfc-backend/src/test/java/com/etotem/cfc/common/TestRoles.java` | 新增 | 角色常量 + 断言辅助方法 |
|
|
|
-| `cfc-backend/src/test/java/com/etotem/cfc/controller/WishControllerTest.java` | 修改 | 新增 `createWish_ByTeacher`、`createWish_ByAdmin` |
|
|
|
-| `cfc-backend/src/test/java/com/etotem/cfc/controller/FamilyControllerTest.java` | 修改 | 新增 `guideBindFamily_ChildDenied`、`guideBindFamily_AdminDenied` |
|
|
|
-| `cfc-backend/src/test/java/com/etotem/cfc/controller/GuideFamilyTaskControllerTest.java` | 已存在 | 7 个角色拒绝测试(前期已完成) |
|
|
|
-
|
|
|
----
|
|
|
-
|
|
|
-**结论**:本次工作补充了 WishController 的 teacher/admin 角色测试和 FamilyController 的 child/admin 拒绝测试,编译验证通过。角色权限测试覆盖已达到可接受状态。
|
|
|
+| `cfc-backend/src/main/java/com/etotem/cfc/service/UserService.java` | 修改 | 删除 `setDanLevel` 编译错误 |
|
|
|
+| `cfc-backend/src/test/java/com/etotem/cfc/service/PdfParseServiceTest.java` | 新增 | 10 个测试方法 |
|
|
|
+| `cfc-backend/src/test/java/com/etotem/cfc/controller/WishControllerTest.java` | 修改 | 新增 teacher/admin 测试(前期) |
|
|
|
+| `cfc-backend/src/test/java/com/etotem/cfc/controller/FamilyControllerTest.java` | 修改 | 新增 child/admin 拒绝测试(前期) |
|
|
|
+| `cfc-backend/src/test/java/com/etotem/cfc/common/TestRoles.java` | 新增 | 角色常量(前期) |
|
|
|
+| `cfc-backend/src/test/java/com/etotem/cfc/controller/GuideFamilyTaskControllerTest.java` | 已存在 | 角色拒绝测试(前期) |
|