sinon否认尽管有存根被调用

时间:2019-03-05 18:52:17

标签: node.js unit-testing express sinon

我正在使用sinon对函数res.status进行存根,在下面的两个单元测试中都调用res.status,一次通过而另一次失败,我什至调试并逐行查看了函数是否被调用但是sinon失败,并说未调用该函数。

这是我正在测试的控制器(ActivityController.js)

exports.findActivity = (req, res) => {
  ActivityService.findActivity(req.params.id, req.user).then(result => {
    res.status(200).json({result: result})
  }).catch(err => {
    res.status(500).json({msg: err.message})
  })
}

这是单元测试,第一个测试通过了,但是第二个测试没有通过

    it('should return Activity obj with status 200', async () => {
      expectedResult = activity
      sinon.stub(ActivityService, 'findActivity').resolves(expectedResult)
      await ActivityController.findActivity(req, res)
      sinon.assert.calledWith(ActivityService.findActivity)
      expect(res.status.calledOnce).to.equal(true)
      sinon.assert.calledWith(res.status, 200)
    })
    it('should return status 500 on server error', async () => {
      sinon.stub(ActivityService, 'findActivity').rejects()
      await ActivityController.findActivity(req, res)
      sinon.assert.calledWith(ActivityService.findActivity, req.params.id, req.user)
      expect(res.status.calledOnce).to.equal(true)
      sinon.assert.calledWith(res.status, 500)
    })

我调试了第二个单元测试,看到ActivityController通过catch并调用res.status(500),但是单元测试在res.status.calledOnce中失败

2 个答案:

答案 0 :(得分:0)

您是否在每次测试之间重置res.status存根? res.calledOnce可能不正确,因为它可能被多次叫“ 更多”。

答案 1 :(得分:0)

似乎expect(res.status.calledOnce).to.equal(true)在等待sinon.assert.calledWith之前被调用了。

await之前放入sinon.assert可以解决问题。

所以正确的代码是

  it('should return status 500 on server error', async () => {
      sinon.stub(ActivityService, 'findActivity').rejects()
      await ActivityController.findActivity(req, res)
      await sinon.assert.calledWith(ActivityService.findActivity, req.params.id, req.user)
      expect(res.status.calledOnce).to.equal(true)
      sinon.assert.calledWith(res.status, 500)
    })