1

Node.js 用の単純なログイン モジュールを作成しようとしています。私は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();
    });
});
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 に答える