为何Sinon测试超时而非失败？如何优化测试代码？

你遇到的超时问题，核心原因是测试中混合了done回调和Promise异步逻辑，但没有正确处理错误传递——并不是必须要用try/catch，换一种更简洁的写法就能解决。

问题根源分析 #

当你在测试里用了done回调，但没有捕获异步函数（router）或断言抛出的错误时，这些错误会被Promise的“静默失败”特性吞噬，导致done()永远不会被调用，测试框架就会触发超时错误。而用try/catch时，你手动捕获了错误并通知了测试框架，所以能正常报错。

更简洁的写法（无需try/catch） #

最推荐的方式是放弃done回调，直接让测试函数返回Promise（或者写成async函数并await被测逻辑），测试框架会自动处理Promise的状态，包括捕获断言失败的错误。

举个例子（以Jest为例，其他测试框架逻辑一致）：

// 测试代码（无需try/catch）
it('should call publish with "requestObject"', async () => {
  // 模拟publish函数
  const mockPublish = jest.fn();
  // 等待router执行完成
  await router(mockPublish);
  // 断言
  expect(mockPublish).toHaveBeenCalledWith('requestObject');
});

如果断言失败（比如你写成wrongValue），async函数会自动抛出错误，测试框架会立即捕获并标记测试失败，不会触发超时。

如果你必须使用done回调（不推荐） #

如果因为某些原因一定要用done，可以通过Promise的.catch()方法把错误传递给done，这样也不需要try/catch：

it('should call publish with correct value', (done) => {
  const mockPublish = jest.fn();
  router(mockPublish)
    .then(() => {
      expect(mockPublish).toHaveBeenCalledWith('requestObject');
      done();
    })
    .catch(done); // 任何错误（包括断言失败）都会传给done，终止测试
});

关键注意点 #

• 不要同时使用done回调和返回Promise/async函数，这会导致测试框架混淆，容易触发超时。
• 你的被测代码并没有“吞错误”的问题——问题出在测试代码的异步处理方式上，而非被测逻辑。

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