0

在下面的代码块（示例 #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()
      })
    })
  })
})
4

1 回答 1

1

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

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

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

于 2020-03-12T17:02:10.670 回答