首页 > 解决方案 > async/await 是否以不同的方式处理未捕获的拒绝错误?

问题描述

在下面的代码块(示例 #1)中,expect(this.req.headers["user-agent"]).to.equal("BOOM")抛出错误并且测试失败。

describe("http.get with headers", () => {
  it("should return response with status code 200", async () => {
    const userAgent =
      "Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:74.0) Gecko/20100101 Firefox/74.0"
    nock("https://api.example.net")
      .get("/hello")
      .reply(function() {
        expect(this.req.headers["user-agent"]).to.equal("BOOM")
        return [200]
      })
    const httpInstance = new http({
      headers: {
        "user-agent": userAgent,
      },
    })
    let response = await httpInstance.get("https://api.example.net/hello")
    expect(response.statusCode).to.equal(200)
  })
})

在以下代码块(示例 #2)中,expect(requestBody.message).to.equal("BOOM")抛出“静默”错误(return [200]从不执行),但测试通过。为什么?

describe("logger.captureMessage(message, callback)", () => {
  it("should send captured exception to sentry", () => {
    return new Promise((resolve, reject) => {
      nock("https://sentry.io")
        .post("/api/3926156/store/")
        .reply((uri, requestBody: any) => {
          expect(requestBody.message).to.equal("BOOM")
          return [200]
        })
      logger.captureMessage("foo", () => {
        resolve()
      })
    })
  })
})

使用catch和触发reject是可行的,但是当示例 #1 中一切正常时,为什么有必要这样做呢?

describe("logger.captureMessage(message, callback)", () => {
  it("should send captured exception to sentry", () => {
    return new Promise((resolve, reject) => {
      nock("https://sentry.io")
        .post("/api/3926156/store/")
        .reply((uri, requestBody: any) => {
          try {
            expect(requestBody.message).to.equal("BOOM")
            return [200]
          } catch (error) {
            reject(error)
          }
        })
      logger.captureMessage("foo", () => {
        resolve()
      })
    })
  })
})

标签: node.jshapijsnocklab

解决方案


reply不建议在 Nock 回调中进行断言。在执行该函数的位置和您的测试运行程序之间有很多层。最值得注意的是,提出请求的客户。通常客户端不能很好地处理非请求错误并最终隐藏甚至掩盖错误。

没有一种“正确”的方法可以做到这一点。从您的第二个测试来看,我建议使用测试级别变量来存储请求的正文。然后在captureMessage回调中进行断言。

另一个注意事项是,您在第二次测试的 Promise 中没有正确处理冒泡错误。此行:return new Promise((resolve, reject) => {不使用async关键字,这意味着您必须reject手动调用,否则任何冒泡错误都将丢失。


推荐阅读