9

Nodeunit を使用して Node.js で記述しているモジュールのテスト スイートを作成しようとしています。このモジュールは、プレイリストへのトラックの追加と削除を可能にする基本的な音楽プレイリストです。

var playlist = function(){
    this.__playlist = [];
    this.__count = 0;
};

playlist.prototype = {
    addtrack:function(track){
        if(typeof track !== "object") throw new Error("Track needs to be an oject");
        this.__count++;
        track.id = this.__count;
        this.__playlist.push(track);
        return this.__playlist;
    },
    removetrack:function(trackid){
        if(typeof trackid !== "number") throw new Error("Pass in a numeric track id");
        var trackFound = false;
        for(var i=0;i<this.__playlist.length;i++){
            var t = this.__playlist[i];
            if(t.id == trackid){
                trackFound = true;
                this.__playlist.splice(i,1);
            }
        }
        if(!trackFound) throw new Error("Track not found in the playlist");
        return this.__playlist
    }
}

exports.playlist = function(){
    return new playlist();
}

ご覧のとおり、渡されたデータが正しくないことに基づいてエラーをスローする場所があります。

これが私のテストスイートです。

var pl = require('./playlist');

exports.testPlaylistInit = function(test){
    var playlist = pl.playlist();
    test.equal(typeof playlist, 'object');
    test.done();
}

exports.testAddingTracks = function(test){
    var playlist = pl.playlist();
    test.throws(playlist.addtrack(), Error, 'Should fail for blank track');
    var track = {
        title: "Golly Sandra",
        artist: "Eisley",
        album: "Room Noises"
    };
    tracks = playlist.addtrack(track);
    test.equals(tracks[0],track);
    test.equals(tracks[0].id,1)
    test.done();
}

exports.testRemoveingTracks = function(test){
    var playlist = pl.playlist();
    test.throws(playlist.removetrack('Imma error'), Error, 'Show fail for non-numeric track id');
    var track = {
        title: "Golly Sandra",
        artist: "Eisley",
        album: "Room Noises"
    };
    playlist.addtrack(track);
    track = {
        title: ".44 Calliber Love Letter",
        artist: "Alexisonfire",
        album: "Alexisonfire"
    }
    playlist.addtrack(track);
    test.equals(playlist.removetrack(1)[0],track);
    test.throws(playlist.removetrack(10), Error, 'Should fail for non-existant track');
    test.done();
}

テスト スイートを作成する際、test.throws を使用しました。これは、基本的にコード ブロックを try-catch ステートメントでラップし、catch をエラー ブロックに対してチェックするという仮定に基づいています。Nodeunit でテストを実行すると、エラーをキャッチするテスト スイートではなく、モジュールによってスローされたエラー メッセージが Node に表示されるため、どうやら私は間違っているようです。test.throw ケースを間違って使用していますか?

4

1 に答える 1

24

test. throws の使用法が正しくありません。あなたが持っているものを見ると：

test.throws(
  playlist.removetrack('Imma error'),
  Error,
  'Show fail for non-numeric track id'
);

を実行playlist.removetrack('Imma error')し、その結果を throws に渡すため、例外が発生した場合は、throws が実行される前に発生します。

次のようなことをもっと行う必要があります。

test.throws(
  function() {
    playlist.removetrack('Imma error');
  }, 
  Error,
  'Show fail for non-numeric track id'
);

実行時にトラックを削除しようとする関数を渡す必要があります。そうすることで、プレイリスト ロジックは throws 関数によって実際に実行されるため、try/catch ブロックで自動的にラップできます。

于 2011-08-11T04:10:11.823 に答える