开玩笑等待 express 的默认错误处理程序

Question

鉴于以下设置：

const express = require("express");
const app = express();

app.get("/", function(req, res, next) {
  // explicitly return an error
  return next("my error");
});

// made this middleware for the example,
// solution should typically also work for express default error handling
app.use(function(error, req, res, next) {
  if (error) {
    res.status(500).send({ error });
    throw new Error(error); // <- how to test this?
  }
  next();
});

app.listen(8080, function() {
  console.log("server running on 8080");
}); //the server object listens on port 8080

对于测试：

const request = require("supertest");
const app = require("../../app.js");

const spy = jest.spyOn(global.console, "error").mockImplementation();

it("throws an error", async done => {
  const res = await request(app).get("/");

  expect(res.status).toBe(500);
  expect(res.error.text).toContain("my error");

  expect(spy).toHaveBeenCalled(); // nothing...

  done();
});

用这个 example code 制作了一个 Codesandbox。不知道如何在其中运行节点测试。

Answer 1

async 不应与done 一起使用，这会导致测试超时，以防无法到达done()。

首先，错误处理程序不应该重新抛出错误，除非它是可重用的路由器实例，应该用另一个处理程序来扩充。如果它是连续的最后一个，它应该捕获其中可能发生的同步和异步错误。

问题是默认错误处理程序是triggered asynchronously，所以应该特别等待：

it("throws an error", async () => {
  const spy = jest.spyOn(global.console, "error");
  const res = await request(app).get("/");

  expect(res.status).toBe(500);
  expect(res.error.text).toContain("my error");
  await new Promise(resolve = > setTimeout(resolve));
  expect(spy).not.toHaveBeenCalled(); // it really shouldn't
});

解决此问题的更正确方法是确保错误得到处理：

it("throws an error", async () => {
  const defaultErrorHandler = jest.fn((err, req, res, next) => {});

  app.use(defaultErrorHandler);
  const res = await request(app).get("/");

  expect(res.status).toBe(500);
  expect(res.error.text).toContain("my error");
  expect(defaultErrorHandler).not.toHaveBeenCalled();
});