Question

我有一个令人困惑的不工作测试我无法以某种方式解决（我来自Java背景，所以如果我的术语有些错误，我会提前感到抱歉）。

有一个对象类Trigger，它有一些公共函数，foo()和bar()。逻辑是：

可以直接在另一个对象上调用foo()，或者如果调用bar()（使用数组），则Trigger.bar()会调用Trigger.foo() array.length 次。

使用蓝鸟Promises实现触发器。这是Trigger的基本摘录（我删除了一些逻辑，这看起来有点多余，但实际上发生了更多）：

// Will be refactored afterwards
var self;

var Trigger  = function() {
  self = this;
};

Trigger.prototype.bar = function bar(members) {
  return new Promise(function (resolve, reject) {
    Promise.map(members, function (member) {
     self.foo(member)
     .then(function(res){
       ...
     })
     // catch
    });
  });
};

Trigger.prototype.foo = function foo(member) {
  return new Promise(function(resolve, reject){
    doSomethingWithMember(member)
    .then(function(res){
      ...
    })
    // catch
  });
}

所以我们可以看到， bar()在成员数组中的每个成员上调用foo() 。

我想编写一个测试，其中我监视触发器对象并检查它是否调用foo() n次。

所以我目前做了什么（改变了函数名称），我只是检查它是否被调用：

var chai = require('chai');
var expect = chai.expect;
var sinon = require('sinon');
var sinonChai = require('sinon-chai');

chai.use(require('chai-as-promised'));
chai.use(sinonChai);

var Trigger = require(...);
var trigger = new Trigger();

      ...

  describe("bla", function() {
    it("should invoke 'foo()' n times", function() {
      var spy = sinon.spy(trigger.foo);
      var member = MemberBuilder.buildCorrect();
      trigger.bar(member);
      return expect(spy).to.have.been.called; // does not pass 
    })
  });

由于Trigger的承诺，这显然不起作用，因此告诉我：

AssertionError: expected foo to have been called at least once, but it was never called

但无论我如何在bar()的 then（）块中尝试解决此问题，它总是声称我的间谍没有被调用。

  describe("bla", function() {
    it("should invoke 'foo()' n times", function() {
      var spy = sinon.spy(trigger.foo);
      var member = MemberBuilder.buildCorrect();
      trigger.bar(member)
        .then(function(){
          return expect(spy).to.have.been.called; // does not pass with the same AssertionError (has not been invoked)
      });
    })

我已完成一些日志记录，并确保调用foo()。对我做错了什么想法？

此致

vegaaaa

Answer 1

我发现了我的（愚蠢！）错误。误读API ......

var spy = sinon.spy(trigger.foo);

应该成为

var spy = sinon.spy(trigger, "foo");

所以这就像一个魅力：

describe("bla", function() {
  it("should invoke 'foo()'", function() {
    var spy = sinon.spy(trigger, "foo");
    var member = MemberBuilder.buildCorrect();
    trigger.bar(member)
    .then(function(res) {
      return expect(spy).to.have.been.called;
    })
  })

或者（因为我发现它更容易理解）：

describe("bla", function() {
    it("should invoke 'foo()'", function() {
      sinon.spy(trigger, "foo");
      var member = MemberBuilder.buildCorrect();
      trigger.bar(member)
      .then(function() {
        return expect(trigger.foo).to.have.been.called;
      })
    })

希望这有助于其他人遇到与api相同的问题。

Sinon检查是否使用promise调用了其他对象方法

1 个答案: