1

我正在尝试为 Node.js 构建一个简单的登录模块。我试图以 TDD 的方式来做，但我还是新手，所以任何可以帮助我更好地理解它的提示或资源都会很棒。

当我用无效数据查询数据库时，我的问题就出现了，我期待一个错误。如果我手动测试应用程序，则会引发错误 - 这很好。但是，当我尝试使用 Mocha 和 Expect.js 对其进行测试时，我得到了Error: expected fn to throw an exception. 如果我将代码从切换to.throwError()到to.not.throwError()错误，则会正确抛出。我认为问题出在我尝试异步测试和错误处理的地方。第一次测试顺利通过。

谢谢参观。

基于 SebastianG 指令的新代码

登录.js

var MongoClient = require('mongodb').MongoClient;

exports.login = function(callback, email, password) {

    MongoClient.connect("mongodb://localhost:27017/stockalertDev", function(err, db) {
        if (err) {

            return err;
        }

        var collection = db.collection('users');
        if (email) {
            collection.findOne({email:email}, function(err, item) {
            try {   
                if (err) {
                    console.log('error');
                    throw new Error('error finding email');

                } else {

                        if (item) {
                            if (item.password == password) {
                            console.log('logged in');
                            //callback(null, item);
                            //return item;
                            } else {
                                console.log('here');
                                throw new Error('Email and password not matching');
                            }
                        } else {
                            throw new Error('Email not found');
                        }
                    } 
            } catch (err) {
                console.log('catch error here');
                callback(err, null);
            } finally {
                console.log('finally here');
                callback(null, item);
            }

            });
        }

    });
}

测试/登录-test.js

var expect = require('expect.js'),
    assert = require('assert'),
    mocha = require('mocha'),
    mongo = require('mongodb');


var login = require('../login');

describe('login', function() {
    it('should login a real user', function(done) {
        expect(function() {
            login.login(function(err, item) {
                //console.log(item);
                if (err) throw err;

                done();
            }, 'email', 'password')
        }).to.not.throwError();
    });
    it('should error on unfound email', function(done) {
        expect(function() {
            login.login(function(err, item) {
                console.log(err);
                if (err) throw err;
                done();
            }, 'ert','wqew')
        }).to.throwError();

    }); 
    it('should error on incorrect match', function(done) {
        expect(function() {
            login.login(function(err, item) {
                console.log(err);
                throw err;
                done();
            }, 'email','wqew')
        }).to.throwError();
    });
});

旧代码

登录.js

var MongoClient = require('mongodb').MongoClient;

exports.login = function(email, password, callback, errCallback) {

    MongoClient.connect("mongodb://localhost:27017/stockalertDev", function(err, db) {
        if (err) {

            return err;
        }

        var collection = db.collection('users');
        if (email) {
            collection.findOne({email:email}, function(err, item) {
            try {   
                if (err) {
                    console.log('error');
                    throw new Error('error finding email');
                    errCallback(err);
                } else {

                        if (item) {
                            if (item.password == password) {
                            console.log('logged in');
                            callback(item);
                            //return item;
                            } else {
                                console.log('here');
                                throw new Error('Email and password not matching');
                            }
                        } else {
                            throw new Error('Email not found');
                        }
                    } 
            } catch (err) {
                errCallback(err);
            }
            });
        }

    });
}

测试/登录-test.js

var expect = require('expect.js'),
    assert = require('assert'),
    mocha = require('mocha'),
    mongo = require('mongodb');


var login = require('../login');

describe('login', function() {
    it('should login a real user', function(done) {
        assert.doesNotThrow(function() {
            login.login('email','password',function() {
                done();
            }, function(err) {
                if (err) throw err;
                done();
            });
        });
    });
    it('should error on unfound email', function(done) {
        expect( function() { 
            login.login('atreq','a', function() {
                console.log('true');        
            }, function(err) {
                console.log(err);
                throw err;
        })}).to.throwError();

    }); 
    it('should error on incorrect match', function(done) {
        expect(function() {
            login.login('email','apassword', function() {
                console.log('true');
                done();
            }, function(err) {
                console.log(err);
                throw err;
            })
        }).to.throwError();
    });
});
4

1 回答 1

4

在异步节点代码中使用异常是一个坏主意（至少目前是这样）。有一个称为域的概念可以帮助您，但还非常具有实验性。

我建议使用 Node 方式：为错误保留回调的第一个参数。小例子：

function getUserData(cb) {
    var userData = // ...
    if (userData === null) {
        cb(new Error('Something bad happend.'));
    } else {
        cb(null, userData)
    }
}

如果您想使用已经使用的 errorCallback：

errCallback(new Error('Email not found'));

你可以做这样的事情（大多数测试框架为此提供了帮助方法，但是我对 Mocha 及其模块并不熟悉）：

it('should login a real user', function(done) {
    login.login(function(err, item) {
        expect(err).to.be(null);
        expect(item).not.to.be(null);
        done();
    }, 'email', 'password');
});


it('should error on unfound email', function(done) {
    login.login(function(err, item) {
        expect(err).not.to.be(null);
        expect(item).to.be(null);
        done();
    }, 'ert','wqew');
});
于 2013-04-13T16:07:00.577 回答