我正在学习如何编写一个针对供应商的REST API的NodeJS模块。编写模块本身的关键代码,但现在我正在尝试学习如何正确地测试它。目前我正在使用MochaJS和ChaiJS作为测试框架。在一个测试中,我创建了一个返回随机ID的用户,我需要保存。然后我想使用所述ID值并测试用户删除。
这是当前不起作用的代码:
var names = require('./names.json');
var ids = [];
describe('users', function() {
describe('addUser', function (){
it('should create ' + names[0].firstname, function (done){
this.slow(3000); this.timeout(10000);
api.addUser(names[0],function(x){
x.should.have.property('id').with.length.of.at.least(2);
ids.push(x.id);
done();
});
});
it('should create ' + names[1].firstname, function (done){
this.slow(3000); this.timeout(10000);
api.addUser(names[1],function(x){
x.should.have.property('activated').and.equal(true);
ids.push(x.id);
done();
});
});
});
describe('deleteUser', function (){
for(var a=0;a<ids.length;a++){
it('should delete ' + ids[a], function (done){
api.deleteUser(ids[a],function(x){
x.should.have.property('id').and.equal(ids[a]);
done();
});
});
}
});
});
即使ids
的范围远远超出测试范围,也不会保存值。现在我已经阅读了关于堆栈溢出的其他评论,其中响应者基本上说“不要重用价值......某些事情是瀑布失败”。我理解,但对我来说,这是预期的功能(TM)。如果由于任何原因(我的代码或供应商API)出现故障而我无法创建用户,那么显然我将无法删除用户。
我想把所有这些都放到Travis CI中,所以我不能指望除非我的测试框架创建,否则特定用户将始终在那里删除。我在供应商系统上的用户数量也很少,所以我需要清理我的测试。还有其他用例(例如修改现有用户)我想测试。
答案 0 :(得分:1)
快速回答是你的for
循环永远不会循环。
当解析测试文件时,for循环(在任何测试运行之前,因此在你可以将任何东西推入ids
之前)执行并因为ids
为空,没有工作要做。
为了证明这一点,请将您的代码调整为:
describe('deleteUser', function (){
console.log("How many IDs?", id);
for(var a=0;a<ids.length;a++){
console.log("This will not be seen...");
it('should delete ' + ids[a], function (done){
api.deleteUser(ids[a],function(x){
x.should.have.property('id').and.equal(ids[a]);
done();
});
});
}
});
解决这个问题的最简单方法是不循环使用ID,而是依次删除这两个用户,然后检查两者是否成功:
var names = require('./names.json');
var ids = [];
describe('users', function() {
describe('addUser', function (){
it('should create ' + names[0].firstname, function (done){
this.slow(3000); this.timeout(10000);
api.addUser(names[0],function(x){
x.should.have.property('id').with.length.of.at.least(2);
ids.push(x.id);
done();
});
});
it('should create ' + names[1].firstname, function (done){
this.slow(3000); this.timeout(10000);
api.addUser(names[1],function(x){
x.should.have.property('activated').and.equal(true);
ids.push(x.id);
done();
});
});
});
describe('deleteUser', function (){
it('should delete users', function (done){
api.deleteUser(ids[0],function(x){
x.should.have.property('id').and.equal(ids[0]);
api.deleteUser(ids[1],function(x){
x.should.have.property('id').and.equal(ids[1]);
done();
});
});
});
});
});
未经测试且无处可去,但应该有效。