我有一些紧密耦合的遗留代码,我想用测试来覆盖.有时确保在另一个方法之前调用一个模拟方法很重要.一个简化的例子：
function PageManager(page) {
this.page = page;
}
PageManager.prototype.openSettings = function(){
this.page.open();
this.page.setTitle("Settings");
};
在测试中,我可以检查open()和setTitle()是否都被调用：
describe("PageManager.openSettings()",function() {
beforeEach(function() {
this.page = jasmine.createSpyObj("MockPage",["open","setTitle"]);
this.manager = new PageManager(this.page);
this.manager.openSettings();
});
it("opens page",function() {
expect(this.page.open).toHaveBeenCalledWith();
});
it("sets page title to 'Settings'",function() {
expect(this.page.setTitle).toHaveBeenCalledWith("Settings");
});
});
但是setTitle()只能在第一次调用open()之后才能工作.我想检查第一个page.open()是否被调用,然后是setTitle().我想写这样的东西：
it("opens page before setting title",function() {
expect(this.page.open).toHaveBeenCalledBefore(this.page.setTitle);
});
但Jasmine似乎并没有内置这样的功能.
我可以破解这样的东西：
beforeEach(function() {
this.page = jasmine.createSpyObj("MockPage","setTitle"]);
this.manager = new PageManager(this.page);
// track the order of methods called
this.calls = [];
this.page.open.and.callFake(function() {
this.calls.push("open");
}.bind(this));
this.page.setTitle.and.callFake(function() {
this.calls.push("setTitle");
}.bind(this));
this.manager.openSettings();
});
it("opens page before setting title",function() {
expect(this.calls).toEqual(["open","setTitle"]);
});
这有效,但我想知道是否有一些更简单的方法来实现这一点.或者一些很好的方法来概括这一点,所以我不需要在其他测试中复制此代码.
PS.当然,正确的方法是重构代码以消除这种时间耦合.但是,它可能并不总是可能的,例如,与第三方库连接时.无论如何…我想首先用测试覆盖现有代码,尽可能少地修改它,然后再深入研究进一步的重构.
解决方法
试试这个：
it("setTitle is invoked after open",function() {
var ordercop = jasmine.createSpy('ordercop');
this.page.open = jasmine.createSpy('openSpy').and.callFake(function() {
ordercop('fisrtInvoke');
});
this.page.setTitle = jasmine.createSpy('setTitleSpy').and.callFake(function() {
ordercop('secondInvoke');
});
this.manager.openSettings();
expect(ordercop.calls.count()).toBe(2);
expect(ordercop.calls.first().args[0]).toBe('firstInvoke');
expect(ordercop.calls.mostRecent().args[0]).toBe('secondInvoke');
}
编辑：我刚刚意识到我的原始答案实际上与你在问题中提到的黑客一样,但是在设置间谍方面有更多的开销.使用“黑客”方式进行操作可能更简单：
it("setTitle is invoked after open",function() {
var ordercop = []
this.page.open = jasmine.createSpy('openSpy').and.callFake(function() {
ordercop.push('fisrtInvoke');
});
this.page.setTitle = jasmine.createSpy('setTitleSpy').and.callFake(function() {
ordercop.push('secondInvoke');
});
this.manager.openSettings();
expect(ordercop.length).toBe(2);
expect(ordercop[0]).toBe('firstInvoke');
expect(ordercop[1]).toBe('secondInvoke');
}