Question

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

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

谢谢你看看。

基于SebastianG指示的新代码

login.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();
    });
});

旧代码

login.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();
    });
});

Answer 1

在异步节点代码中使用exeptions是个坏主意（至少暂时如此）。有一个名为Domains的概念可以帮助你，但它还是非常实验性的。

我建议以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');
});

Node.js：遇到mocha问题并期望to.throwError

1 个答案: