我很难弄清楚如何使用 sinon 来模拟对我正在测试的模块所要求的 postgres 的调用,或者是否有可能。
我不是在尝试测试 postgres 模块本身,只是我的对象以确保它按预期工作,并且它正在调用它在这种情况下应该调用的内容。
我想问题是节点的要求设置,因为我的模块需要 postgres 模块才能访问数据库,但是在这里我不想运行集成测试,我只想确保我的代码独立工作,并不真正关心数据库在做什么,我将把它留给我的集成测试。
我见过一些人设置他们的函数有一个可选参数来发送模拟/存根/假的函数,测试它的存在,如果它存在,在所需的模块上使用它,但这对我来说似乎是一种气味(我是节点的新手,所以也许这不是)。
我宁愿模拟这一点,而不是在可能的情况下尝试劫持需求。
一些代码(请注意,这不是真正的代码,因为我正在使用 TDD 运行并且该函数实际上并没有做任何事情,函数名称是真实的)
测试设置
describe('#execute', function () {
it('should return data rows when executing a select', function(){
//Not sure what to do here
});
});
示例函数
PostgresqlProvider.prototype.execute = function (query, cb) {
var self = this;
if (self.connection === "")
cb(new Error('Connection can not be empty, set Connection using Init function'));
if (query === null)
cb(new Error('Invalid Query Object - Query Object is Null'))
if (!query.buildCommand)
cb(new Error("Invalid Query Object"));
//Valid connection and query
};
像这样包裹 postgres 模块可能看起来有点有趣,但是有一些设计,因为这个应用程序将有几个“提供程序”,我想为它们公开相同的 API,以便我可以互换使用它们。
更新
我认为我的测试太复杂了,因为我想看看是否已经进行了连接调用,然后返回数据,这对我来说很重要,所以我将其剥离并放入两个测试中:
模拟测试
it('should call pg.connect when a valid Query object is parsed', function(){
var mockPg = sinon.mock(pg);
mockPg.expects('connect').once;
Provider.init('ConnectionString');
Provider.execute(stubQueryWithBuildFunc, null, mockPg);
mockPg.verify();
});
这有效(我认为)因为没有 postgres 连接器代码它失败,它通过了(繁荣 :))
现在的问题是第二种方法,我将使用一个存根(可能是一个间谍),当它应该失败时它会通过 100%,所以我会在早上拿起它。
更新 2
我对测试不是 100% 满意,主要是因为我没有劫持 client.query 方法,它是命中数据库的方法,而只是我的 execute 方法并强制它走上一条路径,但它让我看到了结果并反对它以测试行为,但将对任何建议的改进持开放态度。
我正在使用间谍来捕获该方法并返回 null 和一个包含行的人造对象,就像该方法会传回一样,这个测试会随着我添加更多查询行为而改变,但它让我克服了我的障碍。
it('should return data rows when a valid Query object is parsed', function(){
var fauxRows = [
{'id': 1000, 'name':'Some Company A'},
{'id': 1001, 'name':'Some Company B'}
];
var stubPg = sinon.stub(Provider, 'execute').callsArgWith(1, null, fauxRows);
Provider.init('ConnectionString');
Provider.execute(stubQueryWithBuildFunc, function(err, rows){
rows.should.have.length(2);
}, stubPg);
stubPg.called.should.equal.true;
stubPg.restore();
});