Jasmine隔离测试无法覆盖forEach循环内代码的问题

问题表现 #

Angular单元测试编写完成后，代码覆盖率报告显示getLights方法内的forEach循环逻辑始终未被执行，无法覆盖对应分支。

关联代码 #

组件侧代码 #

beforeEach(() => {
  // 省略其他初始化逻辑
  let lightRes= [
    {
      children: [{Id: 10, name: "test"}],
      startDate: "",
      description: "",
      id: 111,
      title: "test for light",
    },
  ];

  const res = {data: lightRes};

  component.lightService.lightDetail.and.returnValue(
    scheduled([res], asyncScheduler)
  );
});

private getLights(id: string): void {
  const sus = this.lightService.lightDetail(id).subscribe((light) => {
    this.light = new Light(light.data);
    this.light.children.forEach((val) =>
      this.clonedArray.push(Object.assign({}, val))
    );
    this.light.fathers = light.data.fathers;
  });
  this.suscripciones.add(sus);
}

原测试用例代码 #

it("getLights", fakeAsync(() => {
  component.getLight();
  res.data[0].children.forEach((val) => {});
  tick();
}));

覆盖率报告 #

问题根因 #

• 方法名拼写错误：组件内定义的方法名为getLights（复数），测试用例调用的是getLight()（单数），目标方法根本没有被触发。
• 方法传参缺失：getLights要求传入id字符串入参，原有调用没有传参，不符合方法签名。
• 无效冗余代码：用例中写的res.data[0].children.forEach(...)完全无效：res是beforeEach内的局部变量，用例作用域无法访问；且这段空循环和组件内部逻辑没有任何关联，不可能触发组件内的forEach执行。
• 无结果断言：用例执行后没有校验clonedArray的赋值结果，无法验证逻辑是否生效。
• 潜在构造函数问题：如果Light类的构造函数没有正确将入参的children属性挂载到实例上，会导致this.light.children为undefined，forEach无法执行。

修复方案 #

1. 修正测试用例逻辑，保证方法调用正确、异步流程执行完成、断言覆盖forEach执行结果：

it("getLights", fakeAsync(() => {
  // 初始化目标数组
  component.clonedArray = [];
  // 正确传参、调用拼写正确的方法名
  component.getLights('test-111');
  // 触发异步队列执行，等待subscribe回调完成
  tick();
  // 断言forEach执行结果，验证逻辑生效
  expect(component.clonedArray.length).withContext('clonedArray应被push1条子项').toBe(1);
  expect(component.clonedArray[0].Id).toBe(10);
  expect(component.light).toBeInstanceOf(Light);
}));

2. 检查Light类构造函数逻辑，保证children属性正确初始化，参考实现：

class Light {
  children: any[] = [];
  fathers: any;
  constructor(sourceData: any[]) {
    // 取第一条数据的children赋值，保证children属性始终为数组
    this.children = sourceData[0]?.children ?? [];
    // 其余属性赋值逻辑
  }
}

3. 提前确认测试Bed中注入的lightService是正确创建的spy对象，保证mock返回值生效：

// beforeEach内TestBed配置参考
const lightServiceSpy = jasmine.createSpyObj('LightService', ['lightDetail']);
await TestBed.configureTestingModule({
  declarations: [YourComponent],
  providers: [
    { provide: LightService, useValue: lightServiceSpy }
  ]
}).compileComponents();

内容的提问来源于stack exchange，提问作者Ivan Martin Blanco