Angular表单提交测试相关技术问题咨询

1. 第一个测试中，new Paciente()是否需要传入参数？ #

这取决于你的测试目标：

• 如果你的测试只是想验证save()的分支逻辑（当实体无id时调用create服务），那么不传参数完全没问题——只要entity.id是undefined，就会走创建分支，当前的断言expect(service.create).toHaveBeenCalledWith(entity)也能验证分支正确性。
• 但如果你的测试目的是验证save()确实把患者的姓名、性别等数据正确传递给了服务，那你必须传入具体参数。比如：

  const entity = new Paciente(undefined, 'Maria Garcia', Sexo.FEMENINO);
  

  这样后续的toHaveBeenCalledWith(entity)才能确认服务接收到的是完整的实体数据，而不是一个空对象。当前你的测试只验证了调用行为，没验证数据传递的准确性，若要覆盖这部分需求，就需要补充参数。

2. 第二个测试用toHaveBeenCalledTimes(0)是否合理？ #

你说得太对了——这个写法完全不符合测试目的！

当前代码的逻辑是：点击按钮后断言save()一次都没被调用，这和“验证按钮点击触发save()”的目标完全相反。正确的做法应该是：

it ('should call save() when submit button is clicked', fakeAsync(() => {
  spyOn(comp,'save');
  // 确保选中的是submit按钮，避免选到其他按钮
  const submitBtn = debugElement.query(By.css('button[type="submit"]')).nativeElement;
  submitBtn.click();
  // 触发变更检测，处理Angular的事件绑定
  fixture.detectChanges();
  tick();
  // 断言save被调用至少一次
  expect(comp.save).toHaveBeenCalled();
  // 或者更精确地断言调用次数
  // expect(comp.save).toHaveBeenCalledTimes(1);
}));

另外要注意：如果你的按钮不是type="submit"，表单的(ngSubmit)可能不会触发，所以一定要确保选择器指向正确的提交按钮。

3. 当前3个测试是否覆盖了表单提交的完整需求？ #

远远不够。这三个测试只是覆盖了save()方法的最基础逻辑，表单提交的完整测试还需要补充以下关键场景：

• 表单验证测试：比如当必填字段nombre为空时，点击提交是否不会触发save()？需要验证表单无效时的拦截逻辑。
• 数据绑定验证：在输入框中输入内容后，检查comp.paciente.nombre是否同步更新，确保ngModel绑定正常工作。
• update分支测试：当前只测了新实体的创建分支，还需要测试当paciente.id存在时，save()是否调用update服务。
• 异步状态完整流程：当前第三个测试只验证了isSaving变为true，但还需要测试请求成功/失败后，isSaving是否重置为false。
• 错误处理测试：模拟服务返回错误，验证组件是否正确处理（比如不关闭模态框、显示错误提示等）。
• 表单提交的完整链路：从用户输入、表单验证通过、到调用服务、再到UI状态更新（比如按钮禁用、加载提示）的全流程测试。

当前的测试只是一个起点，补充上述场景后才能真正覆盖Angular表单提交的核心需求。

内容的提问来源于stack exchange，提问作者Tester