QUnit、Sinon.js 和 Backbone 单元测试失败：sinon spy 似乎无法检测到 Backbone 模型事件回调

Question

在下面的单元测试代码中：

TestModel = Backbone.Model.extend({
    defaults: {
        'selection': null
    },
    initialize: function() {
      this.on('change:selection', this.doSomething);
    },
    doSomething: function() {
        console.log("Something has been done.");
    }
});

module("Test", {
    setup: function() {
        this.testModel = new TestModel();
    }
});

test("intra-model event bindings", function() {
    this.spy(this.testModel, 'doSomething');
    ok(!this.testModel.doSomething.called);
    this.testModel.doSomething();
    ok(this.testModel.doSomething.calledOnce);
    this.testModel.set('selection','something new');
    ok(this.testModel.doSomething.calledTwice); //this test should past, but fails.  Console shows two "Something has been done" logs.
});

第三个 ok 失败，即使该函数是从主干事件绑定中有效调用的，正如控制台演示的那样。

这非常令人沮丧，并动摇了我对 sinon.js 是否适合测试我的主干应用程序的信心。我做错了什么，或者这是 sinon 如何检测是否已调用某事的问题？有解决办法吗？

编辑：这是我的具体示例的解决方案，基于已接受答案的猴子补丁方法。虽然它在测试本身中有几行额外的设置代码，（我不再需要模块功能）它完成了工作。谢谢mu is too short

test("intra-model event bindings", function() {
    var that = this;
    var init = TestModel.prototype.initialize;
    TestModel.prototype.initialize = function() {
        that.spy(this, 'doSomething');
        init.call(this);
    };

    this.testModel = new TestModel();
    . . . // tests pass!
}); 

Answer 1

调用this.spy(this.testModel, 'doSomething') 将testModel.doSomething 方法替换为new wrapper method：

var spy = sinon.spy(object, "method");

为object.method 创建一个spy，并用spy 替换原来的方法。

所以this.spy(this.testModel, 'doSomething') 正在有效地做这样的事情：

var m = this.testModel.doSomething;
this.testModel.doSomething = function() {
    // Spying stuff goes here...
    return m.apply(this, arguments);
};

这意味着当你在initialize中绑定事件处理程序时，testModel.doSomething是一个不同的函数：

this.bind('change:selection', this.doSomething);

比你附上你的间谍后的情况。 Backbone 事件调度程序将调用原始的doSomething 方法，但该方法没有Sinon 检测。当您手动调用doSomething 时，您调用的是spy 添加的新函数，并且该函数确实具有Sinon 检测。

如果您想使用 Sinon 测试您的 Backbone 事件，那么您必须在绑定任何事件处理程序之前安排将 Sinon spy 调用应用于模型，这可能意味着挂钩到 ​​initialize。

也许您可以对模型的 initialize 进行猴子修补，以在绑定任何事件处理程序之前添加必要的 spy 调用：

var init = Model.prototype.initialize;
Model.prototype.initialize = function() {
    // Set up the Spy stuff...
    init.apply(this, arguments);
};

演示：http://jsfiddle.net/ambiguous/C4fnX/1/

您也可以尝试使用以下方式子类化您的模型：

var Model = Backbone.Model.extend({});
var TestModel = Model.extend({
    initialize: function() {
        // Set up the Spy stuff...
        Model.prototype.initialize.apply(this, arguments);
    }
});

然后使用TestModel 代替模型，这将为您提供TestModel 中Model 的插桩版本，而无需在您的正常生产就绪Model 中包含一堆特定于测试的代码。缺点是使用Model 的任何其他东西都需要被子类化/修补/...才能使用TestModel。

演示：http://jsfiddle.net/ambiguous/yH3FE/1/

您可以通过以下方式解决TestModel 问题：

var OriginalModel = Model;
Model = Model.extend({
    initialize: function() {
        // Set up the Spy stuff...
        OriginalModel.prototype.initialize.apply(this, arguments);
    }
});

但您必须正确订购以确保每个人都使用新的Model 而不是旧的。

演示：http://jsfiddle.net/ambiguous/u3vgF/1/