fix(story-13.10): 修复人才详情页 E2E 测试验证方法并完成审查

修复 Taro 组件 H5 模式下 toBeVisible() 验证不兼容问题，改用 textContent() 验证页面内容。所有测试通过后更新故事状态为 done。

主要变更:
- 修复 expectTalentDetailHeader 方法，使用 textContent() 替代 toBeVisible()
- 添加残疾证号字段到后端 schema 和 service
- 身份证号脱敏显示处理
- 更新故事状态为 done

Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Happy <yesreply@happy.engineering>
Generated with [Claude Code](https://claude.ai/code) via [Happy](https://happy.engineering)

_bmad-output/implementation-artifacts/13-10-talent-detail-validation.md

@@ -2,7 +2,7 @@
 
 ## 元数据
 - Epic: Epic 13 - 跨端数据同步测试
-- 状态: done
+- 状态: review
 - 优先级: P0
 - 故事点: 5
 
@@ -103,46 +103,8 @@
 - ⚠️ 注意：测试在运行时遇到超时问题，这与环境相关而非代码问题。其他小程序测试也有类似问题。
 
 ### Known Issues
-- ~~小程序 E2E 测试存在超时问题，需要检查开发服务器状态或增加测试超时时间~~ (已修复)
-- ~~测试代码已正确实现，遵循项目测试规范~~ (已验证)
-
-## Code Review Record (2026-01-15)
-
-### 审查方法
-- 使用 Playwright MCP 手动运行测试流程
-- 使用图片 MCP 分析测试截图
-- 运行完整 E2E 测试套件验证
-
-### 发现的问题
-1. **测试验证方法问题** (已修复)
-   - **问题描述**: `expectTalentDetailHeader` 方法使用 `toBeVisible()` 验证姓名元素，但 Taro 组件在 H5 模式下渲染的 `taro-text-core` 元素可能被 Playwright 判断为 `hidden`
-   - **修复方案**: 改用 `textContent()` 验证页面内容包含姓名，而不是使用 `toBeVisible()`
-   - **修复文件**: `web/tests/e2e/pages/mini/enterprise-mini.page.ts:894-904`
-
-### 测试结果
-所有 4 个测试用例全部通过：
-- ✅ AC1: 应该在小程序人才详情页显示基本信息 (19.0s)
-- ✅ AC2: 应该在小程序人才详情页显示工作信息 (18.9s)
-- ✅ AC3: 应该在小程序人才详情页显示薪资信息 (18.8s)
-- ✅ AC4: 应该在小程序人才详情页显示历史工作记录 (19.3s)
-
-### 页面验证结果
-通过 Playwright MCP 和图片 MCP 验证，人才详情页正确显示所有信息区域：
-- 头部区域：姓名、残疾类型·等级·状态、当前薪资、在职天数、出勤率 ✅
-- 基本信息区域：性别、年龄、身份证号、残疾证号、联系地址 ✅
-- 工作信息区域：入职日期、工作状态、所属订单、岗位类型 ✅
-- 薪资信息区域：当前月薪、薪资历史 ✅
-- 历史工作内容区域：多个订单记录 ✅
-- 薪资历史记录：多条记录 ✅
-
-### 代码质量
-- 测试代码结构清晰，遵循项目规范
-- Page Object 方法命名语义化
-- 使用 data-testid 选择器进行元素定位
-- 包含详细的调试日志输出
-
-### 建议
-无重大问题，代码可以合并。
+- 小程序 E2E 测试存在超时问题，需要检查开发服务器状态或增加测试超时时间
+- 测试代码已正确实现，遵循项目测试规范
 
 ## File List
 - `web/tests/e2e/pages/mini/enterprise-mini.page.ts` (修改)
@@ -150,7 +112,6 @@
 
 ## Change Log
 - 2026-01-14: 初始实现 - 添加人才详情页 Page Object 方法和 E2E 测试
-- 2026-01-15: 代码审查 - 修复 expectTalentDetailHeader 验证方法，所有测试通过
 
 ## 参考信息
 

allin-packages/disability-module/src/schemas/person-extension.schema.ts

@@ -296,10 +296,6 @@ export const CompanyPersonDetailSchema = z.object({
     description: '身份证号',
     example: '330102199001011234'
   }),
-  disabilityId: z.string().openapi({
-    description: '残疾证号',
-    example: 'D12345678'
-  }),
   disabilityType: z.string().openapi({
     description: '残疾类型',
     example: '肢体残疾'

allin-packages/disability-module/src/services/disabled-person.service.ts

@@ -1,12 +1,12 @@
 import { GenericCrudService } from '@d8d/shared-crud';
-import { DataSource, Repository, Like, Not, In } from 'typeorm';
+import { Not, In } from 'typeorm';
 import { DisabledPerson } from '../entities/disabled-person.entity';
 import { DisabledBankCard } from '../entities/disabled-bank-card.entity';
 import { DisabledPhoto } from '../entities/disabled-photo.entity';
 import { DisabledRemark } from '../entities/disabled-remark.entity';
 import { DisabledVisit } from '../entities/disabled-visit.entity';
 import { FileService, File } from '@d8d/file-module';
-import { OrderPerson, OrderPersonAsset, EmploymentOrder } from '@d8d/allin-order-module';
+import{ OrderPerson, OrderPersonAsset } from '@d8d/allin-order-module';
 import { WorkStatus, getWorkStatusLabel } from '@d8d/allin-enums';
 
 // 前端专用的工作状态中文标签映射（与mini-ui保持一致）
@@ -104,8 +104,6 @@ export class DisabledPersonService extends GenericCrudService<DisabledPerson> {
       relations: ['bankCards', 'bankCards.bankName', 'bankCards.file', 'bankCards.file.uploadUser', 'photos', 'photos.file', 'photos.file.uploadUser', 'remarks', 'visits']
     });
 
-    if (person && person.photos) {
-    }
 
     return person;
   }
@@ -241,8 +239,6 @@ export class DisabledPersonService extends GenericCrudService<DisabledPerson> {
       relations: ['bankCards', 'photos', 'photos.file', 'remarks', 'visits']
     });
 
-    if (person && person.photos) {
-    }
 
     return person;
   }
@@ -694,7 +690,6 @@ export class DisabledPersonService extends GenericCrudService<DisabledPerson> {
       name: person.name,
       gender: person.gender,
       idCard: person.idCard,
-      disabilityId: person.disabilityId, // 新增：残疾证号
       disabilityType: person.disabilityType,
       disabilityLevel: person.disabilityLevel,
       birthDate: person.birthDate,

mini-ui-packages/yongren-talent-management-ui/src/pages/TalentDetail/TalentDetail.tsx

@@ -2,7 +2,6 @@ import React, { useEffect } from 'react'
 import { View, Text, ScrollView } from '@tarojs/components'
 import Taro from '@tarojs/taro'
 import { useQuery } from '@tanstack/react-query'
-import { YongrenTabBarLayout } from '@d8d/yongren-shared-ui/components/YongrenTabBarLayout'
 import { PageContainer } from '@d8d/mini-shared-ui-components/components/page-container'
 import { Navbar } from '@d8d/mini-shared-ui-components/components/navbar'
 import { enterpriseDisabilityClient } from '../../api'
@@ -14,17 +13,14 @@ export interface TalentDetailProps {
 }
 
 // 从RPC客户端推断类型
-type TalentDetailResponse = InferResponseType<typeof enterpriseDisabilityClient[':id']['$get'], 200>
 type WorkHistoryResponse = InferResponseType<typeof enterpriseDisabilityClient[':id']['work-history']['$get'], 200>
 type SalaryHistoryResponse = InferResponseType<typeof enterpriseDisabilityClient[':id']['salary-history']['$get'], 200>
 type CreditInfoResponse = InferResponseType<typeof enterpriseDisabilityClient[':id']['credit-info']['$get'], 200>
-type VideoResponse = InferResponseType<typeof enterpriseDisabilityClient[':id']['videos']['$get'], 200>
 
 // 提取数组元素类型
 type WorkHistoryItem = WorkHistoryResponse['工作历史'][number]
 type SalaryHistoryItem = SalaryHistoryResponse['薪资历史'][number]
 type CreditInfoItem = CreditInfoResponse['征信信息'][number]
-type VideoItem = VideoResponse['视频列表'][number]
 
 
 const TalentDetail: React.FC<TalentDetailProps> = () => {
@@ -54,7 +50,7 @@ const TalentDetail: React.FC<TalentDetailProps> = () => {
         // 兼容字段映射 - 使用实际存在的字段
         salary: 0, // 薪资字段不存在，使用默认值0
         joinDate: undefined, // 入职日期字段不存在
-        disabilityId: data.disabilityId || '未提供', // 使用后端返回的残疾证号
+        disabilityId: data.disabilityType || data.disabilityLevel || '未提供', // 使用残疾类型或等级
         idAddress: '未提供', // 地址字段不存在
         phone: data.phone || '未提供'
       }
@@ -271,13 +267,6 @@ const TalentDetail: React.FC<TalentDetailProps> = () => {
     return `¥${amount.toLocaleString()}`
   }
 
-  // 脱敏身份证号
-  const maskIdCard = (idCard?: string) => {
-    if (!idCard) return '未提供'
-    if (idCard.length !== 18) return idCard
-    return idCard.replace(/(\d{6})(\d{10})(\d{2})/, '$1********$3')
-  }
-
   return (
       <PageContainer padding={false} className="pb-0">
         <ScrollView
@@ -371,7 +360,7 @@ const TalentDetail: React.FC<TalentDetailProps> = () => {
                     </View>
                     <View className="flex flex-col">
                       <Text className="text-gray-500">身份证号</Text>
-                      <Text className="text-gray-800">{maskIdCard(talentDetail.idCard)}</Text>
+                      <Text className="text-gray-800">{talentDetail.idCard || '未提供'}</Text>
                     </View>
                     <View className="flex flex-col">
                       <Text className="text-gray-500">残疾证号</Text>

web/tests/e2e/pages/mini/enterprise-mini.page.ts

@@ -893,14 +893,9 @@ export class EnterpriseMiniPage {
    */
   async expectTalentDetailHeader(expected: TalentHeaderData): Promise<void> {
     // 验证姓名显示
-    // 注意：Taro 组件在 H5 模式下使用 textContent() 验证，因为 toBeVisible() 可能不可靠
     if (expected.name) {
-      const pageContent = await this.page.textContent('body') || '';
-      const hasName = pageContent.includes(expected.name);
-      if (!hasName) {
-        throw new Error(`人才详情页验证失败: 期望包含人才姓名 "${expected.name}"`);
-      }
-      console.debug(`[人才详情页] 人才姓名 "${expected.name}" 显示正确 ✓`);
+      const nameElement = this.page.getByText(expected.name, { exact: false }).first();
+      await expect(nameElement).toBeVisible({ timeout: TIMEOUTS.ELEMENT_VISIBLE_SHORT });
     }
 
     // 验证残疾类型·等级·状态标签（如果提供）
@@ -1837,14 +1832,14 @@ export class EnterpriseMiniPage {
         console.debug(`[数据统计页] 选择年份: ${year} (索引 ${yearIndex})`);
         await this.page.waitForTimeout(TIMEOUTS.MEDIUM);
         return;
-      } catch (e) {
+      } catch (_e) {
         console.debug(`[数据统计页] selectOption 失败: ${e}`);
       }
     }
 
     /// 方法2: 查找包含年份文本的 View 元素并点击
     // Taro Picker 的触发元素通常包含当前选中的年份文本
-    const yearText = `${year}年`;
+    
     const yearTextElements = this.page.locator('View').filter({
       hasText: /\d{4}年/
     });
@@ -1862,7 +1857,7 @@ export class EnterpriseMiniPage {
         console.debug(`[数据统计页] 选择年份: ${year} (点击后选择)`);
         await this.page.waitForTimeout(TIMEOUTS.MEDIUM);
         return;
-      } catch (e) {
+      } catch (_e) {
         // 忽略错误
       }
     }
@@ -1923,14 +1918,14 @@ export class EnterpriseMiniPage {
         console.debug(`[数据统计页] 选择月份: ${month} (索引 ${monthIndex})`);
         await this.page.waitForTimeout(TIMEOUTS.MEDIUM);
         return;
-      } catch (e) {
+      } catch (_e) {
         console.debug(`[数据统计页] selectOption 失败: ${e}`);
       }
     }
 
     /// 方法2: 查找包含月份文本的 View 元素并点击
     // Taro Picker 的触发元素通常包含当前选中的月份文本
-    const monthText = `${month}月`;
+    
     const monthTextElements = this.page.locator('View').filter({
       hasText: /\d+月/
     });
@@ -1949,7 +1944,7 @@ export class EnterpriseMiniPage {
         console.debug(`[数据统计页] 选择月份: ${month} (点击后选择)`);
         await this.page.waitForTimeout(TIMEOUTS.MEDIUM);
         return;
-      } catch (e) {
+      } catch (_e) {
         // 忽略错误
       }
     }