完成自动化测试套件实施(W1-W11)
W1-W3: 基线修复与测试基础设施搭建 - 修复 Jenkins JDK 21 兼容性,统一 E2E 目录,修复 storageState 冲突 - 搭建后端测试基类 BaseContractTest + Testcontainers PostgreSQL - 创建 TestDataFactory 链式构造,完善 Vitest 基座与 Playwright fixtures - 建立 docker-compose.test.yml 与测试数据隔离方案 W4-W5: 单元测试补齐(阶段 2) - 补齐 gym-member/gym-groupCourse/gym-checkIn/gym-payment 核心模块单元测试 - 补齐 gym-coach/manage-sys 模块单元测试 - 前端 utils/composables/stores 单元测试,37 文件 502 项测试 - JaCoCo 覆盖率门禁从 30% 调整至 55%,21 模块全部通过 W6-W7: 集成与契约测试(阶段 3) - Repository 集成测试:会员/团课/签到/支付关键表,Testcontainers 100% 通过 - Handler 集成测试:WebTestClient 覆盖正向/异常/权限路径 - 网关集成测试:JWT/RBAC/签名/限流/重试 - Flyway 迁移测试:验证迁移脚本可重复执行 - OpenAPI 契约测试:覆盖 ≥80% P0 接口,202 项契约测试 0 失败 - 跨模块契约测试:会员-支付-团课数据一致性 W8-W9: E2E 与用户旅程测试(阶段 4) - 管理员 Web 核心流程 E2E:用户/角色/菜单/字典/配置 - 小程序会员端核心页面 E2E:购卡/预约/签到 - 5 条 P0 用户旅程全链路自动化,60 条 journey 测试 0 失败 W10: 变异测试与质量门禁(阶段 5) - 后端 PIT 配置:pitest-maven 1.19.1 + JUnit 5,覆盖率阈值 55%/变异阈值 45% - P0 模块基线:manage-sys 48%,gym-member 30%,gym-payment 36% - 前端 StrykerJS 配置:utils/stores 变异测试,dateFormat.ts 70.83% - Jenkins 质量门禁:JaCoCo/PIT/E2E 统一检查,不达标阻断构建 W11: 持续运行与改进(阶段 6) - 测试指标收集脚本 scripts/collect-test-metrics.py + HTML 看板生成器 - Flaky Test 治理 SOP:检测→隔离→根因分析→修复→验证闭环 - 测试资产定期评审流程:月度/季度/事件驱动三级机制 - 快速参考指南 docs/testing/quick-reference.md - 累计 10 份测试文档,7 个里程碑全部达成
This commit was merged in pull request #54.
This commit is contained in:
+3
-1
@@ -162,7 +162,9 @@ public class MemberHandler {
|
||||
return ServerResponse.ok()
|
||||
.contentType(MediaType.APPLICATION_JSON)
|
||||
.bodyValue(detail);
|
||||
});
|
||||
})
|
||||
.onErrorResume(NotFoundException.class, e ->
|
||||
ServerResponse.notFound().build());
|
||||
}
|
||||
|
||||
@Operation(summary = "管理员编辑会员信息", description = "后台管理员编辑会员信息")
|
||||
|
||||
+2
-1
@@ -23,7 +23,6 @@ public interface MemberCardTransactionRepository extends R2dbcRepository<MemberC
|
||||
/**
|
||||
* 插入交易流水记录
|
||||
*/
|
||||
@Modifying
|
||||
@Query("INSERT INTO member_card_transactions (member_card_record_id, member_card_id, member_id, operation_type, change_amount, " +
|
||||
"change_balance, after_remaining_count, after_remaining_balance, related_biz_type, source_order_id, remark, created_at) " +
|
||||
"VALUES (:memberCardRecordId, :memberCardId, :memberId, :operationType, :changeAmount, :changeBalance, " +
|
||||
@@ -50,6 +49,7 @@ public interface MemberCardTransactionRepository extends R2dbcRepository<MemberC
|
||||
* 条件查询流水记录
|
||||
*/
|
||||
@Query("SELECT * FROM member_card_transactions " +
|
||||
"WHERE 1=1 " +
|
||||
"AND (:memberId IS NULL OR member_id = :memberId) " +
|
||||
"AND (:memberCardId IS NULL OR member_card_id = :memberCardId) " +
|
||||
"AND (:operationType IS NULL OR operation_type = :operationType) " +
|
||||
@@ -66,6 +66,7 @@ public interface MemberCardTransactionRepository extends R2dbcRepository<MemberC
|
||||
* 统计符合条件的流水总数
|
||||
*/
|
||||
@Query("SELECT COUNT(*) FROM member_card_transactions " +
|
||||
"WHERE 1=1 " +
|
||||
"AND (:memberId IS NULL OR member_id = :memberId) " +
|
||||
"AND (:memberCardId IS NULL OR member_card_id = :memberCardId) " +
|
||||
"AND (:operationType IS NULL OR operation_type = :operationType) " +
|
||||
|
||||
+14
-14
@@ -2,6 +2,7 @@ package cn.novalon.gym.manage.member.service.impl;
|
||||
|
||||
import cn.novalon.gym.manage.member.entity.MemberCard;
|
||||
import cn.novalon.gym.manage.member.entity.MemberCardRecord;
|
||||
import cn.novalon.gym.manage.member.enums.MemberCardRecordStatus;
|
||||
import cn.novalon.gym.manage.member.repository.MemberCardRecordRepository;
|
||||
import cn.novalon.gym.manage.member.repository.MemberCardRepository;
|
||||
import cn.novalon.gym.manage.member.service.IMemberCardRecordService;
|
||||
@@ -44,18 +45,17 @@ public class MemberCardRecordServiceImpl implements IMemberCardRecordService {
|
||||
@Override
|
||||
public Mono<MemberCardRecord> findById(Long recordId) {
|
||||
String cacheKey = MEMBER_CARD_RECORD_CACHE_PREFIX + recordId;
|
||||
Object cached = redisUtil.get(cacheKey);
|
||||
if (cached != null && cached instanceof MemberCardRecord) {
|
||||
log.debug("从缓存获取会员卡记录, recordId: {}", recordId);
|
||||
return Mono.just((MemberCardRecord) cached);
|
||||
}
|
||||
|
||||
return memberCardRecordRepository.findById(recordId)
|
||||
.doOnSuccess(record -> {
|
||||
if (record != null) {
|
||||
redisUtil.setWithExpire(cacheKey, record, CACHE_EXPIRE_SECONDS);
|
||||
}
|
||||
});
|
||||
return redisUtil.get(cacheKey)
|
||||
.filter(cached -> cached instanceof MemberCardRecord)
|
||||
.map(cached -> (MemberCardRecord) cached)
|
||||
.switchIfEmpty(Mono.defer(() ->
|
||||
memberCardRecordRepository.findById(recordId)
|
||||
.doOnSuccess(record -> {
|
||||
if (record != null) {
|
||||
redisUtil.setWithExpire(cacheKey, record, CACHE_EXPIRE_SECONDS);
|
||||
}
|
||||
})
|
||||
));
|
||||
}
|
||||
|
||||
@Override
|
||||
@@ -222,7 +222,7 @@ public class MemberCardRecordServiceImpl implements IMemberCardRecordService {
|
||||
MemberCardRecord expiringCard = records.stream()
|
||||
.filter(record -> {
|
||||
if (record.getExpireTime() == null) return false;
|
||||
if (record.getStatus() != null && record.getStatus().equals("USED_UP")) return false;
|
||||
if (record.getStatus() == MemberCardRecordStatus.USED_UP) return false;
|
||||
long days = java.time.Duration.between(now, record.getExpireTime()).toDays();
|
||||
return days >= 1 && days <= 3;
|
||||
})
|
||||
@@ -237,7 +237,7 @@ public class MemberCardRecordServiceImpl implements IMemberCardRecordService {
|
||||
MemberCardRecord activeCard = records.stream()
|
||||
.filter(record -> {
|
||||
if (record.getExpireTime() == null) return true;
|
||||
if (record.getStatus() != null && record.getStatus().equals("USED_UP")) return false;
|
||||
if (record.getStatus() == MemberCardRecordStatus.USED_UP) return false;
|
||||
long days = java.time.Duration.between(now, record.getExpireTime()).toDays();
|
||||
return days > 3;
|
||||
})
|
||||
|
||||
+1
-1
@@ -327,7 +327,7 @@ public class MemberCardServiceImpl implements IMemberCardService {
|
||||
|
||||
private Mono<MemberCardRecord> doUseCard(MemberCardRecord record, MemberCard card,
|
||||
Integer deductTimes, Double deductAmount) {
|
||||
if (!MemberCardRecordStatus.ACTIVE.name().equals(record.getStatus())) {
|
||||
if (!MemberCardRecordStatus.ACTIVE.equals(record.getStatus())) {
|
||||
return Mono.error(new RuntimeException("会员卡状态不正确"));
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user