使用Mocha+Chai测试Express路由时出现Can't set headers错误

你遇到的这个UnhandledPromiseRejectionWarning虽然不影响测试通过，但其实是两个问题叠加导致的：一是测试代码的Promise处理不够规范，二是你的Express路由大概率存在重复发送响应的情况（也就是错误提示里的Can't set headers after they are sent）。咱们一步步来解决：

一、先解决路由的核心问题 #

错误提示里的Can't set headers after they are sent几乎可以肯定是你的路由处理函数里多次调用了响应方法（比如res.send()、res.json()、res.status().send()）。举个典型的错误例子：

// 错误示例：没有在发送响应后终止函数执行
app.post('/api/allyears/getBoxPlotAttributes', (req, res) => {
  if (!req.body.gaugeId || !req.body.classId) {
    res.status(400).send('Missing required parameters');
  }
  // 这里没有return，后续代码继续执行，又发送了一次响应
  res.json({ data: 'some data' });
});

解决方法很简单：在发送错误响应后加上return，确保后续代码不会再执行：

// 正确示例
app.post('/api/allyears/getBoxPlotAttributes', (req, res) => {
  if (!req.body.gaugeId || !req.body.classId) {
    // 加上return终止函数
    return res.status(400).send('Missing required parameters');
  }
  res.json({ data: 'some data' });
});

这个是警告的根源——路由重复发送响应，导致Express尝试修改已经发送过的响应头，进而触发Promise拒绝，被Mocha检测到。

二、优化测试代码的Promise处理 #

你的测试用了.catch()来捕获请求错误，但await和.catch()结合时，如果没有正确处理整个Promise链，Mocha可能会认为有未处理的拒绝。推荐用两种更规范的写法：

写法一：try/catch包裹（更直观） #

describe.only('allYears services', () => {
  beforeEach(async () => {
    await db.sequelize.sync({ force: true, logging: false });
  });

  it('should respond with 400 when missing gaugeId or classId', async () => {
    try {
      // 如果请求成功（没返回400），就主动标记测试失败
      await chai.request(app)
        .post('/api/allyears/getBoxPlotAttributes')
        .send({ metric: 'average' });
      assert.fail('Expected 400 response but got success');
    } catch (err) {
      assert.equal(err.response.status, 400);
    }
  });

  it('should respond with 400 when missing metric', async () => {
    try {
      await chai.request(app)
        .post('/api/allyears/getBoxPlotAttributes')
        .send({ gaugeId: 123456 });
      assert.fail('Expected 400 response but got success');
    } catch (err) {
      assert.equal(err.response.status, 400);
    }
  });
});

写法二：直接捕获错误后断言（更简洁） #

const { expect } = chai;

describe.only('allYears services', () => {
  beforeEach(async () => {
    await db.sequelize.sync({ force: true, logging: false });
  });

  it('should respond with 400 when missing gaugeId or classId', async () => {
    // 捕获请求的错误，直接取响应对象
    const response = await chai.request(app)
      .post('/api/allyears/getBoxPlotAttributes')
      .send({ metric: 'average' })
      .catch(err => err.response);
    expect(response.status).to.equal(400);
  });

  it('should respond with 400 when missing metric', async () => {
    const response = await chai.request(app)
      .post('/api/allyears/getBoxPlotAttributes')
      .send({ gaugeId: 123456 })
      .catch(err => err.response);
    expect(response.status).to.equal(400);
  });
});

这样做能确保所有Promise的状态都被Mocha正确识别，不会再出现未处理的Promise拒绝警告。

最后补充 #

如果修复后还有警告，可以在运行Mocha时加上--exit参数（比如mocha --exit），确保Mocha在所有测试完成后正确退出，避免残留的异步操作导致警告。

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