测试审查

这一课你将能做到

1. 识别出「为了覆盖率而写」的形式主义测试，并给出改进方案
2. 审查单元测试、集成测试的边界是否合理（什么该 mock，什么不该 mock）
3. 用「变异测试思维」判断测试是否真的能抓住 bug

为什么要学

覆盖率 80% 不代表代码可靠——很多团队 80% 覆盖率的代码，线上事故一点不少。因为测试的价值不在「跑过」，而在「能不能抓住回归」。

人话：写测试就像给代码买保险。垃圾测试 = 买了一份「只保陨石砸中」的保险——保费交了，真出事一毛钱赔不了。

先建立图像

测试金字塔（本站推荐比例）
        ╱ E2E   ╲       ← 极少（5%），跑核心流程
       ╱ 集成测试 ╲      ← 适量（25%），起真 MySQL/Redis
      ╱  单元测试  ╲     ← 大量（70%），纯逻辑、毫秒级

审查关注点：
1. 每层该测什么、不该测什么
2. 断言是否真的验证了行为（而非只是「跑过不抛异常」）
3. 测试之间是否独立（顺序无关、无共享状态）
4. 测试失败时能否定位问题

跟着做

步骤 1：识别「假测试」

❌ 反例：无断言、或断言形同虚设

@Test
void testEnroll() {
    courseService.enroll(1L, 100L);
    // 没断言！只要不抛异常就算过
}

@Test
void testGetCourse() {
    CourseVO vo = courseService.get(1L);
    assertNotNull(vo);   // 只断言非 null，字段对不对不管
}

@Test
void testListCourses() {
    List<CourseVO> list = courseService.list();
    System.out.println(list);   // 用眼睛看？CI 里谁看？
}

✅ 正例：断言行为与状态

@Test
void enroll_shouldDecreaseQuota_whenQuotaAvailable() {
    // given
    Long courseId = 1L;
    Long userId = 100L;
    Course before = courseMapper.selectById(courseId);
    int quotaBefore = before.getQuota();

    // when
    courseService.enroll(courseId, userId);

    // then
    Course after = courseMapper.selectById(courseId);
    assertEquals(quotaBefore - 1, after.getQuota());

    Enroll record = enrollMapper.selectOne(courseId, userId);
    assertNotNull(record);
    assertEquals(EnrollStatus.ACTIVE, record.getStatus());
}

@Test
void enroll_shouldThrow_whenQuotaExhausted() {
    // given: 名额已满的课程
    Long courseId = 999L;

    // when & then
    BizException ex = assertThrows(BizException.class,
            () -> courseService.enroll(courseId, 100L));
    assertEquals(ErrorCode.QUOTA_EXHAUSTED, ex.getCode());
}

步骤 2：审 mock 的边界

❌ 反例：过度 mock，测试变成「验证 mock 自己被调了」

@Test
void testGetCourse() {
    CourseMapper mockMapper = mock(CourseMapper.class);
    Course fakeCourse = new Course();
    fakeCourse.setTitle("测试课");
    when(mockMapper.selectById(1L)).thenReturn(fakeCourse);

    CourseService service = new CourseService(mockMapper);
    CourseVO result = service.get(1L);

    assertEquals("测试课", result.getTitle());
    verify(mockMapper).selectById(1L);   // 验证 mock 被调？这不废话吗
}