promise返回后Sinon存根未调用

Question

我试图用Mocha、Chai和Sinon为我的代码添加更多的测试，但是我很难理解为什么第二个存根函数没有被识别为被调用。

我有一个向用户发送电子邮件的功能（稍后我将测试电子邮件功能-现在我只想处理我控制的存根依赖项）

// EmailSender.js
const models = require('../models');
const User = models.user;
const emailLogger = require('./emailLogger');

class EmailSender {
  constructor(subject, emailData) {
    this.subject = subject;
    this.emailData = emailData;
  }

  sendToUser() {
    let email = this.emailData.email;

    User.findOne({ where: { $or: [
      { email: email },
      { workEmail: email },
    ] } })
    .then(function (userData) {
      if (userData) {
        emailLogger.log('Send to anon - sending to user ' + userData.id);
      });
    } else {
      emailLogger.log('Send to anon - no user found');
    }
  }
}

和测试文件：

const EmailSender = require('../../../helpers/emailSender');
const models = require('../../../models');
const User = models.user;
const emailLogger = require('../../../helpers/emailLogger');
const chai = require("chai");
const sinon = require('sinon');
const sinonChai = require("sinon-chai");

const expect = chai.expect;
chai.use(sinonChai);

describe('The emailSender', () => {
  let emailData;

  beforeEach(() => {
    emailData = {
      email: '[e-mail]'
    };
    sinon.stub(User, 'findOne').returns(Promise.resolve());
    sinon.stub(emailLogger, 'log');
  })

  afterEach(() => {
    User.findOne.restore();
    emailLogger.log.restore();
  })

  describe('sendToUser method', () => {
    it('logs an email if a user is found', () => {
      let emailSender = new EmailSender('Email subject', emailData);
      emailSender.sendToUser();

      expect(User.findOne).to.have.been.calledOnce; // works
      expect(emailLogger.log).to.have.been.calledOnce; // doesn't
    })
  })
});

我可以用Sinon存根User.findOne（）方法，但是当我尝试存根emailLogger.log（）方法时，我遇到了麻烦。它似乎调用了存根，而不是真正的方法，而是期望（emailLogger.log）。to.have.been.called一旦返回false。

我尝试添加do（）和一个假计时器以防出现延迟问题，以及一系列其他事情，但到目前为止还没有运气。

Answer 1

您永远不会从sendToUser函数返回promise，这意味着无法知道它实际何时完成。当您以同步方式测试async函数时，这意味着您在代码中调用emailLogger.log之前询问它是否已被调用！

你需要回报promise，然后你就可以按照邓肯在回答中的建议去做。

Answer 2

一个很好的技巧是从测试函数返回promise，这会导致Mocha等待promise完成。以下是您的操作方法：

it('logs an email if a user is found', () => {
   const emailSender = new EmailSender('Email subject', emailData);
   return emailSender.sendToUser().then( () => {  
      //check after the sendToUser promise is complete, but before the test is done  
      expect(User.findOne).to.have.been.calledOnce; 
      expect(emailLogger.log).to.have.been.calledOnce; 
   });
 });

这有一个额外的好处，如果promise因任何原因失败，测试将失败（错误正确）。